Skip to content

Commit

Permalink
git-merge: a deprecation notice of the ancient command line syntax
Browse files Browse the repository at this point in the history
The ancient form of git merge command used in the original sample script
has been copied from Linus and are still found everywhere, I think, and
people may still have it in their scripts, but on the other hand, it is so
unintuitive that even people reasonably familiar with git are surprised by
accidentally triggering the support to parse this ancient form.

Gently nudge people to upgrade their script to more recent and readable
style for eventual removal of the original syntax.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
  • Loading branch information
Junio C Hamano committed Dec 1, 2009
1 parent 78d553b commit b81e00a
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions builtin-merge.c
Original file line number Diff line number Diff line change
Expand Up @@ -789,6 +789,11 @@ static int suggest_conflicts(void)
return 1;
}

static const char deprecation_warning[] =
"'git merge <msg> HEAD <commit>' is deprecated. Please update\n"
"your script to use 'git merge -m <msg> <commit>' instead.\n"
"In future versions of git, this syntax will be removed.";

static struct commit *is_old_style_invocation(int argc, const char **argv)
{
struct commit *second_token = NULL;
Expand All @@ -802,6 +807,7 @@ static struct commit *is_old_style_invocation(int argc, const char **argv)
die("'%s' is not a commit", argv[1]);
if (hashcmp(second_token->object.sha1, head))
return NULL;
warning(deprecation_warning);
}
return second_token;
}
Expand Down

0 comments on commit b81e00a

Please sign in to comment.