]> git.ipfire.org Git - thirdparty/git.git/commit - git-cvsimport.perl
merge: deprecate 'git merge <message> HEAD <commit>' syntax
authorJunio C Hamano <gitster@pobox.com>
Thu, 26 Mar 2015 04:58:45 +0000 (21:58 -0700)
committerJunio C Hamano <gitster@pobox.com>
Wed, 29 Apr 2015 20:28:10 +0000 (13:28 -0700)
commitd45366e8aa922037e7e84c3f35924d2b1399a453
treebe02e30d9f0719dcbbc227bcf55dfee32bf3de25
parent74e8bc59cb324d2d7a55c90195db004219770eec
merge: deprecate 'git merge <message> HEAD <commit>' syntax

We had this in "git merge" manual for eternity:

    'git merge' <msg> HEAD <commit>...

    [This] syntax (<msg> `HEAD` <commit>...) is supported for
    historical reasons.  Do not use it from the command line or in
    new scripts.  It is the same as `git merge -m <msg> <commit>...`.

With the update to "git merge" to make it understand what is
recorded in FETCH_HEAD directly, including Octopus merge cases, we
now can rewrite the use of this syntax in "git pull" with a simple
"git merge FETCH_HEAD".

Also there are quite a few fallouts in the test scripts, and it
turns out that "git cvsimport" also uses this old syntax to record
a merge.

Judging from this result, I would not be surprised if dropping the
support of the old syntax broke scripts people have written and been
relying on for the past ten years.  But at least we can start the
deprecation process by throwing a warning message when the syntax is
used.

With luck, we might be able to drop the support in a few years.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin/merge.c
git-cvsimport.perl
git-pull.sh
t/t3402-rebase-merge.sh
t/t6020-merge-df.sh
t/t6021-merge-criss-cross.sh
t/t9402-git-cvsserver-refs.sh