On Fri, Oct 24, 2008 at 02:15:44AM -0700, Eric Wong wrote:
> Deskin Miller <deskinm@xxxxxxxxx> wrote:
> > ---
> > When testing git-svn changes, I'll often use git-svn.perl directly on a
> > repository of mine, to see how things work. Recently I had this happen:
> >
> > $ git-svn.perl info
> > Use of uninitialized value in scalar chomp at ../git/git-svn.perl line 1202.
> > Use of uninitialized value in concatenation (.) or string at ../git/git-svn.perl line 1203.
> >
> > but the expected output showed up correctly, giving the SVN-like info.
> >
> > git svn info appeared to work just fine though, despite there being essentially
> > no difference between the two. However, if I do the following, this is what I
> > see:
> >
> > $ env /usr/libexec/git-core/git-svn info
> > Can't exec "git-config": No such file or directory at /usr/libexec/git-core/git-svn line 1202.
> > Use of uninitialized value in scalar chomp at /usr/libexec/git-core/git-svn line 1202.
> >
> > Again, the SVN info would still appear.
> >
> > Regardless, I think we want to change this call to use the non-dashed form.
>
> Thanks Deskin,
>
> Acked-by: Eric Wong <normalperson@xxxxxxxx>
Sorry, patch should have had a
Signed-off-by: Deskin Miller <deskinm@xxxxxxxxx>
Feel free to forge it.
> > Deskin Miller
> >
> > git-svn.perl | 2 +-
> > 1 files changed, 1 insertions(+), 1 deletions(-)
> >
> > diff --git a/git-svn.perl b/git-svn.perl
> > index 33e1b50..2e68c68 100755
> > --- a/git-svn.perl
> > +++ b/git-svn.perl
> > @@ -1126,7 +1126,7 @@ sub read_repo_config {
> > my $v = $opts->{$o};
> > my ($key) = ($o =~ /^([a-zA-Z\-]+)/);
> > $key =~ s/-//g;
> > - my $arg = 'git-config';
> > + my $arg = 'git config';
> > $arg .= ' --int' if ($o =~ /[:=]i$/);
> > $arg .= ' --bool' if ($o !~ /[:=][sfi]$/);
> > if (ref $v eq 'ARRAY') {
> > --
> > 1.6.0.2.554.g3041b
--
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
