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

2012-11-06 Thread Stefan Fuhrmann
On Mon, Nov 5, 2012 at 2:00 AM, Johan Corveleyn wrote: > Before I look into this further, I'd like to know if anyone else has seen > this or can reproduce this (or doesn't reproduce this with a similar > environment) ... > > Tests 12 and 15 of mergeinfo-test.exe and test 4 of skel-test.exe hang o

Re: Race in svn_atomic_namespace__create

2012-11-06 Thread Stefan Fuhrmann
On Mon, Nov 5, 2012 at 3:16 PM, Stefan Sperling wrote: > On Mon, Nov 05, 2012 at 02:54:07PM +0100, Stefan Fuhrmann wrote: > > On Sun, Nov 4, 2012 at 10:40 AM, Stefan Sperling wrote: > > > I just came across something that reminded me of this thread. > > > It seems PostgreSQL is doing something q

Re: Race in svn_atomic_namespace__create

2012-11-06 Thread Stefan Sperling
On Tue, Nov 06, 2012 at 11:47:51AM +0100, Stefan Fuhrmann wrote: > I'd like to see all that solved and SHM being used > for membuffer - which has been designed with that > goal in mind. It's the robustness part that makes it > so much harder to do than I thought back then. Right. I see. Thanks for

Re: Random serf checkout failures

2012-11-06 Thread C. Michael Pilato
On 11/05/2012 08:30 PM, Justin Erenkrantz wrote: > On Mon, Nov 5, 2012 at 1:33 PM, C. Michael Pilato wrote: >> My debugging indicates that when close_all_dirs() is called, there are a >> slew of unclosed directories which each have a non-zero ref_count. Save for >> the root directory of the edit

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

2012-11-06 Thread Stefan Sperling
On Mon, Nov 05, 2012 at 09:54:56PM +0530, vijay wrote: > Hi, > > This patch implements '--include-externals' option to 'svn list' > [Issue #4225] [1]. > > All tests pass with 'make check' & 'make davautocheck'. > > Attached the patch and log message. > > Please review this patch and share your

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

2012-11-06 Thread Stefan Fuhrmann
On Mon, Nov 5, 2012 at 5:24 PM, vijay wrote: > Hi, > > This patch implements '--include-externals' option to 'svn list' [Issue > #4225] [1]. > > All tests pass with 'make check' & 'make davautocheck'. > > Attached the patch and log message. > > Please review this patch and share your thoughts. >

Re: Random serf checkout failures

2012-11-06 Thread Ben Reser
I worked with Philip today and was able to reproduce the exact problem he's been seeing. I ended up having to get his full httpd.conf to figure it out.. Ultimately the problem proved to be that he had this directive: Timeout 3 Which would mean if we don't tend a connection for 3 seconds Apache w

Re: Random serf checkout failures

2012-11-06 Thread C. Michael Pilato
On 11/06/2012 08:29 AM, C. Michael Pilato wrote: > On 11/05/2012 08:30 PM, Justin Erenkrantz wrote: >> On Mon, Nov 5, 2012 at 1:33 PM, C. Michael Pilato >> wrote: >>> My debugging indicates that when close_all_dirs() is called, there are a >>> slew of unclosed directories which each have a non-ze

Re: disregarding svn:global-ignores

2012-11-06 Thread C. Michael Pilato
On 11/06/2012 10:29 AM, Julian Foad wrote: >>> Sorry, that section is out of date, I corrected it. The >>> --no-ignores option still works for status, it's only for import and >>> add that it can't be overridden. >>> >> >> Perfect, thanks. I think not-overriding for add/import is fine: for >>

Re: Random serf checkout failures

2012-11-06 Thread Lieven Govaerts
Ben, On Tue, Nov 6, 2012 at 4:09 PM, Ben Reser wrote: > I worked with Philip today and was able to reproduce the exact problem > he's been seeing. I ended up having to get his full httpd.conf to > figure it out.. > > Ultimately the problem proved to be that he had this directive: > Timeout 3 > >

Re: disregarding svn:global-ignores

2012-11-06 Thread Julian Foad
C. Michael Pilato wrote: > On 11/06/2012 10:29 AM, Julian Foad wrote: Sorry, that section is out of date, I corrected it.  The --no-ignores option still works for status, it's only for import and add that it can't be overridden. >>> >>> Perfect, thanks.  I think not-overriding

Re: disregarding svn:global-ignores

2012-11-06 Thread Paul Burba
On Tue, Nov 6, 2012 at 11:20 AM, Julian Foad wrote: > C. Michael Pilato wrote: > >> On 11/06/2012 10:29 AM, Julian Foad wrote: > Sorry, that section is out of date, I corrected it. The > --no-ignores option still works for status, it's only for > import and add that it can't be ove

Re: Random serf checkout failures

2012-11-06 Thread Lieven Govaerts
Hi, On Tue, Nov 6, 2012 at 4:50 PM, Lieven Govaerts wrote: > Ben, > > On Tue, Nov 6, 2012 at 4:09 PM, Ben Reser wrote: >> I worked with Philip today and was able to reproduce the exact problem >> he's been seeing. I ended up having to get his full httpd.conf to >> figure it out.. >> >> Ultimate

Re: Kudos for the new inheritable properties feature

2012-11-06 Thread Paul Burba
On Tue, Nov 6, 2012 at 12:31 PM, Julian Foad wrote: > Hi Paul. > > While arguing over some details in other email threads, I just want to say > I'm very excited and impressed that you've taken on the inheritable > properties feature and achieved a design that seems to be quite powerful and > fl

Re: disregarding svn:global-ignores

2012-11-06 Thread Julian Foad
Paul Burba wrote: > On Tue, Nov 6, 2012 at 11:20 AM, Julian Foad wrote: >> C. Michael Pilato wrote: >>> On 11/06/2012 10:29 AM, Julian Foad wrote: >> Sorry, that section is out of date, I corrected it.  The >> --no-ignores option still works for status, it's only for >> import and 

Re: Please review Symmetric Merge

2012-11-06 Thread Paul Burba
On Fri, Nov 2, 2012 at 5:29 PM, Paul Burba wrote: > On Thu, Sep 13, 2012 at 6:55 PM, Julian Foad > wrote: >> Hi, merge fans and QA fans. >> >> As you've seen, the 'symmetric merge' code is live in trunk and destined to >> be in 1.8. >> >> The principle is nice; however, the current implementati

Re: Please review Symmetric Merge

2012-11-06 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 >> iss

Re: Random serf checkout failures

2012-11-06 Thread Ivan Zhakov
On Tue, Nov 6, 2012 at 9:13 PM, Lieven Govaerts wrote: > Hi, > > On Tue, Nov 6, 2012 at 4:50 PM, Lieven Govaerts wrote: >> Ben, >> >> On Tue, Nov 6, 2012 at 4:09 PM, Ben Reser wrote: >>> I worked with Philip today and was able to reproduce the exact problem >>> he's been seeing. I ended up havi

Re: disregarding svn:global-ignores

2012-11-06 Thread Daniel Shahaf
Julian Foad wrote on Tue, Nov 06, 2012 at 15:29:33 +: > - Original Message - > > From: Daniel Shahaf > > To: Paul Burba > > Cc: dev@subversion.apache.org > > Sent: Monday, 5 November 2012, 22:23 > > Subject: Re: disregarding svn:global-ignores > > > > Paul Burba wrote on Mon, Nov 05,

Re: disregarding svn:global-ignores

2012-11-06 Thread Julian Foad
Daniel Shahaf wrote: > Julian Foad wrote: >> Daniel Shahaf wrote: >> > Perfect, thanks.  I think not-overriding for add/import is fine: for >> > 'import' only the repository files are affected, and for 'add' files >> > matching the pattern can be specified explicitly in the argv targets >> >

Re: Random serf checkout failures

2012-11-06 Thread Ivan Zhakov
On Tue, Nov 6, 2012 at 10:24 PM, Ivan Zhakov wrote: > On Tue, Nov 6, 2012 at 9:13 PM, Lieven Govaerts wrote: >> Hi, >> >> On Tue, Nov 6, 2012 at 4:50 PM, Lieven Govaerts wrote: >>> Ben, >>> >>> On Tue, Nov 6, 2012 at 4:09 PM, Ben Reser wrote: I worked with Philip today and was able to repr

Re: disregarding svn:global-ignores

2012-11-06 Thread Paul Burba
On Tue, Nov 6, 2012 at 1:31 PM, Julian Foad wrote: > Daniel Shahaf wrote: > >> Julian Foad wrote: >>> Daniel Shahaf wrote: >>> > Perfect, thanks. I think not-overriding for add/import is fine: for >>> > 'import' only the repository files are affected, and for 'add' files >>> > matching the pa

Re: disregarding svn:global-ignores

2012-11-06 Thread Ivan Zhakov
On Tue, Nov 6, 2012 at 10:49 PM, Paul Burba wrote: > On Tue, Nov 6, 2012 at 1:31 PM, Julian Foad > wrote: >> Daniel Shahaf wrote: >> >>> Julian Foad wrote: Daniel Shahaf wrote: > Perfect, thanks. I think not-overriding for add/import is fine: for > 'import' only the repository

Re: disregarding svn:global-ignores

2012-11-06 Thread Stefan Sperling
On Tue, Nov 06, 2012 at 11:06:51PM +0400, Ivan Zhakov wrote: > /bike-shed note: why property named "svn:global-ignores", not > "svn:global-ignore"? It will be looking more consistent with > "svn:ignore". The name mirrors the "global-ignores" option in the configuration file. That's not a very stro

Re: disregarding svn:global-ignores

2012-11-06 Thread Johan Corveleyn
On Tue, Nov 6, 2012 at 8:22 PM, Stefan Sperling wrote: > > On Tue, Nov 06, 2012 at 11:06:51PM +0400, Ivan Zhakov wrote: > > /bike-shed note: why property named "svn:global-ignores", not > > "svn:global-ignore"? It will be looking more consistent with > > "svn:ignore". > > The name mirrors the "glo

Re: disregarding svn:global-ignores

2012-11-06 Thread Paul Burba
On Tue, Nov 6, 2012 at 2:33 PM, Johan Corveleyn wrote: > On Tue, Nov 6, 2012 at 8:22 PM, Stefan Sperling wrote: >> >> On Tue, Nov 06, 2012 at 11:06:51PM +0400, Ivan Zhakov wrote: >> > /bike-shed note: why property named "svn:global-ignores", not >> > "svn:global-ignore"? It will be looking more c

When does update/merge conflict produce .edited file?

2012-11-06 Thread Mark Phippard
Please see this thread from Subclipse mailing list: http://svn.haxx.se/subusers/archive-2012-11/0001.shtml Basically, a user has a recipe where an svn update has text conflicts. In addition to the usual conflict files, there is also a file named "foo.txt.edited" created. I can see in libsvn_wc/

Re: When does update/merge conflict produce .edited file?

2012-11-06 Thread Mark Phippard
It seems like this is the relevant commit: http://svn.apache.org/viewvc?view=revision&revision=871101 Not sure how the behavior may have been amended since that commit as it is kind of old. But the log message would explain why I did not see it when I tried to do this as I did not take the (E)di

Re: Random serf checkout failures

2012-11-06 Thread C. Michael Pilato
On 11/06/2012 10:15 AM, C. Michael Pilato wrote: > On 11/06/2012 08:29 AM, C. Michael Pilato wrote: >> I recall this patch of yours -- I even asked you about it post-Berlin. I'll >> take a look at it now and see if it can help us out. > > Not seeing the active_dir_propfinds list corruption you me

Re: disregarding svn:global-ignores

2012-11-06 Thread Johan Corveleyn
On Tue, Nov 6, 2012 at 8:55 PM, Paul Burba wrote: > On Tue, Nov 6, 2012 at 2:33 PM, Johan Corveleyn wrote: >> On Tue, Nov 6, 2012 at 8:22 PM, Stefan Sperling wrote: >>> >>> On Tue, Nov 06, 2012 at 11:06:51PM +0400, Ivan Zhakov wrote: >>> > /bike-shed note: why property named "svn:global-ignores"

Re: When does update/merge conflict produce .edited file?

2012-11-06 Thread Mark Phippard
Think this turns out to be a bug with SVNKit only. It appears to create this file even when postponing conflict resolution. Sorry for the noise. On Tue, Nov 6, 2012 at 3:17 PM, Mark Phippard wrote: > It seems like this is the relevant commit: > > http://svn.apache.org/viewvc?view=revision&rev

Re: 1.8 Progress

2012-11-06 Thread Paul Burba
On Sat, Nov 3, 2012 at 9:33 AM, Stefan Sperling wrote: > On Fri, Nov 02, 2012 at 06:00:52PM -0400, Paul Burba wrote: >> These six XFailing upate tests are all part of this effort correct? >> >> 61XFAIL update locally moved dir with leaf del >> 62XFAIL update locally moved dir with edited

Re: When does update/merge conflict produce .edited file?

2012-11-06 Thread Stefan Sperling
On Tue, Nov 06, 2012 at 03:03:32PM -0500, Mark Phippard wrote: > Please see this thread from Subclipse mailing list: > > http://svn.haxx.se/subusers/archive-2012-11/0001.shtml > > Basically, a user has a recipe where an svn update has text conflicts. > In addition to the usual conflict files, th

Re: disregarding svn:global-ignores

2012-11-06 Thread Julian Foad
Johan Corveleyn wrote: > On Tue, Nov 6, 2012 at 8:55 PM, Paul Burba wrote: >> On Tue, Nov 6, 2012 at 2:33 PM, Johan Corveleyn wrote: >>> At the risk of getting something thrown at me: after reading >>> Julian's arguments, I was thinking "svn:recursive-ignore" might >>> be a better name. It d

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

2012-11-06 Thread Johan Corveleyn
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 like to know if anyone else has seen >> this or can reproduce this (or doesn't reproduce this with a similar >> environment) ... >> >> Tests

Re: Random serf checkout failures

2012-11-06 Thread Ivan Zhakov
On Tue, Nov 6, 2012 at 10:24 PM, Ivan Zhakov wrote: > On Tue, Nov 6, 2012 at 9:13 PM, Lieven Govaerts wrote: >> Hi, >> >> On Tue, Nov 6, 2012 at 4:50 PM, Lieven Govaerts wrote: >>> Ben, >>> >>> On Tue, Nov 6, 2012 at 4:09 PM, Ben Reser wrote: I worked with Philip today and was able to repr

Re: Random serf checkout failures

2012-11-06 Thread Lieven Govaerts
Hi, On Tue, Nov 6, 2012 at 6:13 PM, Lieven Govaerts wrote: > Hi, > > On Tue, Nov 6, 2012 at 4:50 PM, Lieven Govaerts wrote: >> Ben, >> >> On Tue, Nov 6, 2012 at 4:09 PM, Ben Reser wrote: >>> I worked with Philip today and was able to reproduce the exact problem >>> he's been seeing. I ended up

Re: Random serf checkout failures

2012-11-06 Thread Lieven Govaerts
patch attached On Tue, Nov 6, 2012 at 10:45 PM, Lieven Govaerts wrote: > Hi, > > On Tue, Nov 6, 2012 at 6:13 PM, Lieven Govaerts wrote: >> Hi, >> >> On Tue, Nov 6, 2012 at 4:50 PM, Lieven Govaerts wrote: >>> Ben, >>> >>> On Tue, Nov 6, 2012 at 4:09 PM, Ben Reser wrote: I worked with Phili

Re: Random serf checkout failures

2012-11-06 Thread Philip Martin
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/subversion/libsvn_subr/config.c ../src/subversion/svn/checkout-cmd.c:168: (apr_

Re: Random serf checkout failures

2012-11-06 Thread Philip Martin
Philip Martin writes: > but I still see this as well: > > A > wc/1.3.0-rc1/subversion/bindings/java/javahl/src/org/tigris/subversion/javahl/SVNAdmin.java > ../src/subversion/libsvn_ra_serf/update.c:680: (apr_err=235000) > svn: E235000: In file '../src/subversion/libsvn_ra_serf/update.c' line

Re: Random serf checkout failures

2012-11-06 Thread Justin Erenkrantz
On Tue, Nov 6, 2012 at 3:22 PM, C. Michael Pilato wrote: > On 11/06/2012 10:15 AM, C. Michael Pilato wrote: >> On 11/06/2012 08:29 AM, C. Michael Pilato wrote: >>> I recall this patch of yours -- I even asked you about it post-Berlin. I'll >>> take a look at it now and see if it can help us out.

Re: Random serf checkout failures

2012-11-06 Thread Justin Erenkrantz
On Tue, Nov 6, 2012 at 4:41 PM, Ivan Zhakov wrote: > Another problem is how serf reads data from network in case of > multiple connections: it reads data from one connection until EAGAIN. > But if data comes from network really fast (from local server for > example) it continue reading data from t

Re: Random serf checkout failures

2012-11-06 Thread Justin Erenkrantz
On Tue, Nov 6, 2012 at 12:13 PM, Lieven Govaerts wrote: > okay, so with the Timeout directive added I can reproduce this issue. > > What I see is that the server closes the connection in the middle of > sending a response to the client. It doesn't even finalize the > response first. > So ra_serf i

Re: [PATCH] enabling ruby in the subversion build

2012-11-06 Thread mpchlets
I am also very interested in seeing this get to a state of usability - I am willing to help also. If there is a current status or anyone working on this, I would love to pitch in and get this finalized. -- View this message in context: http://subversion.1072662.n5.nabble.com/PATCH-enabling-rub

Re: Random serf checkout failures

2012-11-06 Thread Ivan Zhakov
On Wed, Nov 7, 2012 at 1:41 AM, Ivan Zhakov wrote: > On Tue, Nov 6, 2012 at 10:24 PM, Ivan Zhakov wrote: >> On Tue, Nov 6, 2012 at 9:13 PM, Lieven Govaerts wrote: >>> Hi, >>> >>> On Tue, Nov 6, 2012 at 4:50 PM, Lieven Govaerts wrote: Ben, On Tue, Nov 6, 2012 at 4:09 PM, Ben Reser

Re: disregarding svn:global-ignores

2012-11-06 Thread Daniel Shahaf
Paul Burba wrote on Tue, Nov 06, 2012 at 12:01:54 -0500: > Also keep in mind that if you really want to add an ignored file you > can make the file the target of the 'svn add' command. Just like with > the runtime config global-ignores and the svn:ignore property in 1.7, > if the file in question