Re: mod_dontdothat enabled by mistake

2012-08-16 Thread Daniel Shahaf
Bostjan Skufca wrote on Wed, Aug 15, 2012 at 21:48:36 +0200: > I am hitting this issue with 1.7.6: > http://mail-archives.apache.org/mod_mbox/subversion-dev/201205.mbox/%3C20120502131732.GB10429@EbonHawk%3E > > Configure command: > ./configure --prefix=$PDESTDIR_SVN \ > --with-apr=/usr/local/apr

Windows buildbot failures

2012-08-16 Thread Branko Čibej
>From IRC: brane: hm … i don't really understand. it does look like i'll have to install vcexpress 2008 if i want to check what's going on. buildbot logs are less than useful. sigh. [1:58pm] danielsh: philipm: moderating you through to announce@ [1:59pm] philipm: thanks! [2:01pm] philipm: brane:

Re: Windows buildbot failures

2012-08-16 Thread Johan Corveleyn
On Thu, Aug 16, 2012 at 11:53 AM, Branko Čibej wrote: > From IRC: > > brane: hm … i don't really understand. it does look like i'll have to install > vcexpress 2008 if i want to check what's going on. buildbot logs are less > than useful. sigh. > [1:58pm] danielsh: philipm: moderating you throug

Re: Finalizing the definition of 1.8.0

2012-08-16 Thread C. Michael Pilato
Thanks for that reminder! mphippard, pburba and I were just talking about this yesterday as something we want to do for 1.8. http://subversion.tigris.org/issues/show_bug.cgi?id=4124 tracks this request. On 08/15/2012 04:07 PM, Cedric JP. Brasey wrote: > Please please please. Can we make the clien

Re: Finalizing the definition of 1.8.0

2012-08-16 Thread Branko Čibej
I've been looking at the compressed-pristines branch and I don't think there's any chance it could be ready for 1.8; just thought I'd point this out because compressed-pristines are listed in the 1.8 roadmap. -- Brane -- Certified & Supported Apache Subversion Downloads: http://www.wandisco.com/

Re: [Issue 4124] New - Give servers sufficient means to disallow commits from clients based on version numbers

2012-08-16 Thread C. Michael Pilato
Trying to revive an old thread... Daniel Shahaf wrote: > Markus Schaber wrote on Fri, Feb 24, 2012 at 08:09:30 +: >> There are two other cases we should define a rule for: >> >> 1) Clients like SharpSVN or TortoiseSVN, or distribution builds, when >> they include their own patches, or backport

Re: Finalizing the definition of 1.8.0

2012-08-16 Thread Walter Klust
On 15.08.2012 18:21, C. Michael Pilato wrote: Hello, all. Echoing in the back of my head are promises we devs have made to avoid long release cycles, and 1.7.0 recently turned 10 months old. There are a few ongoing bits of feature work happening on branches or in various states of completion on

Re: [Issue 4124] New - Give servers sufficient means to disallow commits from clients based on version numbers

2012-08-16 Thread Mark Phippard
On Thu, Aug 16, 2012 at 10:10 AM, C. Michael Pilato wrote: > So, incorporating some of the great suggestions on this thread so far, but > adding some of my own, I'd like to reset this design discussion with the > following proposal: > ># Reports the general name of the client. ># ># E

Re: [Issue 4124] New - Give servers sufficient means to disallow commits from clients based on version numbers

2012-08-16 Thread Cedric JP. Brasey
Just my two pence worth... >From a hooking point; I would have to say that i would only really care about >what the client level is. For example. On pre-commit i would like to know if >the client is at least 1.7, so that I can then reject the commit. I can't >imagine, for the mo anywa

Re: [Issue 4124] New - Give servers sufficient means to disallow commits from clients based on version numbers

2012-08-16 Thread Daniel Shahaf
Mark Phippard wrote on Thu, Aug 16, 2012 at 10:18:13 -0400: > On Thu, Aug 16, 2012 at 10:10 AM, C. Michael Pilato > wrote: > > > So, incorporating some of the great suggestions on this thread so far, but > > adding some of my own, I'd like to reset this design discussion with the > > following p

Re: [Issue 4124] New - Give servers sufficient means to disallow commits from clients based on version numbers

2012-08-16 Thread Daniel Shahaf
C. Michael Pilato wrote on Thu, Aug 16, 2012 at 10:10:49 -0400: ># Reports the general name of the client. ># ># Examples: "svn", "tortoisesvn", "svnkit" ># ># Default value: "svn". ># >client-name=[-_a-zA-Z0-9]+ I suggest the libsvn_client default be different from wha

Re: Finalizing the definition of 1.8.0

2012-08-16 Thread Ben Reser
On Wed, Aug 15, 2012 at 9:21 AM, C. Michael Pilato wrote: > PS: Is it safe to assume that "commit shelving/checkpointing" is *not* > going to happen for 1.8? ;-) I want to work on this functionality but don't think my time will free up to work on it for at least another few weeks if not a month

Re: Finalizing the definition of 1.8.0

2012-08-16 Thread Cedric JP. Brasey
Sorry about this but could I possibly throw something else into the mix? What would the chances be of close folder/readonly function being introduced? Currently i have implemented a hook that looks for a specific property on the folder and if it is present then the commit is rejected. The rea

Re: [Issue 4124] New - Give servers sufficient means to disallow commits from clients based on version numbers

2012-08-16 Thread Stefan Küng
On Thu, Aug 16, 2012 at 4:46 PM, Cedric JP. Brasey wrote: > Just my two pence worth... > > > > From a hooking point; I would have to say that i would only really care > about what the client level is. For example. On pre-commit i would like to > know if the client is at least 1.7, so that I can th

Re: svn delete fails with "403 Forbidden" if root is not readable

2012-08-16 Thread Julian Foad
I filed this as issue #4219, 'svn delete fails with "403 Forbidden" if root is not readable'. With Philip's help we determined this bug only occurs with RA-neon.  I can now reproduce it on 1.7.x. I looked into why Dmitry's patch doesn't work.  It calls svn_uri_condense_targets(), which finds

Re: [Issue 4124] New - Give servers sufficient means to disallow commits from clients based on version numbers

2012-08-16 Thread C. Michael Pilato
On 08/16/2012 10:50 AM, Daniel Shahaf wrote: > C. Michael Pilato wrote on Thu, Aug 16, 2012 at 10:10:49 -0400: >># Reports the general name of the client. >># >># Examples: "svn", "tortoisesvn", "svnkit" >># >># Default value: "svn". >># >>client-name=[-_a-zA-Z0-9]+ > >

RE: svn delete fails with "403 Forbidden" if root is not readable

2012-08-16 Thread Bert Huijben
Following editor v2 (and v1), deletion of a child is changing the list of children in the parent directory and as such requires (at least) write access to the parent. What happens if paths to within multiple repositories are passed to the function? Was/is this supported? Should the dele

Re: Finalizing the definition of 1.8.0

2012-08-16 Thread Mat Booth
On 16 August 2012 15:57, Cedric JP. Brasey wrote: > Sorry about this but could I possibly throw something else into the mix? > > > What would the chances be of close folder/readonly function being > introduced? Currently i have implemented a hook that looks for a specific > property on the folder

Re: [Issue 4124] New - Give servers sufficient means to disallow commits from clients based on version numbers

2012-08-16 Thread Daniel Shahaf
C. Michael Pilato wrote on Thu, Aug 16, 2012 at 12:11:56 -0400: > On 08/16/2012 10:50 AM, Daniel Shahaf wrote: > > C. Michael Pilato wrote on Thu, Aug 16, 2012 at 10:10:49 -0400: > >># Reports the general name of the client. > >># > >># Examples: "svn", "tortoisesvn", "svnkit" > >>#

Re: svn delete fails with "403 Forbidden" if root is not readable

2012-08-16 Thread Julian Foad
Bert Huijben wrote: > Following editor v2 (and v1), deletion of a child is changing the > list of children in the parent directory and as such requires (at > least) write access to the parent. The editors don't control authorization.  The parent path may be mentioned in an editor call, but that do

Fwd: [Roman Plessl: Issues building svn 1.7.6 in non apache mode (module mod_dontdothat)]

2012-08-16 Thread Daniel Shahaf
Forwarding bug report (with patch) to dev@ - Forwarded message from Roman Plessl - > Date: Thu, 16 Aug 2012 09:48:32 +0200 (CEST) > From: Roman Plessl > To: us...@subversion.apache.org > Subject: Issues building svn 1.7.6 in non apache mode (module mod_dontdothat) > Message-ID: > > Hi

Re: Finalizing the definition of 1.8.0

2012-08-16 Thread Hyrum K Wright
On Wed, Aug 15, 2012 at 12:21 PM, C. Michael Pilato wrote: > Hello, all. > > Echoing in the back of my head are promises we devs have made to avoid long > release cycles, and 1.7.0 recently turned 10 months old. There are a few > ongoing bits of feature work happening on branches or in various st

Re: Finalizing the definition of 1.8.0

2012-08-16 Thread Ben Reser
On Aug 16, 2012 9:43 AM, "Mat Booth" wrote: > Aren't you able to do this with path-based authorisation? Or a commit hook if you really want to use properties.