]> git.ipfire.org Git - thirdparty/git.git/commitdiff
error_resolve_conflict: drop quotations around operation
authorJeff King <peff@peff.net>
Tue, 3 Jun 2014 07:23:49 +0000 (03:23 -0400)
committerJunio C Hamano <gitster@pobox.com>
Tue, 3 Jun 2014 17:04:21 +0000 (10:04 -0700)
When you try to commit with unmerged entries, you get an
error like:

  $ git commit
  error: 'commit' is not possible because you have unmerged files.

The quotes around "commit" are clunky; the user doesn't care
that this message is a template with the command-name filled
in.  Saying:

  error: commit is not possible because you have unmerged files

is easier to read. As this code is called from other places,
we may also end up with:

  $ git merge
  error: merge is not possible because you have unmerged files

  $ git cherry-pick foo
  error: cherry-pick is not possible because you have unmerged files

  $ git revert foo
  error: revert is not possible because you have unmerged files

All of which look better without the quotes. This also
happens to match the behavior of "git pull", which generates
a similar message (but does not share code, as it is a shell
script).

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
advice.c

index d458ae77481128680f0cc67aae6845d16ea548d4..07d0109d69364a1e225557d1fd6d5d459c9117fa 100644 (file)
--- a/advice.c
+++ b/advice.c
@@ -78,7 +78,7 @@ int git_default_advice_config(const char *var, const char *value)
 
 int error_resolve_conflict(const char *me)
 {
-       error("'%s' is not possible because you have unmerged files.", me);
+       error("%s is not possible because you have unmerged files.", me);
        if (advice_resolve_conflict)
                /*
                 * Message used both when 'git commit' fails and when