Re: [PATCH v2] Documentation/diff-options: reword description of --submodule option

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

 



Tim Henigan <tim.henigan@xxxxxxxxx> writes:

> The previous description was confusing.  This rewrite makes it easier
> to understand.
>
> Signed-off-by: Tim Henigan <tim.henigan@xxxxxxxxx>
> ---
>
> Changes in v2:
>   - v1 was a series of two patches. The first was a simple typo fix
>     (s/Chose/Choose/). v2 eliminates the first commit as it is no longer
>     needed.
>   - Changed wording again based on direction from Junio Hamano.
>   - Removed explanation of formats from the commit message. The diff
>     now adequately explains how they work.

Thanks for a credit, but I rarely if ever give "direction". I just mumble
my observations, and the readers realize improvements for themselves ;-)

I'll squash the "submodule summary" (as opposed to "submodule --summary")
fix Jens mentioned in the other thread and apply.

Thanks.

>  Documentation/diff-options.txt |   11 ++++++-----
>  1 file changed, 6 insertions(+), 5 deletions(-)
>
> diff --git a/Documentation/diff-options.txt b/Documentation/diff-options.txt
> index 7d4566f..6aa0907 100644
> --- a/Documentation/diff-options.txt
> +++ b/Documentation/diff-options.txt
> @@ -165,11 +165,12 @@ any of those replacements occurred.
>  	of the `--diff-filter` option on what the status letters mean.
>  
>  --submodule[=<format>]::
> -	Chose the output format for submodule differences. <format> can be one of
> -	'short' and 'log'. 'short' just shows pairs of commit names, this format
> -	is used when this option is not given. 'log' is the default value for this
> -	option and lists the commits in that commit range like the 'summary'
> -	option of linkgit:git-submodule[1] does.
> +	Specify how differences in submodules are shown.  When `--submodule`
> +	or `--submodule=log` is given, the 'log' format is used.  This format lists
> +	the commits in the range like linkgit:git-submodule[1] `--summary` does.
> +	Omitting the `--submodule` option or specifying `--submodule=short`,
> +	uses the 'short' format. This format just shows the names of the commits
> +	at the beginning and end of the range.
>  
>  --color[=<when>]::
>  	Show colored diff.
--
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]