On 23 Jan 2003, Karl Fogel wrote:
Karl Fogel email@example.com writes:
Hmmm. We very recently added this test; before 4495, there was no
regression test for the `--revprop' functionality.
But why would it fail on RedHat 7.3 and not 8.0? Strange. If you can
debug further, that would be great...
Heh. David, I think this post from Eric Gillespie is probably
relevant to our discussion:
From: Eric Gillespie firstname.lastname@example.org
Subject: [PATCH] svn_ra_svn_write_cmd format string bug
Date: Thu, 23 Jan 2003 20:36:06 -0500
svn was segfaulting in prop_tests 11 using ra_svn. I stepped
through it with gdb and found this bug. The test tries to set a
property on revision 0. The svn_revnum_t is cast to
apr_uint64_t, where it somehow turned from 0 into
The patch fixes that, but the test still fails due to the
apr_proc_wait issue i raised elsewhere (when i change
apr_proc_wait to check for EINTR and loop it passess, marking the
first time i've had the entire test suite pass over ra_svn).
--- subversion/libsvn_ra_svn/client.c (revision 4549)
+++ subversion/libsvn_ra_svn/client.c (working copy)
@@ -424,7 +424,7 @@
svn_ra_svn_conn_t *conn = sess;
apr_pool_t *pool = conn-pool;
- SVN_ERR(svn_ra_svn_write_cmd(conn, pool, change-rev-prop, ncs,
+ SVN_ERR(svn_ra_svn_write_cmd(conn, pool, change-rev-prop, rcs,
rev, name, value));
SVN_ERR(svn_ra_svn_read_cmd_response(conn, pool, ));
This patch fixed the problem on RedHat 7.3.
I didn't run tests over ra_dav, I hope to figure out how to do that in the
RPM this weekend.
David Wayne Summers Linux: Because reboots are for hardware upgrades!
david_at_summersoft.fay.ar.us PGP Key: http://summersoft.fay.ar.us/~david/pgp.txt
PGP Key fingerprint = C0 E0 4F 50 DD A9 B6 2B 60 A1 31 7E D2 28 6D A8
To unsubscribe, e-mail: email@example.com
For additional commands, e-mail: firstname.lastname@example.org
Received on Sat Oct 14 02:12:38 2006