Re: Hanging tests mergeinfo-test.exe 12, 15 and skel-test.exe 4 on WinXP with VS2010 release build

2012-11-08 Thread Johan Corveleyn
On Thu, Nov 8, 2012 at 7:30 AM, Stefan Fuhrmann wrote: > On Tue, Nov 6, 2012 at 10:29 PM, Johan Corveleyn wrote: >> >> On Tue, Nov 6, 2012 at 10:53 AM, Stefan Fuhrmann >> wrote: >> > On Mon, Nov 5, 2012 at 2:00 AM, Johan Corveleyn >> > wrote: >> >> >> >> Before I look into this further, I'd lik

Re: Issue #4239 'svn mergeinfo' should show a user-friendly summary

2012-11-08 Thread Julian Foad
C. Michael Pilato wrote: > On 11/08/2012 03:50 PM, Julian Foad wrote: >> C-Mike Pilato asks in >> , >> >> "Julian, what is the exit criteria for this issue's completion?  >> At what point do we call it "finished" -- or at least finish

Re: svn commit: r1407279 - in /subversion/trunk/subversion: svnadmin/main.c tests/cmdline/svntest/main.py

2012-11-08 Thread Stefan Sperling
On Thu, Nov 08, 2012 at 09:42:34PM -, cmpil...@apache.org wrote: > Author: cmpilato > Date: Thu Nov 8 21:42:34 2012 > New Revision: 1407279 > > URL: http://svn.apache.org/viewvc?rev=1407279&view=rev > Log: > Rather than continue the trend of accumulating new --pre-X.Y-compatible > options to

Re: ra_serf: truncated responses by low Timeout are not handled graceful

2012-11-08 Thread Philip Martin
Lieven Govaerts writes: > My expectation is that with serf trunk up to date r1685 you'll now > only see the error "The server sent a truncated HTTP response body." > instead of asserts, checksum or other svn errors. I do see that error: ../src/subversion/svn/checkout-cmd.c:168: (apr_err=120106)

Re: Issue #4239 'svn mergeinfo' should show a user-friendly summary

2012-11-08 Thread C. Michael Pilato
On 11/08/2012 03:50 PM, Julian Foad wrote: > C-Mike Pilato asks in > , > > "Julian, what is the exit criteria for this issue's completion? At what > point do we call it "finished" -- or at least finished enough that future > improvements c

Issue #4239 'svn mergeinfo' should show a user-friendly summary

2012-11-08 Thread Julian Foad
C-Mike Pilato asks in , "Julian, what is the exit criteria for this issue's completion?  At what point do we call it "finished" -- or at least finished enough that future improvements can be tracked as unique issues?" I guess there are tw

Re: ra_serf: truncated responses by low Timeout are not handled graceful

2012-11-08 Thread Lieven Govaerts
Hi, On Thu, Nov 8, 2012 at 1:34 PM, Philip Martin wrote: > Lieven Govaerts writes: > >> Any truncated response should now result in an error from serf (since >> serf trunk r1678). It's not clear to me if this error >> SERF_ERROR_TRUNCATED_HTTP_RESPONSE was returned from serf but then >> ignored

Re: [PATCH] enabling ruby in the subversion build

2012-11-08 Thread Philip Martin
Michael Chletsos writes: > Initial support for ruby 1.9.3 when creating swig bindings for subversion. I tweaked the format a little and committed in r1407206. Thanks! -- Certified & Supported Apache Subversion Downloads: http://www.wandisco.com/subversion/download

Re: [PATCH] enabling ruby in the subversion build

2012-11-08 Thread Michael Chletsos
Thank you! Ruby community will be excited, I will continue to improve the tests. On Thu, Nov 8, 2012 at 8:29 AM, Philip Martin wrote: > Michael Chletsos writes: > >> Initial support for ruby 1.9.3 when creating swig bindings for subversion. > > I tweaked the format a little and committed in r14

Re: svn commit: r1406870 - in /subversion/trunk/subversion: libsvn_wc/update_editor.c libsvn_wc/wc_db_wcroot.c tests/cmdline/special_tests.py

2012-11-08 Thread Peter Samuelson
[Daniel Shahaf] > Did you mean: > $ printf 'link bar' > foo Yes I did mean that - thanks. > > $ svn add --config-option config:miscellany:enable-auto-props=yes \ > > --config-option config:auto-props:foo=svn:special=1 foo > > A foo > > > > $ ls -l foo > >

Re: [PATCH] enabling ruby in the subversion build

2012-11-08 Thread Michael Chletsos
Initial support for ruby 1.9.3 when creating swig bindings for subversion. * Makefile.in: Ruby 1.8 uses --verbose, 1.9 does not for run-test.rb * subversion/bindings/swig/ruby/test/test_core.rb: Updated for ruby1.9 time, which uses nanoseconds * subversion/bindings/swig/ruby/test/test_repos.rb

Re: svn commit: r1406870 - in /subversion/trunk/subversion: libsvn_wc/update_editor.c libsvn_wc/wc_db_wcroot.c tests/cmdline/special_tests.py

2012-11-08 Thread Daniel Shahaf
Peter Samuelson wrote on Thu, Nov 08, 2012 at 11:01:37 -0600: > > [Daniel Shahaf] > > it's what allows Windows users to create versioned symlinks: > > > > printf "link bar" > foo && svn add foo && svn ps svn:special yes foo && > > svn ci > > > > If we don't like changing the specialness of a

Assert IS_VALID_FORWARD_RANGE fails in merge_tests 125 [was: Hanging tests mergeinfo-test.exe 12, 15 and skel-test.exe 4 on WinXP with VS2010 release build]

2012-11-08 Thread Julian Foad
Stefan Fuhrmann wrote: > But I did run across an assertion in mergeinfo.c, :line 1174 > >> during merge_tests.py 125: >>> >>>   SVN_ERR_ASSERT_NO_RETURN(IS_VALID_FORWARD_RANGE(first)); > > The weird thing about the assertion is, that the tests > will continue just fine afterwards and no error was

Re: svn commit: r1406870 - in /subversion/trunk/subversion: libsvn_wc/update_editor.c libsvn_wc/wc_db_wcroot.c tests/cmdline/special_tests.py

2012-11-08 Thread Daniel Shahaf
Peter Samuelson wrote on Thu, Nov 08, 2012 at 11:01:37 -0600: > > [Daniel Shahaf] > > it's what allows Windows users to create versioned symlinks: > > > > printf "link bar" > foo && svn add foo && svn ps svn:special yes foo && > > svn ci > > > > If we don't like changing the specialness of a

Re: svn commit: r1406870 - in /subversion/trunk/subversion: libsvn_wc/update_editor.c libsvn_wc/wc_db_wcroot.c tests/cmdline/special_tests.py

2012-11-08 Thread Peter Samuelson
[Daniel Shahaf] > it's what allows Windows users to create versioned symlinks: > > printf "link bar" > foo && svn add foo && svn ps svn:special yes foo && svn > ci > > If we don't like changing the specialness of a local addition, we could > deprecate (or break) that behaviour and have people

Content-Length in HEAD responses (was: Re: Authz on Collection of Repositories)

2012-11-08 Thread Daniel Shahaf
Thomas Åkesson wrote on Thu, Nov 08, 2012 at 15:15:03 +0100: > > On 5 nov 2012, at 09:11, Branko Čibej wrote: > > > On 05.11.2012 00:21, Thomas Åkesson wrote: > >> I did some tests with curl --head just as a sanity check. It seems to be a > >> good choice for access control. I primarily wanted t

RE: svn commit: r1407127 - /subversion/branches/1.7.x/STATUS

2012-11-08 Thread Bert Huijben
> -Original Message- > From: Paul Burba [mailto:ptbu...@gmail.com] > Sent: donderdag 8 november 2012 17:13 > To: Bert Huijben > Cc: dev@subversion.apache.org > Subject: Re: svn commit: r1407127 - /subversion/branches/1.7.x/STATUS > > On Thu, Nov 8, 2012 at 10:32 AM, Bert Huijben wrote: >

Re: svn commit: r1407127 - /subversion/branches/1.7.x/STATUS

2012-11-08 Thread Paul Burba
On Thu, Nov 8, 2012 at 10:32 AM, Bert Huijben wrote: > > >> -Original Message- >> From: phi...@apache.org [mailto:phi...@apache.org] >> Sent: donderdag 8 november 2012 16:27 >> To: comm...@subversion.apache.org >> Subject: svn commit: r1407127 - /subversion/branches/1.7.x/STATUS >> >> Auth

RE: svn commit: r1406870 - in /subversion/trunk/subversion: libsvn_wc/update_editor.c libsvn_wc/wc_db_wcroot.c tests/cmdline/special_tests.py

2012-11-08 Thread Bert Huijben
> -Original Message- > From: Philip Martin [mailto:philip.mar...@wandisco.com] > Sent: donderdag 8 november 2012 13:31 > To: Bert Huijben > Cc: 'Daniel Shahaf'; dev@subversion.apache.org > Subject: Re: svn commit: r1406870 - in /subversion/trunk/subversion: > libsvn_wc/update_editor.c lib

RE: svn commit: r1407127 - /subversion/branches/1.7.x/STATUS

2012-11-08 Thread Bert Huijben
> -Original Message- > From: phi...@apache.org [mailto:phi...@apache.org] > Sent: donderdag 8 november 2012 16:27 > To: comm...@subversion.apache.org > Subject: svn commit: r1407127 - /subversion/branches/1.7.x/STATUS > > Author: philip > Date: Thu Nov 8 15:26:37 2012 > New Revision: 14

Re: Please review Symmetric Merge

2012-11-08 Thread Paul Burba
On Thu, Nov 8, 2012 at 9:49 AM, Julian Foad wrote: > Paul Burba wrote: > >> On Fri, Nov 2, 2012 at 5:29 PM, Paul Burba wrote: >>> What about all these two XFailing tests? >>> >>>merge_automatic_tests.py 16 'cherry2_fwd' >>>merge_automatic_tests.py 17 'cherry3_fwd >>> >>> Are you working

Re: Authz on Collection of Repositories

2012-11-08 Thread Thomas Åkesson
On 5 nov 2012, at 00:21, Thomas Åkesson wrote: > > I have meant to set up a test server with our reference configuration to > validate the patch under realistic circumstances. Unfortunately, the SLES > activation servers have been down for several hours (we don't have dev tools > on our VM Appl

Re: Please review Symmetric Merge

2012-11-08 Thread Julian Foad
Paul Burba wrote: > On Fri, Nov 2, 2012 at 5:29 PM, Paul Burba wrote: >> What about all these two XFailing tests? >> >>   merge_automatic_tests.py 16 'cherry2_fwd' >>   merge_automatic_tests.py 17 'cherry3_fwd >> >> Are you working on those?  Are they blockers for 1.8?  Any existing >> issu

Re: [PATCH] Implement '--include-externals' option to 'svn list'

2012-11-08 Thread Stefan Sperling
On Thu, Nov 08, 2012 at 05:04:35PM +0530, vijay wrote: > On Wednesday 07 November 2012 08:57 PM, Stefan Fuhrmann wrote: > >* result of ls on a sub-folder results in less output > > $>svn ls $URL/sub1 -R --include-externals > > fileA > > fileB > > > > Desired output: > > fileA > > fileB

Re: Authz on Collection of Repositories

2012-11-08 Thread Thomas Åkesson
On 5 nov 2012, at 09:11, Branko Čibej wrote: > On 05.11.2012 00:21, Thomas Åkesson wrote: >> I did some tests with curl --head just as a sanity check. It seems to be a >> good choice for access control. I primarily wanted to see that HEAD requests >> were not allowed in situations where GET is

AW: Random serf checkout failures

2012-11-08 Thread Markus Schaber
Hi, Lieven, Von: lieven.govae...@gmail.com [mailto:lieven.govae...@gmail.com] Im > On Thu, Nov 8, 2012 at 12:02 PM, Markus Schaber > wrote: > > Von: lieven.govae...@gmail.com > > [] > >> 5. In one of Philip error-reporting mails, there was mention of a > >> "408 Request Time-out" response. >

Re: ra_serf: truncated responses by low Timeout are not handled graceful

2012-11-08 Thread Philip Martin
Lieven Govaerts writes: > Any truncated response should now result in an error from serf (since > serf trunk r1678). It's not clear to me if this error > SERF_ERROR_TRUNCATED_HTTP_RESPONSE was returned from serf but then > ignored in svn, or not returned at all. I was using 1.1.x with the patche

Re: svn commit: r1406870 - in /subversion/trunk/subversion: libsvn_wc/update_editor.c libsvn_wc/wc_db_wcroot.c tests/cmdline/special_tests.py

2012-11-08 Thread Philip Martin
"Bert Huijben" writes: > In the update editor we could change the behavior of incoming symlink > changes to be handled as a delete + add, introducing a tree conflict > (and a local replacement of the incoming node to keep the original > state) if there are local changes. > > I think I should be a

Re: Random serf checkout failures

2012-11-08 Thread Lieven Govaerts
Hi Markus, On Thu, Nov 8, 2012 at 12:02 PM, Markus Schaber wrote: > Hi, Lieven, > > Von: lieven.govae...@gmail.com > [] >> 5. In one of Philip error-reporting mails, there was mention of a "408 >> Request Time-out" response. >> (http://svn.haxx.se/dev/archive-2012-11/0076.shtml). >> No idea i

ra_serf: truncated responses by low Timeout are not handled graceful (was Re: Random serf checkout failures)

2012-11-08 Thread Lieven Govaerts
Philip, On Tue, Nov 6, 2012 at 11:20 PM, Philip Martin wrote: > Lieven Govaerts writes: > >>> Philip or Ben, can you test that with this patch svn will always stop >>> with a communication error? > > Using serf 1.1.x patched and Subversion 1406366 I get this new error: > > Awc/1.3.0-rc1/subv

Re: [PATCH] Implement '--include-externals' option to 'svn list'

2012-11-08 Thread vijay
On Wednesday 07 November 2012 08:57 PM, Stefan Fuhrmann wrote: On Wed, Nov 7, 2012 at 3:02 PM, vijay mailto:vi...@collab.net>> wrote: On Tuesday 06 November 2012 08:09 PM, Stefan Fuhrmann wrote: On Mon, Nov 5, 2012 at 5:24 PM, vijay mailto:vi...@collab.net>

Re: svn commit: r1406870 - in /subversion/trunk/subversion: libsvn_wc/update_editor.c libsvn_wc/wc_db_wcroot.c tests/cmdline/special_tests.py

2012-11-08 Thread Philip Martin
"Bert Huijben" writes: > +1 on not allowing to change the symlinkness of existing files. > And +1 on still allowing it on *new* files. > > In the update editor we could change the behavior of incoming symlink > changes to be handled as a delete + add, introducing a tree conflict > (and a local re

Re: [PATCH] enabling ruby in the subversion build

2012-11-08 Thread Philip Martin
Michael Chletsos writes: > OK - this should fix your issues, tested on ruby1.9.3 and ruby1.8.7 - > passed for me, let me know what you get. That works for me. It needs a log message before it can be committed. If you want to write it look at some old log messages and http://subversion.apache.or

AW: Random serf checkout failures

2012-11-08 Thread Markus Schaber
Hi, Lieven, Von: lieven.govae...@gmail.com [] > 5. In one of Philip error-reporting mails, there was mention of a "408 > Request Time-out" response. > (http://svn.haxx.se/dev/archive-2012-11/0076.shtml). > No idea if the server sends this response because one of the above issues, > or if this

Re: [PATCH] Implement '--include-externals' option to 'svn list'

2012-11-08 Thread vijay
On Wednesday 07 November 2012 08:19 PM, Stefan Sperling wrote: On Wed, Nov 07, 2012 at 07:54:33PM +0530, vijay wrote: Thanks stefan for your detailed review. I will keep in mind all your review comments. I will correct my mistakes in future patches. Attached the updated patch and log message.

Re: svn commit: r1406967 - in /subversion/site/publish: docs/community-guide/repro-template.sh roadmap.html

2012-11-08 Thread Daniel Shahaf
danie...@apache.org wrote on Thu, Nov 08, 2012 at 09:03:48 -: > Author: danielsh > Date: Thu Nov 8 09:03:48 2012 > New Revision: 1406967 > > URL: http://svn.apache.org/viewvc?rev=1406967&view=rev > Log: > * roadmap.html: Typo fix > > Modified: > subversion/site/publish/docs/community-gui

Re: svn commit: r1406870 - in /subversion/trunk/subversion: libsvn_wc/update_editor.c libsvn_wc/wc_db_wcroot.c tests/cmdline/special_tests.py

2012-11-08 Thread 'Daniel Shahaf'
Bert Huijben wrote on Thu, Nov 08, 2012 at 10:01:18 +0100: > > -Original Message- > > From: Daniel Shahaf [mailto:d...@daniel.shahaf.name] > > Sent: donderdag 8 november 2012 08:10 > > To: dev@subversion.apache.org > > Subject: Re: svn commit: r1406870 - in /subversion/trunk/subversion: > >

RE: svn commit: r1406870 - in /subversion/trunk/subversion: libsvn_wc/update_editor.c libsvn_wc/wc_db_wcroot.c tests/cmdline/special_tests.py

2012-11-08 Thread Bert Huijben
> -Original Message- > From: Daniel Shahaf [mailto:d...@daniel.shahaf.name] > Sent: donderdag 8 november 2012 08:10 > To: dev@subversion.apache.org > Subject: Re: svn commit: r1406870 - in /subversion/trunk/subversion: > libsvn_wc/update_editor.c libsvn_wc/wc_db_wcroot.c > tests/cmdline/spe

RE: svn commit: r1406870 - in /subversion/trunk/subversion: libsvn_wc/update_editor.c libsvn_wc/wc_db_wcroot.c tests/cmdline/special_tests.py

2012-11-08 Thread Bert Huijben
> -Original Message- > From: Daniel Shahaf [mailto:d...@daniel.shahaf.name] > Sent: donderdag 8 november 2012 08:07 > To: dev@subversion.apache.org > Subject: Re: svn commit: r1406870 - in /subversion/trunk/subversion: > libsvn_wc/update_editor.c libsvn_wc/wc_db_wcroot.c > tests/cmdline/s

Make ra_serf handle 408 Request Timeout response (was Re: Random serf checkout failures)

2012-11-08 Thread Lieven Govaerts
On Wed, Nov 7, 2012 at 10:28 PM, Lieven Govaerts wrote: > Hi, > > > we seem to be raising and discussing a few different issues in this > thread. I've tried to summarize current status with some added > observations. > I propose to start up separate mail threads when discussing one of > these spec