git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Junio C Hamano <gitster@pobox.com>
To: Ori Avtalion <ori@avtalion.name>
Cc: git@vger.kernel.org
Subject: Re: [PATCH] Change mentions of "git programs" to "git commands"
Date: Sat, 08 Aug 2009 09:12:54 -0700	[thread overview]
Message-ID: <7v8whul1x5.fsf@alter.siamese.dyndns.org> (raw)
In-Reply-To: 4A7D7CF2.40902@avtalion.name

Ori Avtalion <ori@avtalion.name> writes:

> On 08/07/2009 09:15 PM, Junio C Hamano wrote:
>> Ori Avtalion<ori@avtalion.name>  writes:
>>
>>> Most of the docs and printouts refer to "commands".
>>> This patch changes the other terminology to be consistent.
>>
>> Thanks, but not really.
>>
>>> diff --git a/Documentation/fetch-options.txt b/Documentation/fetch-options.txt
>>> index d313795..20bf512 100644
>>> --- a/Documentation/fetch-options.txt
>>> +++ b/Documentation/fetch-options.txt
>>> @@ -1,7 +1,7 @@
>>>   -q::
>>>   --quiet::
>>>   	Pass --quiet to git-fetch-pack and silence any other internally
>>> -	used programs.
>>> +	used utilities.
>>
>> This does not have much to do with what you claim to have done in the
>> commit log message nor the title.  Probably "utilities" is a slightly
>> better word than "programs" in this context but not by a wide margin.
>
> If that doesn't fit,...

My comment on this part was not "Don't change this wording, the original
is much better", but was primarily "you said this commit is about
replacing 'program' with 'command' for consistency, but this change does
not belong to that category.  Either make a separate commit for this
change separate from all the rest, or change the wording of the commit log
message to cover this one as well".

>    "Pass --quiet to git-fetch-pack and silence other output" ?
>
>>> -'git-rev-list' is a very essential git program, since it
>>> +'git-rev-list' is a very essential git command, since it
>>>   provides the ability to build and traverse commit ancestry graphs. For
>>>   this reason, it has a lot of different options that enables it to be
>>>   used by commands as different as 'git-bisect' and
>>
>> Ok, but probably we would want to say "git rev-list" here.
> ...
> I see the "git-foo" notation as a convention for git commands.
> It may be for historical reasons, but the manpage for the "pull" git
> command is "git-pull", and that is how commands are mentioned all over
> the documentation.

This was "because you are changing the exact sentence anyway, why not do
this while you are at it".  But you are probably right.  We try to write
"git foo" when we talk about what users would type, but say "git-foo" when
we talk about the command itself.  When we talk about the concept of what
the command achieves, we sometimes say just "foo".

> In git.txt:
> "The following are helper programs used by the above"
>                           ^^^^^^^^
>                           commands
>
> In git-mailsplit.txt (and cmds-purehelpers.txt):
>    "Simple UNIX mbox splitter program."
>
> Maybe the word "program" should just be dropped.

That may be an option and I kind of like it; but I'd rather hear from
others first.

  reply	other threads:[~2009-08-08 16:13 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-08-07 14:24 [PATCH] Change mentions of "git programs" to "git commands" Ori Avtalion
2009-08-07 18:15 ` Junio C Hamano
2009-08-08 12:12   ` Erik Faye-Lund
2009-08-08 13:26   ` Ori Avtalion
2009-08-08 16:12     ` Junio C Hamano [this message]
2009-08-11  3:58 ` Nanako Shiraishi
2009-08-11 11:49   ` Ori Avtalion
2009-08-11 22:59     ` Nanako Shiraishi
2009-08-11 23:29       ` Ori Avtalion
2009-08-13 12:02         ` Ori Avtalion
2009-08-12 22:39   ` Junio C Hamano

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=7v8whul1x5.fsf@alter.siamese.dyndns.org \
    --to=gitster@pobox.com \
    --cc=git@vger.kernel.org \
    --cc=ori@avtalion.name \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).