Re: svn commit: r978841 - in /subversion/trunk/subversion/tests/cmdline: svnrdump_tests.py svnrdump_tests_data/ svnrdump_tests_data/revision0.dump

2010-07-24 Thread Ramkumar Ramachandra
Hi Daniel, Daniel Shahaf writes: > Please don't add "Review by" before I've actually reviewed the patch, thanks > :-) Um, right. I guess I misunderstood you when you asked me to put the 'Review by' in your previous email. I'll include it while actually committing, but not in the email. > Perhap

Re: svn commit: r978841 - in /subversion/trunk/subversion/tests/cmdline: svnrdump_tests.py svnrdump_tests_data/ svnrdump_tests_data/revision0.dump

2010-07-24 Thread Ramkumar Ramachandra
Hi Daniel, Daniel Shahaf writes: > Not your fault, but that's not what I meant. What I meant was to check for no > unexpected stderr (e.g., no "svn: warning %s" or similar). > > For example, you could do that by running 'svnrdump -q' and then verifying > that > *nothing* was printed to stderr.

svn commit: r978902 - /subversion/trunk/subversion/libsvn_client/changelist.c

2010-07-24 Thread danielsh
Author: danielsh Date: Sat Jul 24 17:04:04 2010 New Revision: 978902 URL: http://svn.apache.org/viewvc?rev=978902&view=rev Log: Make 'svn cl --remove -R' not spam stdout. * subversion/libsvn_client/changelist.c (set_node_changelist): Suppress notifications when removing a dir from a changel

svn commit: r978859 - /subversion/branches/atomic-revprop/BRANCH-README

2010-07-24 Thread danielsh
Author: danielsh Date: Sat Jul 24 12:03:09 2010 New Revision: 978859 URL: http://svn.apache.org/viewvc?rev=978859&view=rev Log: On the 'atomic-revprop' branch: * BRANCH-README (svn:eol-style): Set to 'native'. Modified: subversion/branches/atomic-revprop/BRANCH-README (contents, props cha

svn commit: r978858 - /subversion/branches/atomic-revprop/BRANCH-README

2010-07-24 Thread danielsh
Author: danielsh Date: Sat Jul 24 12:02:13 2010 New Revision: 978858 URL: http://svn.apache.org/viewvc?rev=978858&view=rev Log: On the 'atomic-revprop' branch, update BRANCH-README. Modified: subversion/branches/atomic-revprop/BRANCH-README Modified: subversion/branches/atomic-revprop/BRANCH

svn propchange: r978842 - svn:log

2010-07-24 Thread danielsh
Author: danielsh Revision: 978842 Modified property: svn:log Modified: svn:log at Sat Jul 24 11:57:00 2010 -- --- svn:log (original) +++ svn:log Sat Jul 24 11:57:00 2010 @@ -1,4 +1,4 @@ -On the 'atomic-revprop branch': +On

svn propchange: r978843 - svn:log

2010-07-24 Thread danielsh
Author: danielsh Revision: 978843 Modified property: svn:log Modified: svn:log at Sat Jul 24 11:56:47 2010 -- --- svn:log (original) +++ svn:log Sat Jul 24 11:56:47 2010 @@ -1,4 +1,4 @@ -On the 'atomic-revprop' branch +On

svn commit: r978856 - /subversion/trunk/subversion/tests/cmdline/diff_tests.py

2010-07-24 Thread dannas
Author: dannas Date: Sat Jul 24 11:55:17 2010 New Revision: 978856 URL: http://svn.apache.org/viewvc?rev=978856&view=rev Log: Add XFailing test for a property diff that has missing intermediate context lines. Runnning: diff -u <(svn pg prop i...@base) <(svn pg prop iota) gives the right result

svn commit: r978855 - in /subversion/branches/atomic-revprop/subversion/tests/cmdline: atomic-ra-revprop-change.c svntest/actions.py svntest/main.py

2010-07-24 Thread danielsh
Author: danielsh Date: Sat Jul 24 11:53:27 2010 New Revision: 978855 URL: http://svn.apache.org/viewvc?rev=978855&view=rev Log: On the 'atomic-revprop' branch: Teach the wrapper utility to respect HTTP_LIBRARY. * subversion/tests/cmdline/svntest/actions.py (run_and_verify_atomic_ra_revprop_cha

svn commit: r978853 - in /subversion/branches/atomic-revprop: ./ subversion/include/ subversion/libsvn_subr/ subversion/tests/cmdline/ subversion/tests/cmdline/svnrdump_tests_data/

2010-07-24 Thread danielsh
Author: danielsh Date: Sat Jul 24 11:47:30 2010 New Revision: 978853 URL: http://svn.apache.org/viewvc?rev=978853&view=rev Log: On the 'atomic-revprop' branch, merge r967266 through r978852 from trunk. Added: subversion/branches/atomic-revprop/subversion/tests/cmdline/svnrdump_tests_data/

svn commit: r978851 - in /subversion/trunk/subversion: include/svn_config.h libsvn_subr/config.c

2010-07-24 Thread danielsh
Author: danielsh Date: Sat Jul 24 11:41:59 2010 New Revision: 978851 URL: http://svn.apache.org/viewvc?rev=978851&view=rev Log: Add a constructor for svn_config_t. (The only existing constructor was svn_config_read(), but it required passing a filepath.) * subversion/libsvn_subr/config.c (svn_

svn commit: r978852 - /subversion/trunk/subversion/tests/cmdline/svnrdump_tests.py

2010-07-24 Thread artagnon
Author: artagnon Date: Sat Jul 24 11:42:01 2010 New Revision: 978852 URL: http://svn.apache.org/viewvc?rev=978852&view=rev Log: * subversion/tests/cmdline/svnrdump_tests.py (run_test): Run svnrdump with '-q' and check that nothing is printed to stderr. Review by: danielsh Approved by: danielsh

Re: svn commit: r978841 - in /subversion/trunk/subversion/tests/cmdline: svnrdump_tests.py svnrdump_tests_data/ svnrdump_tests_data/revision0.dump

2010-07-24 Thread Daniel Shahaf
Ramkumar Ramachandra wrote on Sat, Jul 24, 2010 at 16:58:34 +0530: > Hi Daniel, > > Daniel Shahaf writes: > > Please don't add "Review by" before I've actually reviewed the patch, > > thanks :-) > > Um, right. I guess I misunderstood you when you asked me to put the > 'Review by' in your previou

svn propchange: r978841 - svn:log

2010-07-24 Thread stsp
Author: stsp Revision: 978841 Modified property: svn:log Modified: svn:log at Sat Jul 24 11:22:05 2010 -- --- svn:log (original) +++ svn:log Sat Jul 24 11:22:05 2010 @@ -13,5 +13,5 @@ properties, so it's somewhat special.

Re: svn commit: r978841 - in /subversion/trunk/subversion/tests/cmdline: svnrdump_tests.py svnrdump_tests_data/ svnrdump_tests_data/revision0.dump

2010-07-24 Thread Daniel Shahaf
Ramkumar Ramachandra wrote on Sat, Jul 24, 2010 at 16:28:49 +0530: > Hi Daniel, > > Daniel Shahaf writes: > > Not your fault, but that's not what I meant. What I meant was to check for > > no > > unexpected stderr (e.g., no "svn: warning %s" or similar). > > > > For example, you could do that b

svn commit: r978843 - in /subversion/branches/atomic-revprop/subversion: include/ libsvn_ra/ libsvn_ra_local/ libsvn_ra_neon/ libsvn_ra_serf/ libsvn_ra_svn/ tests/cmdline/

2010-07-24 Thread danielsh
Author: danielsh Date: Sat Jul 24 10:39:26 2010 New Revision: 978843 URL: http://svn.apache.org/viewvc?rev=978843&view=rev Log: On the 'atomic-revprop' branch Add a skeleton RA interface, but do not implement it yet. (This revision only does the "Revv the vtable member" dance.) First, add the

Re: svn commit: r978841 - in /subversion/trunk/subversion/tests/cmdline: svnrdump_tests.py svnrdump_tests_data/ svnrdump_tests_data/revision0.dump

2010-07-24 Thread Daniel Shahaf
artag...@apache.org wrote on Sat, Jul 24, 2010 at 10:18:58 -: > + # Create a dump file using svnrdump > + r, svnrdump_dumpfile, err = svntest.main.run_svnrdump(sbox.repo_url) > + > + # Check error code > + if (r != 0): > +raise svntest.Failure('Result code not 0') > + > + # Check the o

svn commit: r978842 - /subversion/branches/atomic-revprop/subversion/tests/cmdline/atomic-ra-revprop-change.c

2010-07-24 Thread danielsh
Author: danielsh Date: Sat Jul 24 10:22:23 2010 New Revision: 978842 URL: http://svn.apache.org/viewvc?rev=978842&view=rev Log: On the 'atomic-revprop branch': Unbreak the build. * subversion/tests/cmdline/atomic-ra-revprop-change.c (construct_auth_baton): Rename RESULT_POOL to POOL. Modifie

svn commit: r978841 - in /subversion/trunk/subversion/tests/cmdline: svnrdump_tests.py svnrdump_tests_data/ svnrdump_tests_data/revision0.dump

2010-07-24 Thread artagnon
Author: artagnon Date: Sat Jul 24 10:18:58 2010 New Revision: 978841 URL: http://svn.apache.org/viewvc?rev=978841&view=rev Log: svnrdump: Add a regression test which verifies that revision 0 of a repository is dumped correctly. Revision 0 only has revision properties, so it's somewhat special. *

svn commit: r978835 - /subversion/branches/atomic-revprop/subversion/tests/cmdline/atomic-ra-revprop-change.c

2010-07-24 Thread danielsh
Author: danielsh Date: Sat Jul 24 09:36:36 2010 New Revision: 978835 URL: http://svn.apache.org/viewvc?rev=978835&view=rev Log: On the 'atomic-revprop' branch: Teach the utility wrapper to authenticate as jrandom:rayjandom. * subversion/tests/cmdline/atomic-ra-revprop-change.c (aborting_prompt

svn commit: r978830 - /subversion/trunk/subversion/tests/cmdline/patch_tests.py

2010-07-24 Thread dannas
Author: dannas Date: Sat Jul 24 08:51:33 2010 New Revision: 978830 URL: http://svn.apache.org/viewvc?rev=978830&view=rev Log: Add a XFailing test for running 'svn patch' with property diffs against dirs. * subversion/tests/cmdline/patch_tests.py (patch_dir_properties): New. Note that the outpu

svn commit: r978821 - /subversion/trunk/subversion/tests/cmdline/patch_tests.py

2010-07-24 Thread dannas
Author: dannas Date: Sat Jul 24 08:05:26 2010 New Revision: 978821 URL: http://svn.apache.org/viewvc?rev=978821&view=rev Log: Adjust test for patching properties to include deleting and adding properties. * subversion/tests/cmdline/patch_tests.py (patch_with_properties): Include a deleted and