On Mon, 2008-11-03 at 07:42 -0600, Hyrum K. Wright wrote:
> Julian Foad wrote:
> > On Mon, 2008-11-03 at 11:08 +0000, Julian Foad wrote:
> >> Bug: A repos-repos diff of two or more modified files fails on the
> >> second file with a problem accessing a local temporary file.
> >>
> >> Using trunk_at_34019 over HTTP (neon):
> >>
> >> [[[
> > [...]
> >> diff: /tmp/tempfile.5.tmp: No such file or directory
> >> /home/julianfoad/src/subversion-trunk/subversion/libsvn_subr/io.c:2266:
> >> (apr_err=200012)
> >> svn: '/home/julianfoad/bin/svn-external-diff' returned 2
> >> /home/julianfoad/src/subversion-trunk/subversion/libsvn_ra_neon/util.c:1240: (apr_err=175002)
> >> svn: Error reading spooled REPORT request response
> >> ]]]
> >
> > As you can see, I'm using an external diff. Here's a more direct way to
> > reproduce it, avoiding my "svn-external-diff" script:
> >
> > [[[
> > $ # The directory '/tmp/svn-null-config' does not exist.
> > $ # The external command 'diff' is GNU diff 2.8.7, found in my PATH.
> > $ svn --config-dir /tmp/svn-null-config diff -c34003 $SVN_REPOS
> > --diff-cmd=diff
> > Index: trunk/subversion/include/svn_auth.h
> > ===================================================================
> > --- trunk/subversion/include/svn_auth.h (revision 34002)
> > +++ trunk/subversion/include/svn_auth.h (revision 34003)
> > [...the diff of this first file is fine...]
> > Index: trunk/subversion/libsvn_ra_serf/serf.c
> > ===================================================================
> > diff: /tmp/tempfile.5.tmp: No such file or directory
> > /home/julianfoad/src/subversion-tc3/subversion/libsvn_subr/io.c:2266:
> > (apr_err=200012)
> > svn: 'diff' returned 2
> > /home/julianfoad/src/subversion-tc3/subversion/libsvn_ra_neon/util.c:1240: (apr_err=175002)
> > svn: Error reading spooled REPORT request response
> > ]]]
>
> If I had to guess, I'd say that it's related to r33977. All of the changes in
> that rev were supposed to be idempotent, but that may not be true. (Or,
> something was relying upon implementation-specific functionality which wasn't in
> the public API.)
Testing...
r33977 exhibits this bug.
r33976 does not.
Looks like you're right. Thanks.
- Julian
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-11-03 15:47:19 CET