Should mod_dav_svn turn SVN_ERR_FS_LOCK_OWNER_MISMATCH into HTTP_LOCKED?

2011-04-28 Thread Philip Martin
If I lock a file in a wc using one user ID and then commit that wc using a different user ID the commit fails, which is the expected behaviour. Over HTTP I get "500 internal server error" which is not a good error for something that is a client problem. We could use "423 locked" instead, would tha

Re: [hackathon] Keysigning

2011-04-28 Thread Daniel Shahaf
Daniel Shahaf wrote on Wed, Apr 27, 2011 at 09:42:44 +0300: > So, I propose that we recommend committers to add their then-current > preferred PGP keys (used for key signing and release signing, feel free > to add other keys if you want) to ^/subversion/site/keys/$username.asc . No objections, so

Re: svn commit: r1097257 - in /subversion/trunk/subversion: include/private/svn_wc_private.h libsvn_client/info.c libsvn_wc/info.c

2011-04-28 Thread Julian Foad
On Wed, 2011-04-27 at 19:13 -0400, Greg Stein wrote: > On Wed, Apr 27, 2011 at 17:49, wrote: > > Author: hwright > > Date: Wed Apr 27 21:49:00 2011 > > New Revision: 1097257 > > > > URL: http://svn.apache.org/viewvc?rev=1097257&view=rev > > Log: > > Move the working-copy scraping part of svn_clie

Re: svn commit: r1096921 - in /subversion/trunk/subversion: libsvn_client/merge.c libsvn_client/repos_diff.c tests/cmdline/merge_tree_conflict_tests.py tests/cmdline/tree_conflict_tests.py

2011-04-28 Thread Paul Burba
On Tue, Apr 26, 2011 at 5:38 PM, wrote: > Author: rhuijben > Date: Tue Apr 26 21:38:34 2011 > New Revision: 1096921 > > URL: http://svn.apache.org/viewvc?rev=1096921&view=rev > Log: > Remove the last bit of working copy obstruction detection from the repository > diff editor. > > This patch makes

Re: svn commit: r1096921 - in /subversion/trunk/subversion: libsvn_client/merge.c libsvn_client/repos_diff.c tests/cmdline/merge_tree_conflict_tests.py tests/cmdline/tree_conflict_tests.py

2011-04-28 Thread Paul Burba
On Tue, Apr 26, 2011 at 5:38 PM, wrote: > Author: rhuijben > Date: Tue Apr 26 21:38:34 2011 > New Revision: 1096921 > > URL: http://svn.apache.org/viewvc?rev=1096921&view=rev > Log: > Remove the last bit of working copy obstruction detection from the repository > diff editor. > > This patch makes

Re: svn commit: r1097608 - /subversion/trunk/subversion/libsvn_wc/info.c

2011-04-28 Thread Greg Stein
Cool. Are you going to deprecate the constant in svn_types.h? On Thu, Apr 28, 2011 at 16:57, wrote: > Author: hwright > Date: Thu Apr 28 20:57:30 2011 > New Revision: 1097608 > > URL: http://svn.apache.org/viewvc?rev=1097608&view=rev > Log: > * subversion/libsvn_wc/info.c >  (build_info_for_unve

Re: svn commit: r1097600 - /subversion/trunk/subversion/bindings/javahl/src/org/tigris/subversion/javahl/Info.java

2011-04-28 Thread Greg Stein
Why would getSchedule() ever return null? Seems that it should always have some kind of value in it... ?? On Thu, Apr 28, 2011 at 16:39, wrote: > Author: hwright > Date: Thu Apr 28 20:39:21 2011 > New Revision: 1097600 > > URL: http://svn.apache.org/viewvc?rev=1097600&view=rev > Log: > * subvers

Re: svn commit: r1097600 - /subversion/trunk/subversion/bindings/javahl/src/org/tigris/subversion/javahl/Info.java

2011-04-28 Thread Greg Stein
(and likewise for r1097599) On Thu, Apr 28, 2011 at 17:42, Greg Stein wrote: > Why would getSchedule() ever return null? Seems that it should always > have some kind of value in it... ?? > > On Thu, Apr 28, 2011 at 16:39,   wrote: >> Author: hwright >> Date: Thu Apr 28 20:39:21 2011 >> New Revisi

/tmp/default-perms.5.tmp

2011-04-28 Thread Daniel Shahaf
% ls -1 /tmp/default-perms* /tmp/default-perms.2.tmp /tmp/default-perms.3.tmp /tmp/default-perms.4.tmp /tmp/default-perms.5.tmp /tmp/default-perms.tmp Looks like O(n²) filename-search again...

RE: /tmp/default-perms.5.tmp

2011-04-28 Thread Bert Huijben
> -Original Message- > From: Daniel Shahaf [mailto:d...@daniel.shahaf.name] > Sent: donderdag 28 april 2011 23:46 > To: dev@subversion.apache.org > Subject: /tmp/default-perms.5.tmp > > % ls -1 /tmp/default-perms* > /tmp/default-perms.2.tmp > /tmp/default-perms.3.tmp > /tmp/default-perms.4

Re: svn commit: r1097600 - /subversion/trunk/subversion/bindings/javahl/src/org/tigris/subversion/javahl/Info.java

2011-04-28 Thread Hyrum K Wright
If Info was called on a URL, schedule is non-existent (and non-sensical). The new C info stuff doesn't even return a schedule for a URL, so we mimic that in Java by using null. -Hyrum On Thu, Apr 28, 2011 at 4:42 PM, Greg Stein wrote: > Why would getSchedule() ever return null? Seems that it sh