Re: Error running context

2012-12-03 Thread Philip Martin
Ben Reser writes: > On Sun, Dec 2, 2012 at 9:18 AM, Justin Erenkrantz > wrote: >> As for the looping, didn't we put in logic to stop retrying after a number >> of request failures? That's probably a reasonable thing to do...I >> definitely think a very nice feature of serf is that it *will* re

RE: Error running context

2012-12-02 Thread Bert Huijben
:* Subversion Development *Subject:* Re: Error running context Hi, On Sun, Dec 2, 2012 at 3:18 PM, Justin Erenkrantz wrote: > On Sun, Dec 2, 2012 at 9:05 AM, Lieven Govaerts wrote: >> >> Attached the patch. ( I get paid per mail I send to this list in case >> no one noticed. ;) )

Re: Error running context

2012-12-02 Thread Lieven Govaerts
Hi, On Sun, Dec 2, 2012 at 3:18 PM, Justin Erenkrantz wrote: > On Sun, Dec 2, 2012 at 9:05 AM, Lieven Govaerts wrote: >> >> Attached the patch. ( I get paid per mail I send to this list in case >> no one noticed. ;) ) > > > The patch looks right to me - the short-circuit to do the simple window

Re: Error running context

2012-12-02 Thread Stefan Fuhrmann
On Sun, Dec 2, 2012 at 3:05 PM, Lieven Govaerts wrote: > Attached the patch. ( I get paid per mail I send to this list in case > no one noticed. ;) ) > > On Sun, Dec 2, 2012 at 2:57 PM, Lieven Govaerts wrote: > > Hi, > > > > On Fri, Nov 30, 2012 at 8:19 PM, Philip Martin > > wrote: > >> Stefan

Re: Error running context

2012-12-02 Thread Ben Reser
On Sun, Dec 2, 2012 at 9:18 AM, Justin Erenkrantz wrote: > As for the looping, didn't we put in logic to stop retrying after a number > of request failures? That's probably a reasonable thing to do...I > definitely think a very nice feature of serf is that it *will* retry (which > is helpful in f

Re: Error running context

2012-12-02 Thread Justin Erenkrantz
On Sun, Dec 2, 2012 at 9:05 AM, Lieven Govaerts wrote: > Attached the patch. ( I get paid per mail I send to this list in case > no one noticed. ;) ) The patch looks right to me - the short-circuit to do the simple window can clearly be called for any window regardless of offset. Perhaps we ca

Re: Error running context

2012-12-02 Thread Lieven Govaerts
Attached the patch. ( I get paid per mail I send to this list in case no one noticed. ;) ) On Sun, Dec 2, 2012 at 2:57 PM, Lieven Govaerts wrote: > Hi, > > On Fri, Nov 30, 2012 at 8:19 PM, Philip Martin > wrote: >> Stefan Küng writes: >> >>> Here's how to reproduce: >>> >>> $ svn co https://tor

Re: Error running context

2012-12-02 Thread Lieven Govaerts
On Sun, Dec 2, 2012 at 2:57 PM, Lieven Govaerts wrote: > Hi, > > On Fri, Nov 30, 2012 at 8:19 PM, Philip Martin > wrote: >> Stefan Küng writes: >> >>> Here's how to reproduce: >>> >>> $ svn co https://tortoisesvn.googlecode.com/svn/trunk/src/Resources/tools >>> tools >>> >>> get the file here:

Re: Error running context

2012-12-02 Thread Lieven Govaerts
Hi, On Fri, Nov 30, 2012 at 8:19 PM, Philip Martin wrote: > Stefan Küng writes: > >> Here's how to reproduce: >> >> $ svn co https://tortoisesvn.googlecode.com/svn/trunk/src/Resources/tools >> tools >> >> get the file here: >> https://skydrive.live.com/redir?resid=D000F60A347E5B37!11352 >> and

Re: Error running context

2012-11-30 Thread Philip Martin
Philip Martin writes: > Philip Martin writes: > >> As far as I can tell the problem is the client causing mod_dav_svn to >> SEGV (serf trunk keep retrying and causing multiple SEGVs). The >> mod_dav_svn stack trace isn't very useful, I'll need a httpd debug >> build: > > Debug build with pool d

Re: Error running context

2012-11-30 Thread Philip Martin
Philip Martin writes: > As far as I can tell the problem is the client causing mod_dav_svn to > SEGV (serf trunk keep retrying and causing multiple SEGVs). The > mod_dav_svn stack trace isn't very useful, I'll need a httpd debug > build: Debug build with pool debugging: Program received signal

Re: Error running context

2012-11-30 Thread Philip Martin
Stefan Küng writes: > Here's how to reproduce: > > $ svn co https://tortoisesvn.googlecode.com/svn/trunk/src/Resources/tools > tools > > get the file here: > https://skydrive.live.com/redir?resid=D000F60A347E5B37!11352 > and replace the one in 'tools' with this one. I can reproduce locally by i