Re: Issues building svn 1.7.6 in non apache mode (module mod_dontdothat)G

2012-08-20 Thread Daniel Shahaf
Bob Chapman wrote on Sun, Aug 19, 2012 at 15:00:40 -0500: > I have used the INSTALL > >$ ./configure >$ make >$ make install > > method to successfully build and install subversion from a tar ball for > versions from 1.4.x to 1.7.6 on a variety of Centos and Ubuntu > platforms. > > T

Re: [PATCH] subversion issue #4174 - serf: checkout / export over https errors out with "svn: E730053: Error retrieving REPORT (730053)"

2012-08-20 Thread Johan Corveleyn
On Sun, Aug 19, 2012 at 1:53 PM, Lieven Govaerts wrote: > Hi Johan, > > > as you seem to be the only one encountering issue 4174, would you mind > testing attached serf patch? > You'll have to update serf trunk to r1627 to apply it cleanly. > > It fixes the issue for me on Windows 7 with svn trunk

Feature Request:- VsnLook

2012-08-20 Thread Cedric JP. Brasey
Chaps, Not 100% sure where i should do this so apologies if i've done this wrong. Would it be possible to add the --xml switch to the svnlook functions? This is only coz i'm a lazy bugger and prefer parsing xml than text. Regards, Cedric ]

Re: Feature Request:- VsnLook

2012-08-20 Thread Cedric JP. Brasey
Sorry, should of said in relation to the info subcommand Regards, Cedric ]---[ Mr Cedric Brasey BSc Managing Director Tel: (07891) 170784 _ From: Cedric JP. Brasey [mailto:cedric.bra...@planet-side.co.uk] To: dev@s

Re: [PATCH] subversion issue #4174 - serf: checkout / export over https errors out with "svn: E730053: Error retrieving REPORT (730053)"

2012-08-20 Thread Lieven Govaerts
On Mon, Aug 20, 2012 at 10:18 AM, Johan Corveleyn wrote: > On Sun, Aug 19, 2012 at 1:53 PM, Lieven Govaerts wrote: >> Hi Johan, >> >> >> as you seem to be the only one encountering issue 4174, would you mind >> testing attached serf patch? >> You'll have to update serf trunk to r1627 to apply it

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

2012-08-20 Thread C. Michael Pilato
On 08/19/2012 12:21 PM, Daniel Shahaf wrote: > C. Michael Pilato wrote on Fri, Aug 17, 2012 at 14:01:03 -0400: >> On 08/17/2012 12:43 PM, Daniel Shahaf wrote: >>> Stefan Sperling wrote on Fri, Aug 17, 2012 at 18:24:03 +0200: On Fri, Aug 17, 2012 at 12:14:31PM -0400, C. Michael Pilato wrote: >>

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

2012-08-20 Thread C. Michael Pilato
On 08/17/2012 12:14 PM, C. Michael Pilato wrote: > Well, I just ran into my first hurdle: the current svnserve protocol > demands that capabilities in the standard capabilities string list be of the > protocol data class "word", which is limited to strings with letters, > numbers, and hyphens only

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

2012-08-20 Thread C. Michael Pilato
On 08/20/2012 10:36 AM, C. Michael Pilato wrote: > [1] mod_dav_svn is an exception here, because the log message is > PROPPATCHed into the transaction after it's creation. start-commit > hooks would have to assume that a non-existent log message is simply > suffering delayed arrival, a

RE: svn commit: r1375052 - /subversion/trunk/subversion/libsvn_subr/auth.c

2012-08-20 Thread Bert Huijben
> -Original Message- > From: cmpil...@apache.org [mailto:cmpil...@apache.org] > Sent: maandag 20 augustus 2012 17:19 > To: comm...@subversion.apache.org > Subject: svn commit: r1375052 - > /subversion/trunk/subversion/libsvn_subr/auth.c > > Author: cmpilato > Date: Mon Aug 20 15:19:06 20

RE: [PATCH] subversion issue #4174 - serf: checkout / export over https errors out with "svn: E730053: Error retrieving REPORT (730053)"

2012-08-20 Thread Bert Huijben
> -Original Message- > From: lieven.govae...@gmail.com [mailto:lieven.govae...@gmail.com] On > Behalf Of Lieven Govaerts > Sent: maandag 20 augustus 2012 12:43 > To: Johan Corveleyn; Subversion Development; serf-...@googlegroups.com > Subject: Re: [PATCH] subversion issue #4174 - serf: ch

Re: svn commit: r1375052 - /subversion/trunk/subversion/libsvn_subr/auth.c

2012-08-20 Thread C. Michael Pilato
On 08/20/2012 11:22 AM, Bert Huijben wrote: > Your log message says no 'logical changes', while this seems to remove > the error for unknown password stores? Doh! Good catch, Bert. Well, as it turns out, I like that change, so I'll just fix the log message. -- C. Michael Pilato CollabNet <

Re: Issues building svn 1.7.6 in non apache mode (module mod_dontdothat)

2012-08-20 Thread Bob Chapman
Daniel - > Check 'make -n install | grep dontdothat'. In the 'patched' 1.7.6 version: $ make -n install | grep dontdothat $ > (There is a longer answer telling you about . . . Which I'm probably not smart enough to understand. It didn't cross my mind to run 'make check' until _after_ m

Re: Issues building svn 1.7.6 in non apache mode (module mod_dontdothat) [and unrelated 'make check' clarification]

2012-08-20 Thread Daniel Shahaf
Bob Chapman wrote on Mon, Aug 20, 2012 at 10:32:01 -0500: > Daniel - > > > Check 'make -n install | grep dontdothat'. > > In the 'patched' 1.7.6 version: > > $ make -n install | grep dontdothat > $ > That's the expected/bug-free output. > > (There is a longer answer telling you about .

Re: Issues building svn 1.7.6 in non apache mode (module mod_dontdothat)

2012-08-20 Thread Stefan Sperling
On Mon, Aug 20, 2012 at 10:32:01AM -0500, Bob Chapman wrote: > On Unbuntu 10.04 LTS, the 87/87 'success' reports looked good but I > haven't figured out what to make of the subsequent string of 'SKIP' and > 'X-FAIL' messages or the summary: These tests are either not applicable to your test setup

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

2012-08-20 Thread Mark Moe
> ALSO: I got a private mail response to my previous post on this thread > which expressed a good deal of excitement regarding the possibility of > start-commit having access to the revprops (especially svn:log). That would be very useful. We have transactions that can be 1GB in size or more and

RE: svn commit: r1375052 - /subversion/trunk/subversion/libsvn_subr/auth.c

2012-08-20 Thread Bert Huijben
> -Original Message- > From: C. Michael Pilato [mailto:cmpil...@collab.net] > Sent: maandag 20 augustus 2012 17:44 > To: Bert Huijben > Cc: cmpil...@apache.org; dev@subversion.apache.org > Subject: Re: svn commit: r1375052 - > /subversion/trunk/subversion/libsvn_subr/auth.c > > On 08/20/

Re: Issues building svn 1.7.6 in non apache mode (module mod_dontdothat) [and unrelated 'make check' clarification]

2012-08-20 Thread Bob Chapman
On Mon, Aug 20, 2012 at 10:50 AM, Daniel Shahaf wrote: > . . . > That's "ALL TESTS SUCCESSFUL". > > Maybe we should print a one-bit summary like that at the end of the run? > . . . Well, can't say but it would have averted my (dumb) question. ;) Thanks again. --

Re: Issues building svn 1.7.6 in non apache mode (module mod_dontdothat) [and unrelated 'make check' clarification]

2012-08-20 Thread Daniel Shahaf
Daniel Shahaf wrote on Mon, Aug 20, 2012 at 16:50:11 +0100: > Bob Chapman wrote on Mon, Aug 20, 2012 at 10:32:01 -0500: > > Daniel - > > > > > Check 'make -n install | grep dontdothat'. > > > > In the 'patched' 1.7.6 version: > > > > $ make -n install | grep dontdothat > > $ > > > > That'

Re: svn commit: r1375052 - /subversion/trunk/subversion/libsvn_subr/auth.c

2012-08-20 Thread C. Michael Pilato
On 08/20/2012 12:00 PM, Bert Huijben wrote: > I like that change too. Maybe we should backport it to 1.7.x in some way? Agreed! I'll propose it in a bit if no one beats me to it. -- C. Michael Pilato CollabNet <> www.collab.net <> Enterprise Cloud Development signature.asc Descript

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

2012-08-20 Thread C. Michael Pilato
On 08/20/2012 11:02 AM, C. Michael Pilato wrote: > On 08/20/2012 10:36 AM, C. Michael Pilato wrote: >> [1] mod_dav_svn is an exception here, because the log message is >> PROPPATCHed into the transaction after it's creation. start-commit >> hooks would have to assume that a non-existent lo

Re: [PATCH] subversion issue #4174 - serf: checkout / export over https errors out with "svn: E730053: Error retrieving REPORT (730053)"

2012-08-20 Thread Lieven Govaerts
On Mon, Aug 20, 2012 at 5:35 PM, Bert Huijben wrote: > > > Maybe you should call SSL_get_error() in the handler to find out the exact > reason on why OpenSSL reports the connection as closed? > > From man SSL_read on my FreeBSD box: > > 0 The read operation was not successful. The reason m

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

2012-08-20 Thread Mark Moe
> Now, if we can find a way to expose the revprops to the start-commit hook, > all three RA layers should be able to provide early detection of commits > that would fail in pre-commit for revprop-related reasons. (And we should > consider this independently of the whole client-version capability s

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

2012-08-20 Thread C. Michael Pilato
On 08/20/2012 03:38 PM, Mark Moe wrote: >> Now, if we can find a way to expose the revprops to the start-commit hook, >> all three RA layers should be able to provide early detection of commits >> that would fail in pre-commit for revprop-related reasons. (And we should >> consider this independen

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

2012-08-20 Thread Daniel Shahaf
Mark Moe wrote on Mon, Aug 20, 2012 at 14:38:43 -0500: > > Now, if we can find a way to expose the revprops to the start-commit hook, > > all three RA layers should be able to provide early detection of commits > > that would fail in pre-commit for revprop-related reasons. (And we should > > consi

having revprops (eg svn:log) in start-commit Re: [Issue 4124] New - Give servers sufficient means to disallow commits from clients based on version numbers

2012-08-20 Thread Daniel Shahaf
[ Nice work indeed, but I'm here only to change the subject to make it easier to notice. ] C. Michael Pilato wrote on Mon, Aug 20, 2012 at 15:15:05 -0400: > On 08/20/2012 11:02 AM, C. Michael Pilato wrote: > > On 08/20/2012 10:36 AM, C. Michael Pilato wrote: > >> [1] mod_dav_svn is an exception he