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 l...@apache.org 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

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

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

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: 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=1406967view=rev Log: * roadmap.html: Typo fix Modified:

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.

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 has a

Re: [PATCH] enabling ruby in the subversion build

2012-11-08 Thread Philip Martin
Michael Chletsos mpchl...@gmail.com 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

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 b...@qqmail.nl 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

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 vi...@collab.net 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 vi...@collab.net

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 philip.mar...@wandisco.com wrote: Lieven Govaerts l...@mobsol.be 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

Re: Random serf checkout failures

2012-11-08 Thread Lieven Govaerts
Hi Markus, On Thu, Nov 8, 2012 at 12:02 PM, Markus Schaber m.scha...@codesys.com 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).

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 b...@qqmail.nl 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

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

2012-11-08 Thread Philip Martin
Lieven Govaerts l...@apache.org 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

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 m.scha...@codesys.com wrote: Von: lieven.govae...@gmail.com [] 5. In one of Philip error-reporting mails, there was mention of a 408 Request Time-out

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 not

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 subsub

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 issue they can be

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

Re: Please review Symmetric Merge

2012-11-08 Thread Paul Burba
On Thu, Nov 8, 2012 at 9:49 AM, Julian Foad julianf...@btopenworld.com 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

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: 1407127

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

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 b...@qqmail.nl 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

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 b...@qqmail.nl

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 to see that

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 run 'svn add

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 local addition, we

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 reported. I

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 local addition, we

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 *

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
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 philip.mar...@wandisco.com wrote: Michael Chletsos mpchl...@gmail.com writes: Initial support for ruby 1.9.3 when creating swig bindings for subversion. I tweaked

Re: [PATCH] enabling ruby in the subversion build

2012-11-08 Thread Philip Martin
Michael Chletsos mpchl...@gmail.com 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: 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 philip.mar...@wandisco.com wrote: Lieven Govaerts l...@apache.org 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

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

2012-11-08 Thread Julian Foad
C-Mike Pilato asks in http://subversion.tigris.org/issues/show_bug.cgi?id=4239, 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 two

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 http://subversion.tigris.org/issues/show_bug.cgi?id=4239, 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

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

2012-11-08 Thread Philip Martin
Lieven Govaerts l...@apache.org 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:

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=1407279view=rev Log: Rather than continue the trend of accumulating new --pre-X.Y-compatible options to

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 http://subversion.tigris.org/issues/show_bug.cgi?id=4239, 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

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 stefan.fuhrm...@wandisco.com wrote: On Tue, Nov 6, 2012 at 10:29 PM, Johan Corveleyn jcor...@gmail.com wrote: On Tue, Nov 6, 2012 at 10:53 AM, Stefan Fuhrmann stefan.fuhrm...@wandisco.com wrote: On Mon, Nov 5, 2012 at 2:00 AM, Johan Corveleyn