Re: [PATCH] Man page section reference should be in uppercase

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

 



2010. 05. 25, kedd keltezéssel 11.01-kor Nazri Ramliy ezt írta:
> From: nazri <ayiehere@xxxxxxxxx>
> 
> The advice given by git push when doing a non-fast-forward push refers
> to the section 'Note about fast-forwards' in the git-push(1) man page.
> 
> The section name should really be all in uppercase, as in 'NOTE ABOUT
> FAST-FORWARDS' because that is how it is shown by git push --help.
> 

I don't really agree with this, but it might be only my personal
preference. Reasons:

The section title in the man page is "typeset" in a "different font" to
be easily distinguishable from the surrounding text -- therefore it's
all capital. On the other hand, changing the reference in the advice
given by git-push to be all uppercase will give a needless emphasis on
the referred section title. This is wrong IMHO, because the main point
of the advice is not this reference, but the earlier sentences (i.e. the
push was rejected).

> Signed-off-by: Nazri Ramliy <ayiehere@xxxxxxxxx>
> ---
>  builtin/push.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/builtin/push.c b/builtin/push.c
> index f4358b9..0694200 100644
> --- a/builtin/push.c
> +++ b/builtin/push.c
> @@ -130,8 +130,8 @@ static int push_with_options(struct transport *transport, int flags)
>  
>  	if (nonfastforward && advice_push_nonfastforward) {
>  		fprintf(stderr, "To prevent you from losing history, non-fast-forward updates were rejected\n"
> -				"Merge the remote changes before pushing again.  See the 'Note about\n"
> -				"fast-forwards' section of 'git push --help' for details.\n");
> +				"Merge the remote changes before pushing again.  See the 'NOTE ABOUT\n"
> +				"FAST-FORWARDS' section of 'git push --help' for details.\n");
>  	}
>  
>  	return 1;

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