Re: [PATCH 1/2 resend] Documentation: user-manual: add information about "git help" at the beginning

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

 



Christian Couder <chriscool@xxxxxxxxxxxxx> writes:

> Talking about "git help" is useful because it has a few more
> features (like when using it without arguments or with "-a") and
> it may work on non unix like platforms.

First of all, I disagree with your idea to advocate "git help" as _the
first way_ to read the manual pages.  The way the current user manual is
organized is to help command line users, and _the_ way to get to the
manual is through the "man" command.  Yes, it is very nice of us to
provide them _other ways_ to get to them, but that is additional frill.
IOW, we _should not_ give impression that you have to use "git" (and "git
help") to get to the documentation.

> diff --git a/Documentation/user-manual.txt b/Documentation/user-manual.txt
> index 645d752..48f7189 100644
> --- a/Documentation/user-manual.txt
> +++ b/Documentation/user-manual.txt
> @@ -17,13 +17,27 @@ People needing to do actual development will also want to read
>  
>  Further chapters cover more specialized topics.
>  
> -Comprehensive reference documentation is available through the man
> -pages.  For a command such as "git clone <repo>", just use
> +Comprehensive reference documentation is available through either the
> +linkgit:git-help[1] command or the man pages.  For a command such as
> +"git clone <repo>", you can use:
> +
> +------------------------------------------------
> +$ git help clone
> +------------------------------------------------
> +
> +or:
>  
>  ------------------------------------------------
>  $ man git-clone
>  ------------------------------------------------

I am not opposed to mentioning "git help" in the early part of the manual,
but for the above stated reason, I'd rather swap the above:

	... through the man pages, or linkgit:git-help[1] command.  For
	example, for the command "git clone <repo>", you can either use:

	------------
        $ man git-clone
        ------------

	or:

	------------
        $ git help clone
        ------------

This is a very early part of the user manual, and it would be a good idea
to tell the user about the presense of the comprehensive set of manual
pages.  I also think it is a good idea to tell them that they do not have
to use "man" but they can also use "git help", and hint that the "git
help" is a separate command (which is already done with the above
rewording).

However, this part troubles me heavily in a few ways:

> +linkgit:git-help[1] has a few more features and is self-documenting
> +using:

 - "A few more features" may be good in the commit log message for this
   change, but look out of place here.  What additional benefit are the
   readers of the user manual getting here?

   You already told them that a separate "git help" command is available,
   so they can learn how to use it by either "man git-help" or "git help
   help" if they are interested (and I do not think we need a separate
   example for that, after we've shown the use of these two ways for "git
   clone").

 - "self-documenting" is like saying git is self documenting, and/or there
   is a documentation.  Again, I find this an additional noise that does
   not help the readers very much.

It would probably be less objectionable if instead this part said
something like:

	With the latter, you can use the manual viewer of your choice; see
	linkgit:git-help[1] for more information.

which would be way more informative than "with a few more features" and
"self documenting", I think.  Notice that I said "with the _latter_" --
which is consistent to the order I think man/help should be mentioned in
the paragraph before this part.

--
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]

  Powered by Linux