Am Donnerstag, 10. Januar 2008 21:08:52 schrieb Junio C Hamano:
quoted text
> Dennis Schridde <devurandom@gmx.net> writes:
> >> [svn-remote "svn"]
> >>     reposRoot =3D file:///var/svn/warzone2100
> >>     uuid =3D 4a71c877-e1ca-e34f-864e-861f7616d084
> >>     branches-maxRev =3D 14
> >>     tags-maxRev =3D 14
> >>     svnsync-uuid =3D 4a71c877-e1ca-e34f-864e-861f7616d084\n
> >>     svnsync-url =3D http://svn.gna.org/svn/warzone
> >> [svn-remote "tags/1.10a.12"]
> >>     reposRoot =3D file:///var/svn/warzone2100
> >>     uuid =3D 4a71c877-e1ca-e34f-864e-861f7616d084
> >> ---
> >
> > The rest of the file is rather boring. The "svn" remote is not changed
> > (besides having higher revisions) and the other remotes look exactly li=
ke
quoted text
> > the "tags/1.10a.12" one.
> >
> > Somehow I think that the \n at the end of the svnsync-uuid shouldn't be
> > there... It could be that this is the same linebreak which prevents
> > people from relocating (svn switch --relocate) from
> > svn://svn.gna.org/svn/warzone to http://svn.gna.org/svn/warzone, so that
> > would be a Gna bug.
> > However git-svn shouldn't throw any warnings (or even (make perl) crash=
?)
quoted text
> > on such occasions, either...
> >
> > I now got it to run through without a segfault, by compiling an unstrip=
ed
quoted text
> > perl binary with debug symbols (Gentoo: FEATURES=3Dnostrip CFLAGS=3D"...
> > -g"). Maybe this is a bug in GCC or something...
> >
> > The "Odd number of elements in anonymous hash" still stays, though.
>
> The code in question is:
>
> 	my $svnsync;
> 	# see if we have it in our config, first:
> 	eval {
> 		my $section =3D "svn-remote.$self->{repo_id}";
> 		$svnsync =3D {
> 		  url =3D> tmp_config('--get', "$section.svnsync-url"),
> 		  uuid =3D> tmp_config('--get', "$section.svnsync-uuid"),
> 		}
> 	};
>
> I think the "Odd number" is an indication that one of the
> tmp_config() calls is returning an even number of elements (so
> the hash whose ref will be stored in $svnsync ends up having an
> odd number of elements), and that is why I initially asked you
> about "more than one" svnsync-url.  0 is also an even number,
> and it could be that it is not finding any.
>
> How about doing something ugly like this _just for diagnosis_?
>
> 	my $svnsync;
> 	# see if we have it in our config, first:
> 	eval {
> 		my $section =3D "svn-remote.$self->{repo_id}";
> 		my @u =3D tmp_config('--get', "$section.svnsync-url");
> 		my @v =3D tmp_config('--get', "$section.svnsync-uuid");
> 		if (@u !=3D 1 || @v !=3D 1) {
>                 	print STDERR "Oops: <$section> $#u <@u> $#v <@v>\n";
> 		}
> 		$svnsync =3D {
> 		  url =3D> @u,
> 		  uuid =3D> @v,
> 		}
> 	};
I've created /usr/bin/git-svndbg and changed that part, like you proposed.
I now get this output. (As it continues to run, there are probably more=20
occassions of the Oops.)
=2D--
Oops: <svn-remote.svn> 0 <http://svn.gna.org/svn/warzone> 1=20
<4a71c877-e1ca-e34f-864e-861f7616d084 >
Odd number of elements in anonymous hash at /usr/bin/git-svndbg line 1768.
r13 =3D ee6d5a48dd5cf1a96ed5217d638f372d2c173d89 (tags/1.10a)
=2D--

=2D-Dennis