# [PATCH] git-svn: quiet some warnings when run only with --version/--help

3 messages from 2007-04-17 to 2007-04-17. Participants: Eric Wong, Junio C Hamano.
Thread: https://gitlist.dev/t/7714

## Eric Wong, 2007-04-17 09:41

Subject: [PATCH] git-svn: quiet some warnings when run only with --version/--help
Message-ID: <20070417094143.GB2778@muzzle>
URL: https://gitlist.dev/e/20070417094143.GB2778%40muzzle

```
These are harmless but annoying.  They were introduced in
512b620bd9fef7f170562ecad835e37479f051ce

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

 Junio: did you miss
 http://permalink.gmane.org/gmane.comp.version-control.git/44493 ?

 Thanks,

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

diff --git a/git-svn.perl b/git-svn.perl
index eb3b79c..d2a5fe4 100755
--- a/git-svn.perl
+++ b/git-svn.perl
@@ -168,14 +168,14 @@ for (my $i = 0; $i < @ARGV; $i++) {
 my %opts = %{$cmd{$cmd}->[2]} if (defined $cmd);
 
 read_repo_config(\%opts);
-Getopt::Long::Configure('pass_through') if $cmd eq 'log';
+Getopt::Long::Configure('pass_through') if ($cmd && $cmd eq 'log');
 my $rv = GetOptions(%opts, 'help|H|h' => \$_help, 'version|V' => \$_version,
                     'minimize-connections' => \$Git::SVN::Migration::_minimize,
                     'id|i=s' => \$Git::SVN::default_ref_id,
                     'svn-remote|remote|R=s' => sub {
                        $Git::SVN::no_reuse_existing = 1;
                        $Git::SVN::default_repo_id = $_[1] });
-exit 1 if (!$rv && $cmd ne 'log');
+exit 1 if (!$rv && $cmd && $cmd ne 'log');
 
 usage(0) if $_help;
 version() if $_version;

-- 
Eric Wong

```

## Junio C Hamano, 2007-04-17 09:46

Subject: Re: [PATCH] git-svn: quiet some warnings when run only with --version/--help
Message-ID: <7vvefvpc3b.fsf@assigned-by-dhcp.cox.net>
URL: https://gitlist.dev/e/7vvefvpc3b.fsf%40assigned-by-dhcp.cox.net
In-Reply-To: <20070417094143.GB2778@muzzle>

```
Eric Wong <normalperson@yhbt.net> writes:

> These are harmless but annoying.  They were introduced in
> 512b620bd9fef7f170562ecad835e37479f051ce
>
> Signed-off-by: Eric Wong <normalperson@yhbt.net>
> ---
>
>  Junio: did you miss
>  http://permalink.gmane.org/gmane.comp.version-control.git/44493 ?

Indeed.  Are these two 'maint' material?

```

## Eric Wong, 2007-04-17 10:28

Subject: Re: [PATCH] git-svn: quiet some warnings when run only with --version/--help
Message-ID: <20070417102818.GA20984@hand.yhbt.net>
URL: https://gitlist.dev/e/20070417102818.GA20984%40hand.yhbt.net
In-Reply-To: <7vvefvpc3b.fsf@assigned-by-dhcp.cox.net>

```
Junio C Hamano <junkio@cox.net> wrote:
> Eric Wong <normalperson@yhbt.net> writes:
> 
> > These are harmless but annoying.  They were introduced in
> > 512b620bd9fef7f170562ecad835e37479f051ce
> >
> > Signed-off-by: Eric Wong <normalperson@yhbt.net>
> > ---
> >
> >  Junio: did you miss
> >  http://permalink.gmane.org/gmane.comp.version-control.git/44493 ?
> 
> Indeed.  Are these two 'maint' material?

Yes.  Definitely so.

-- 
Eric Wong

```