Re: [PATCH 01/13] Use 'Git' in help messages

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 23/02/13 23:41, David Aguilar wrote:
On Sat, Feb 23, 2013 at 3:05 PM, Philip Oakley <philipoakley@xxxxxxx> wrote:
Signed-off-by: Philip Oakley <philipoakley@xxxxxxx>
---
  help.c | 12 ++++++------
  1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/help.c b/help.c
index 1dfa0b0..1c0e17d 100644
--- a/help.c
+++ b/help.c
@@ -209,14 +209,14 @@ void list_commands(unsigned int colopts,
  {
         if (main_cmds->cnt) {
                 const char *exec_path = git_exec_path();
-               printf_ln(_("available git commands in '%s'"), exec_path);
+               printf_ln(_("available Git commands in '%s'"), exec_path);
                 putchar('\n');
                 pretty_print_string_list(main_cmds, colopts);
                 putchar('\n');
         }

         if (other_cmds->cnt) {
-               printf_ln(_("git commands available from elsewhere on your $PATH"));
+               printf_ln(_("Git commands available from elsewhere on your $PATH"));
                 putchar('\n');
                 pretty_print_string_list(other_cmds, colopts);
                 putchar('\n');
@@ -232,7 +232,7 @@ void list_common_cmds_help(void)
                         longest = strlen(common_cmds[i].name);
         }

-       puts(_("The most commonly used git commands are:"));
+       puts(_("The most commonly used Git commands are:"));
         for (i = 0; i < ARRAY_SIZE(common_cmds); i++) {
                 printf("   %s   ", common_cmds[i].name);
                 mput_char(' ', longest - strlen(common_cmds[i].name));
@@ -289,7 +289,7 @@ static void add_cmd_list(struct cmdnames *cmds, struct cmdnames *old)
  #define SIMILAR_ENOUGH(x) ((x) < SIMILARITY_FLOOR)

  static const char bad_interpreter_advice[] =
-       N_("'%s' appears to be a git command, but we were not\n"
+       N_("'%s' appears to be a Git command, but we were not\n"
         "able to execute it. Maybe git-%s is broken?");

  const char *help_unknown_cmd(const char *cmd)
@@ -380,7 +380,7 @@ const char *help_unknown_cmd(const char *cmd)
                 return assumed;
         }

-       fprintf_ln(stderr, _("git: '%s' is not a git command. See 'git --help'."), cmd);
+       fprintf_ln(stderr, _("git: '%s' is not a Git command. See 'git --help'."), cmd);

         if (SIMILAR_ENOUGH(best_similarity)) {
                 fprintf_ln(stderr,
@@ -397,6 +397,6 @@ const char *help_unknown_cmd(const char *cmd)

  int cmd_version(int argc, const char **argv, const char *prefix)
  {
-       printf("git version %s\n", git_version_string);
+       printf("Git version %s\n", git_version_string);
         return 0;
  }

This is referring to "git the command", not "git the system",
so it should not be changed according to the rule that was
applied when many "git" strings were changed to "Git".

I'd felt that they were referring to 'Git the system', hence the changes.

There are scripts, etc. in the wild that parse this output.

My first pass avoided the 'git --version' response on the basis
that it might be used in the tests or elsewhere, but I didn't
find an occurrence of it's use (in Git), so I thought 'why not'.
However I'm not wedded to it. If the exact phrase 'git version'
is parsed in the wild it then we should let it be.

which is another reason we would not want to change this.

Does the build system use this output somewhere?

The other strings in this patch mention "git commands" which
the user is expected to type, so it might make sense to leave
them as-is as well.


--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]