Re: [PATCH] Change mentions of "git programs" to "git commands"

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

 



Ori Avtalion <ori@xxxxxxxxxxxxx> writes:

> On 08/07/2009 09:15 PM, Junio C Hamano wrote:
>> Ori Avtalion<ori@xxxxxxxxxxxxx>  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.
--
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]