# [PATCH] git-svn: sort the options in the --help message.

2 messages from 2007-11-03 to 2007-11-03. Participants: Benoit Sigoure, Eric Wong.
Thread: https://gitlist.dev/t/10624

## Benoit Sigoure, 2007-11-03 18:53

Subject: [PATCH] git-svn: sort the options in the --help message.
Message-ID: <1194116014-13964-1-git-send-email-tsuna@lrde.epita.fr>
URL: https://gitlist.dev/e/1194116014-13964-1-git-send-email-tsuna%40lrde.epita.fr

```
	* git-svn.perl (&usage): Sort the various options.

Signed-off-by: Benoit Sigoure <tsuna@lrde.epita.fr>
---
It always annoys me when I read the output of git svn <cmd> --help that the
options are printed in no particular order.

 git-svn.perl |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/git-svn.perl b/git-svn.perl
index 22bb47b..4900f57 100755
--- a/git-svn.perl
+++ b/git-svn.perl
@@ -252,7 +252,7 @@ Usage: $0 <command> [options] [arguments]\n
 		next if $cmd && $cmd ne $_;
 		next if /^multi-/; # don't show deprecated commands
 		print $fd '  ',pack('A17',$_),$cmd{$_}->[1],"\n";
-		foreach (keys %{$cmd{$_}->[2]}) {
+		foreach (sort keys %{$cmd{$_}->[2]}) {
 			# mixed-case options are for .git/config only
 			next if /[A-Z]/ && /^[a-z]+$/i;
 			# prints out arguments as they should be passed:
-- 
1.5.3.4.398.g859b

```

## Eric Wong, 2007-11-03 19:42

Subject: Re: [PATCH] git-svn: sort the options in the --help message.
Message-ID: <20071103194212.GA28755@muzzle>
URL: https://gitlist.dev/e/20071103194212.GA28755%40muzzle
In-Reply-To: <1194116014-13964-1-git-send-email-tsuna@lrde.epita.fr>

```
Benoit Sigoure <tsuna@lrde.epita.fr> wrote:
> 	* git-svn.perl (&usage): Sort the various options.
> 
> Signed-off-by: Benoit Sigoure <tsuna@lrde.epita.fr>
> ---
> It always annoys me when I read the output of git svn <cmd> --help that the
> options are printed in no particular order.

The above sentence belongs in the commit message.

Thanks,

Acked-by: Eric Wong <normalperson@yhbt.net>

>  git-svn.perl |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/git-svn.perl b/git-svn.perl
> index 22bb47b..4900f57 100755
> --- a/git-svn.perl
> +++ b/git-svn.perl
> @@ -252,7 +252,7 @@ Usage: $0 <command> [options] [arguments]\n
>  		next if $cmd && $cmd ne $_;
>  		next if /^multi-/; # don't show deprecated commands
>  		print $fd '  ',pack('A17',$_),$cmd{$_}->[1],"\n";
> -		foreach (keys %{$cmd{$_}->[2]}) {
> +		foreach (sort keys %{$cmd{$_}->[2]}) {
>  			# mixed-case options are for .git/config only
>  			next if /[A-Z]/ && /^[a-z]+$/i;
>  			# prints out arguments as they should be passed:

-- 
Eric Wong

```