Re: MacOS buildbot?

2016-09-04 Thread Lieven Govaerts
On Sat, Sep 3, 2016 at 9:55 PM, Greg Stein wrote: > Hey all, > > Who owns/runs our MacOS buildbots? AOO is seeking a build server. It's a > big build, so might not be possible to sit next to the svn builds, and I > don't know how often they want to run it (eg. buildbot, or just to compile > execu

Re: [serf-dev] Re: [PATCH] Serf crash on spurious data between responses

2015-01-14 Thread Lieven Govaerts
On Wed, Jan 14, 2015 at 12:54 PM, Philip Martin wrote: > Lieven Govaerts writes: > >> The easiest way is to use existing tests as a starting point.: >> - Test test_serf_connection_request_create sends 2 requests and responses. >> This tests sends two pipelined requests

Re: [serf-dev] Re: [PATCH] Serf crash on spurious data between responses

2015-01-12 Thread Lieven Govaerts
On Mon, Jan 12, 2015 at 12:22 PM, Philip Martin wrote: > Lieven Govaerts writes: > >> You're trying to capture the situation where serf has: >> - finished writing a request ( req_bkt == NULL && writing_started ) >> - has no more requests in the pending qu

Re: [serf-dev] Re: [PATCH] Serf crash on spurious data between responses

2015-01-12 Thread Lieven Govaerts
Hi, On Sat, Jan 10, 2015 at 1:07 PM, Philip Martin wrote: > Philip Martin writes: > >> Philip Martin writes: >> >>> While debugging a problem reported on users I accidentally sent an extra >>> byte to the client: I sent Content-Length of N and then sent N+1 bytes. >>> The first N bytes made a

Re: ra_serf bug(s)

2014-10-10 Thread Lieven Govaerts
On Fri, Oct 10, 2014 at 6:08 PM, Julian Foad wrote: > Lieven Govaerts wrote: >>> ra_serf makes a series of valid requests (200 OK) before finally >>> asking the server for REPLAY of r0. In this case the server is >>> googlecode.com. In response to the invalid REPLAY

Re: ra_serf bug(s)

2014-10-10 Thread Lieven Govaerts
On Tue, Oct 7, 2014 at 2:57 PM, Conrad Rad wrote: > Hi all, > > I'm working on a thing that links libsvn_ra_serf (default RA in Fedora > Linux) as well as other pieces of the SVN libraries. > > I am using the 'replay' feature to dump edits from a remote repository. > > Inadvertently, I requested r

Re: [serf-dev] serf errors on responses bigger than 4GB

2014-10-01 Thread Lieven Govaerts
Hi, On Wed, Oct 1, 2014 at 12:48 PM, Philip Martin wrote: > Andreas Stieger writes: > >> I >> will once again point to the serf issues below and httpd/network config. >> https://code.google.com/p/serf/issues/detail?id=152 >> https://code.google.com/p/serf/source/detail?r=2419 > > Andreas identif

Re: [serf-dev] Re: [PATCH] Add a configuration option to disable HTTP pipelining.

2014-08-21 Thread Lieven Govaerts
On Thu, Aug 21, 2014 at 5:47 PM, Justin Erenkrantz wrote: > On Thu, Aug 21, 2014 at 5:30 PM, Lieven Govaerts wrote: >> Attached patch implements a slightly different approach but I think >> it's a valid workaround for the problem. > > It may be usef

Re: [PATCH] Add a configuration option to disable HTTP pipelining.

2014-08-21 Thread Lieven Govaerts
On Thu, Aug 21, 2014 at 1:04 PM, Justin Erenkrantz wrote: > On Tue, Aug 19, 2014 at 2:53 PM, Lieven Govaerts wrote: >> However, when the client certificate is requested for a resource >> deeper in the repository, it's likely that say during a large >> checkout, many

Re: [PATCH] Add a configuration option to disable HTTP pipelining.

2014-08-19 Thread Lieven Govaerts
On Tue, Aug 19, 2014 at 2:34 PM, Mark Phippard wrote: > On Tue, Aug 19, 2014 at 9:25 AM, Lieven Govaerts wrote: > >> There's a bug in OpenSSL's SSL renegotiation algorithm. When it's >> initiated by the server to request a client certificate, it'll fa

[PATCH] Add a configuration option to disable HTTP pipelining.

2014-08-19 Thread Lieven Govaerts
Hi, attached patch adds a new configuration flag to disable HTTP pipelining. There's a bug in OpenSSL's SSL renegotiation algorithm. When it's initiated by the server to request a client certificate, it'll fail when on the connection pipelined requests are incoming at the server side. Short su

Re: [serf-dev] Plan to release serf 1.4.0 (+ TODO list)

2014-08-19 Thread Lieven Govaerts
On Thu, Jun 26, 2014 at 1:14 PM, Ivan Zhakov wrote: > On 22 June 2014 14:13, Lieven Govaerts wrote: >> Hi, >> >> >> I propose we start planning a serf 1.4 release. >> >> Serf trunk has some features that will be used in the upcoming >> Subver

Re: [serf-dev] Plan to release serf 1.4.0 (+ TODO list)

2014-07-01 Thread Lieven Govaerts
Hi, On Thu, Jun 26, 2014 at 11:51 PM, Greg Stein wrote: > On Sun, Jun 22, 2014 at 5:13 AM, Lieven Govaerts wrote: >>... > >> TODO: >> = >> Blocking the release: >> --- >> * Finalize get_remaining(): Ivan has added this fu

Re: [serf-dev] Plan to release serf 1.4.0 (+ TODO list)

2014-06-27 Thread Lieven Govaerts
On Thu, Jun 26, 2014 at 2:14 PM, Ivan Zhakov wrote: > On 22 June 2014 14:13, Lieven Govaerts wrote: >> Hi, >> >> >> I propose we start planning a serf 1.4 release. >> >> Serf trunk has some features that will be used in the upcoming >> Subver

Plan to release serf 1.4.0 (+ TODO list)

2014-06-22 Thread Lieven Govaerts
Hi, I propose we start planning a serf 1.4 release. Serf trunk has some features that will be used in the upcoming Subversion 1.9, so I'd like to get 1.4 out when the Subversion devs start releasing 1.9 release candidates. (When is that planned?) Also, I want to start working on HTTP 2.0 suppor

Re: svn commit: r1557686 - in /subversion/trunk/subversion/libsvn_ra_serf: ra_serf.h util.c

2014-06-22 Thread Lieven Govaerts
Bert, On Mon, Jan 13, 2014 at 12:59 PM, wrote: > Author: rhuijben > Date: Mon Jan 13 11:59:56 2014 > New Revision: 1557686 > > URL: http://svn.apache.org/r1557686 > Log: > Make ra_serf use a pool cleanup handler on its request handlers to allow > reusing the ra session in cases that before this

Re: Error retrieving REPORT

2014-05-06 Thread Lieven Govaerts
Hi Eric, On Fri, May 2, 2014 at 11:18 PM, Eric Lemes wrote: > > Dear Subversion developers, > > First I'd like to thank you for this great peace of success. I'm a user > since the 1.4.x version in a lot of big and complex teams, allways with > sucess. SVN is allways my first choice when I need so

Re: [VOTE] Adjust voting required for alpha/beta releases.

2014-04-07 Thread Lieven Govaerts
On Mon, Apr 7, 2014 at 4:58 PM, Ben Reser wrote: > Let's adopt Johan's suggestion from the other thread. > > Specifically, for alpha/beta releases (no change for release candidates or > normal releases). Require at least 1 vote for each platform (Windows/Unix) > and > at least 3 votes total. > >

Re: crash in serf credential callback

2014-03-02 Thread Lieven Govaerts
Should be fixed in r2316. On Wed, Feb 26, 2014 at 8:18 PM, Stefan Kueng wrote: > Hi, > > There's a crash happening in libsvn_ra_serf\util.c in the function > svn_ra_serf__credentials_callback. There's a full crash dump available from > here: > https://www.crash-server.com/Problem.aspx?ClientID=ts

Re: crash in serf credential callback

2014-03-02 Thread Lieven Govaerts
On Sun, Mar 2, 2014 at 12:17 AM, Lieven Govaerts wrote: > Update: > > On Fri, Feb 28, 2014 at 12:45 PM, Lieven Govaerts wrote: >> Hi, >> >> On Wed, Feb 26, 2014 at 8:18 PM, Stefan Kueng wrote: >>> Hi, >>> >>> There's

Re: crash in serf credential callback

2014-03-01 Thread Lieven Govaerts
Update: On Fri, Feb 28, 2014 at 12:45 PM, Lieven Govaerts wrote: > Hi, > > On Wed, Feb 26, 2014 at 8:18 PM, Stefan Kueng wrote: >> Hi, >> >> There's a crash happening in libsvn_ra_serf\util.c in the function >> svn_ra_serf__credentials_callback. There'

Re: crash in serf credential callback

2014-02-28 Thread Lieven Govaerts
Hi, On Wed, Feb 26, 2014 at 8:18 PM, Stefan Kueng wrote: > Hi, > > There's a crash happening in libsvn_ra_serf\util.c in the function > svn_ra_serf__credentials_callback. There's a full crash dump available from > here: > https://www.crash-server.com/Problem.aspx?ClientID=tsvn&ProblemID=58311 > >

Re: Bug in ra_serf with client certificates

2014-01-28 Thread Lieven Govaerts
On Tue, Jan 28, 2014 at 2:46 PM, Branko Čibej wrote: > On 28.01.2014 14:37, Lieven Govaerts wrote: > > On Tue, Jan 28, 2014 at 1:53 PM, Branko Čibej wrote: > > I just got a private report from a user that has a setup with a private > certificate. This user happened t

Re: Bug in ra_serf with client certificates

2014-01-28 Thread Lieven Govaerts
On Tue, Jan 28, 2014 at 1:53 PM, Branko Čibej wrote: > I just got a private report from a user that has a setup with a private > certificate. This user happened to select the wrong certificate for a > server, and got the following response: > > svn: E120171: Unable to connect to a repository at UR

Segfault in mod_dav_svn with repositories on /

2014-01-10 Thread Lieven Govaerts
Hi, a customer is seeing reproducible httpd child process segfaults when using Subversion. Setup: - Subversion 1.8.1 - WANDisco package subversion-1.8.1-1.x86_64.rpm - Apache 2.2.15 - RedHat package httpd-2.2.15-26.el6.x86_64 - OS: Red Hat Enterprise Linux Server release 6.4 - Client: svn, versi

Serf 1.3.3 released!

2013-12-09 Thread Lieven Govaerts
Hi, I'm pleased to announce the release of serf 1.3.3. This is a small patch release containing a fix to solve a problem connecting to multi-homed servers (e.g. ipv4/ipv6) and some improvements in the use of error codes during ssl certificate validation and handling of timed out connections. Yo

Re: Presenting net code changes for a branch

2013-10-13 Thread Lieven Govaerts
Hi Gabriela, On Sun, Oct 13, 2013 at 3:52 PM, Gabriela Gibson wrote: > On 13/10/13 13:34, Lieven Govaerts wrote: > >> >> Is this any different than running: >> svn diff -x -pwb ^/subversion/trunk >> ^/subversion/branches/invoke-**diff-cmd-feature >> ? >&g

Re: Presenting net code changes for a branch

2013-10-13 Thread Lieven Govaerts
Hi, On Sun, Oct 13, 2013 at 12:50 PM, Gabriela Gibson wrote: > Hi, > > my branch has grown into a veritable forest, and so, I thought that > it would be convenient to present the net code changes in a file called > 'entire-branch-code' (attached). > > This is generated like so: > > 1) Merge tru

Serf 1.3.2 released!

2013-10-04 Thread Lieven Govaerts
Hi everyone! I'm pleased to announce the release of serf 1.3.2. This is a patch release containing fixes to mostly minor issues that break http communication in specific scenario's. We also have more improvements for the scons configuration introduced in serf 1.3.0. You can see all the changes

Re: r1492168 - svn mergeinfo --log

2013-09-02 Thread Lieven Govaerts
On Wed, Jun 12, 2013 at 5:29 PM, Julian Foad wrote: > Lieven Govaerts wrote: > >> On Wed, Jun 12, 2013 at 2:06 PM, i...@apache.org wrote: >>> URL: http://svn.apache.org/r1492168 >>> Log: >>> Implement '--log' option for 'svn mergein

Re: Error during 'svn export' over http with serf 1.3.1

2013-08-24 Thread Lieven Govaerts
On Sun, Aug 25, 2013 at 1:36 AM, Johan Corveleyn wrote: > On Sun, Aug 25, 2013 at 1:25 AM, Lieven Govaerts > wrote: >> Summarising our investigation: >> 1. The error that Johan receives is caused by the body of a 501 Method >> Not Implemented response (see [1]) bein

Re: Error during 'svn export' over http with serf 1.3.1

2013-08-24 Thread Lieven Govaerts
Summarising our investigation: 1. The error that Johan receives is caused by the body of a 501 Method Not Implemented response (see [1]) being directly appended to a response to a GET request. 2. The request that triggered this response is this: DAV: http://subversion.tigris.org/xmlns/dav/svn/log-r

Re: Error during 'svn export' over http with serf 1.3.1

2013-08-24 Thread Lieven Govaerts
On Sat, Aug 24, 2013 at 12:03 PM, Johan Corveleyn wrote: > On Sat, Aug 24, 2013 at 11:13 AM, Ivan Zhakov wrote: > > On Saturday, August 24, 2013, Johan Corveleyn wrote: > >> > >> On Sat, Aug 24, 2013 at 10:17 AM, Johan Corveleyn > >> wrote: > >> > On Sat, Aug 24, 2013 at 6:20 AM, Ivan Zhakov >

Serf 1.3.1 released.

2013-08-15 Thread Lieven Govaerts
Hello everyone! I'm pleased to announce the release of serf 1.3.1. This release fixes issues in the authentication modules and ssl tunnel setup code, and an issue with TLS renegotiation. We also have more improvements for the scons configuration introduced in serf 1.3.0. As a consequence, serf n

Re: crash in svn_ra_serf__credentials_callback

2013-08-09 Thread Lieven Govaerts
Hi, these crashes can be explained by serf issue 120: "Error with ssl tunnel over proxy with KeepAlive off and Basic authentication". We're assuming here that these users were using a proxy with KeepAlive Off active (at least for the CONNECT request), but it's not possible to see that in this stac

Re: crash in svn_ra_serf__credentials_callback

2013-07-26 Thread Lieven Govaerts
On Fri, Jul 26, 2013 at 7:24 PM, Stefan Küng wrote: > Hi, > > And yet another crash, this time in svn_ra_serf__credentials_callback: > https://www.crash-server.com/Problem.aspx?ClientID=tsvn&ProblemID=31507 > >> libsvn_tsvn.dll!svn_ra_serf__credentials_callback(char * * >> username=0xf

Re: Trouble compiling using "--with-serf=/home/myUserName/local"

2013-07-23 Thread Lieven Govaerts
Hi Michael, On Tue, Jul 23, 2013 at 5:08 PM, Schuh, Michael (ARC-AA) wrote: [..] > > I tried installing APR 1.4.8 to see if it would work better with Serf. > > wget http://mirror.reverse.net/pub/apache//apr/apr-1.4.8.tar.bz2 > tar -xf apr-1.4.8.tar.bz2 > cd apr-1.4.8 > ./configure --prefi

Re: svnsync crashes on a huge commit

2013-07-17 Thread Lieven Govaerts
(bringing this to dev) Devs, Anatoly sent me some more info in this issue and log files of svnsync 1.8.0+serf 1.2.1 with logging enabled. He's running svnsync from a https repository (ra_serf) to a local repository (ra_local). This on an Ubuntu 12.08 VM with 64-bit binary. In the log files I rec

Re: svn commit: r1495419 - in /subversion/trunk/subversion/libsvn_ra_serf: options.c ra_serf.h serf.c util.c

2013-07-10 Thread Lieven Govaerts
On Wed, Jul 10, 2013 at 10:41 AM, Branko Čibej wrote: > On 10.07.2013 10:20, Branko Čibej wrote: >> On 10.07.2013 05:43, Greg Stein wrote: >>> On Tue, Jul 9, 2013 at 10:50 PM, Branko Čibej wrote: On 09.07.2013 05:53, Greg Stein wrote: ... > For *this* project, that is absolutely the

Re: svn commit: r1495419 - in /subversion/trunk/subversion/libsvn_ra_serf: options.c ra_serf.h serf.c util.c

2013-07-09 Thread Lieven Govaerts
On Tue, Jul 9, 2013 at 7:33 PM, Ben Reser wrote: > On Tue, Jul 9, 2013 at 1:25 AM, Stefan Sperling wrote: >> On Tue, Jul 09, 2013 at 12:45:49AM -0700, Ben Reser wrote: >>> The option doesn't seem like a big barrier when you're applying it to >>> a handful of machines for yourself. Scale that eff

Re: svn commit: r1501038 - /subversion/branches/1.8.x/STATUS

2013-07-09 Thread Lieven Govaerts
Hi Philip, On Tue, Jul 9, 2013 at 12:01 PM, Philip Martin wrote: > Lieven Govaerts writes: > >> Note, again: Serf will not have a feature to automatically detect that >> servers don't support chunked requests. Such a check is implemented in >> ra_serf with the extr

Re: svn commit: r1501038 - /subversion/branches/1.8.x/STATUS

2013-07-09 Thread Lieven Govaerts
On Tue, Jul 9, 2013 at 9:55 AM, Ben Reser wrote: [..] > Have we considered just turning off chunked requests entirely until > Serf has a fix to automatically detect and handle servers that don't > support chunked requests? Note, again: Serf will not have a feature to automatically detect that ser

Re: Commit error using 1.8.0 final

2013-07-05 Thread Lieven Govaerts
On Wed, Jun 19, 2013 at 11:26 AM, Joe Orton wrote: > On Tue, Jun 18, 2013 at 06:32:13PM +0200, Lieven Govaerts wrote: >> This 204 response is not a problem, there's a special case for >> Content-Length == 0 && code == 204 in serf (response_buckets.c). > > Look

Re: subversion client having issues talking to subversion repository over HTTPS via proxy

2013-07-03 Thread Lieven Govaerts
Hi, On Tue, Jun 25, 2013 at 11:17 PM, Lieven Govaerts wrote: > Hi, > > On Tue, Jun 25, 2013 at 9:37 PM, Greg Stein wrote: >> Ashish, >> >> This is normally a question for users@, but I think Lieven was working >> on something exactly like this already. I'v

Re: [serf-dev] Re: ssl tunnel with basic authentication currently broken (was Re: Another crash in ra_serf in 1.8.0)

2013-07-02 Thread Lieven Govaerts
On Wed, Jun 26, 2013 at 3:22 AM, Greg Stein wrote: > On Tue, Jun 25, 2013 at 5:43 PM, Lieven Govaerts > wrote: >>... >> When playing around with 'KeepAlive Off' on the server, I noticed that >> serf asks Subversion for the proxy credentials for every new >&

Re: 1.8.1

2013-07-01 Thread Lieven Govaerts
On Mon, Jul 1, 2013 at 8:30 PM, Daniel Shahaf wrote: > Branko Čibej wrote on Mon, Jul 01, 2013 at 20:12:11 +0200: >> On 01.07.2013 20:08, Daniel Shahaf wrote: >> > Lieven Govaerts wrote on Mon, Jul 01, 2013 at 19:44:24 +0200: >> >> On Mon, Jul 1, 2013 at 7:27 PM, M

Re: 1.8.1

2013-07-01 Thread Lieven Govaerts
On Mon, Jul 1, 2013 at 7:27 PM, Mark Phippard wrote: > On Mon, Jul 1, 2013 at 1:25 PM, Ben Reser wrote: >> I know this is a holiday week for some people but I'd like to start >> producing a 1.8.1 next Monday. So if everyone can look at the STATUS >> file for 1.8.x and get any things they want in

Re: svn commit: r1497980 - in /subversion/trunk/subversion/libsvn_ra_serf: options.c ra_serf.h serf.c

2013-06-29 Thread Lieven Govaerts
On Sat, Jun 29, 2013 at 5:17 AM, gst...@apache.org wrote: > Author: gstein > Date: Sat Jun 29 03:17:58 2013 > New Revision: 1497980 > > URL: http://svn.apache.org/r1497980 > Log: > Introduce dynamic detection proxies that cannot handle chunked requests. > > * subversion/libsvn_ra_serf/ra_serf.h: >

Re: svn commit: r1495419 - in /subversion/trunk/subversion/libsvn_ra_serf: options.c ra_serf.h serf.c util.c

2013-06-28 Thread Lieven Govaerts
On Fri, Jun 28, 2013 at 6:28 PM, Philip Martin wrote: > "Bert Huijben" writes: > >>> -Original Message- >>> From: Philip Martin [mailto:philip.mar...@wandisco.com] >>> Sent: vrijdag 28 juni 2013 17:18 >>> To: Ivan Zhakov >>> Cc: Greg Stein; dev@subversion.apache.org >>> Subject: Re: svn c

Re: svn commit: r1495419 - in /subversion/trunk/subversion/libsvn_ra_serf: options.c ra_serf.h serf.c util.c

2013-06-25 Thread Lieven Govaerts
On Wed, Jun 26, 2013 at 12:24 AM, Daniel Shahaf wrote: > Ivan Zhakov wrote on Wed, Jun 26, 2013 at 02:21:05 +0400: >> On Wed, Jun 26, 2013 at 2:12 AM, Daniel Shahaf wrote: >> > Johan Corveleyn wrote on Tue, Jun 25, 2013 at 23:22:12 +0200: >> >> On Tue, Jun 25, 2013 at 11:03 PM, Ivan Zhakov wrote

Re: [serf-dev] Re: ssl tunnel with basic authentication currently broken (was Re: Another crash in ra_serf in 1.8.0)

2013-06-25 Thread Lieven Govaerts
On Sun, Jun 23, 2013 at 9:15 PM, Lieven Govaerts wrote: > On Sun, Jun 23, 2013 at 3:20 AM, Greg Stein wrote: >> On Sat, Jun 22, 2013 at 3:26 PM, Lieven Govaerts wrote: >>> On Sat, Jun 22, 2013 at 7:32 PM, Lieven Govaerts wrote: >>>> Stefan, >>>> >

Re: svn commit: r1495419 - in /subversion/trunk/subversion/libsvn_ra_serf: options.c ra_serf.h serf.c util.c

2013-06-25 Thread Lieven Govaerts
On Tue, Jun 25, 2013 at 5:15 PM, Branko Čibej wrote: > On 25.06.2013 15:37, Ivan Zhakov wrote: >> On Tue, Jun 25, 2013 at 4:15 PM, Ivan Zhakov wrote: >>> On Sun, Jun 23, 2013 at 3:43 AM, Greg Stein wrote: On Fri, Jun 21, 2013 at 11:36 AM, Ivan Zhakov wrote: > ... > The fix seems to

Re: subversion client having issues talking to subversion repository over HTTPS via proxy

2013-06-25 Thread Lieven Govaerts
Hi, On Tue, Jun 25, 2013 at 9:37 PM, Greg Stein wrote: > Ashish, > > This is normally a question for users@, but I think Lieven was working > on something exactly like this already. I've cc'd him for more detail. Ashish already reported this in the serf issue tracker: https://code.google.com/p/s

Re: svn commit: r1495419 - in /subversion/trunk/subversion/libsvn_ra_serf: options.c ra_serf.h serf.c util.c

2013-06-25 Thread Lieven Govaerts
On Tue, Jun 25, 2013 at 8:45 PM, Greg Stein wrote: > On Tue, Jun 25, 2013 at 11:55 AM, Philip Martin > wrote: >> Branko Čibej writes: >> >>> I'm really not a fan of this config knob. Anyone who carries their >>> laptop around will effectively have to set this as the default, because >>> you neve

Re: [serf-dev] Re: ssl tunnel with basic authentication currently broken (was Re: Another crash in ra_serf in 1.8.0)

2013-06-23 Thread Lieven Govaerts
On Sun, Jun 23, 2013 at 3:20 AM, Greg Stein wrote: > On Sat, Jun 22, 2013 at 3:26 PM, Lieven Govaerts wrote: >> On Sat, Jun 22, 2013 at 7:32 PM, Lieven Govaerts wrote: >>> Stefan, >>> >>> attached patch to serf 1.2.1 should solve this particular type of >&

Re: [PATCH] Crash in TSVN with directory property conflict during update (was Re: Another crash in ra_serf in 1.8.0)

2013-06-23 Thread Lieven Govaerts
On Sun, Jun 23, 2013 at 4:29 PM, Stefan Küng wrote: > On 23.06.2013 15:07, Lieven Govaerts wrote: >> Stefan, >> >> >> this issue is unrelated to ssl tunnelling or authentication, or more >> in general unrelated to ra_serf. >> >> The particular scen

[PATCH] Crash in TSVN with directory property conflict during update (was Re: Another crash in ra_serf in 1.8.0)

2013-06-23 Thread Lieven Govaerts
patch fixes the issue for TSVN users? thanks, Lieven On Fri, Jun 21, 2013 at 9:26 PM, Stefan Küng wrote: > On 20.06.2013 23:05, Lieven Govaerts wrote: > > Looks like it's the authentication handling when setting up a SSL tunnel > that's at fault here, at least I can easil

Re: ssl tunnel with basic authentication currently broken (was Re: Another crash in ra_serf in 1.8.0)

2013-06-22 Thread Lieven Govaerts
On Sat, Jun 22, 2013 at 7:32 PM, Lieven Govaerts wrote: > Stefan, > > attached patch to serf 1.2.1 should solve this particular type of > crash you reported. > > The patch is made against a serf 1.2.x working copy as follows: > $ svn merge ^/trunk -c 1943,1944 Unfortunate

Re: ssl tunnel with basic authentication currently broken (was Re: Another crash in ra_serf in 1.8.0)

2013-06-22 Thread Lieven Govaerts
Stefan, attached patch to serf 1.2.1 should solve this particular type of crash you reported. The patch is made against a serf 1.2.x working copy as follows: $ svn merge ^/trunk -c 1943,1944 On Fri, Jun 21, 2013 at 8:55 AM, Lieven Govaerts wrote: > Follow up also to serf-dev. > > On

Re: svn commit: r1495419 - in /subversion/trunk/subversion/libsvn_ra_serf: options.c ra_serf.h serf.c util.c

2013-06-22 Thread Lieven Govaerts
On Sat, Jun 22, 2013 at 10:24 AM, Johan Corveleyn wrote: > On Sat, Jun 22, 2013 at 10:07 AM, Lieven Govaerts wrote: >> On Sat, Jun 22, 2013 at 9:08 AM, Philip Martin >> wrote: >>> Ivan Zhakov writes: >>> >>>> Problem with this approach that some ser

Re: svn commit: r1495419 - in /subversion/trunk/subversion/libsvn_ra_serf: options.c ra_serf.h serf.c util.c

2013-06-22 Thread Lieven Govaerts
On Sat, Jun 22, 2013 at 10:07 AM, Lieven Govaerts wrote: > On Sat, Jun 22, 2013 at 9:08 AM, Philip Martin > wrote: >> Ivan Zhakov writes: >> >>> Problem with this approach that some servers may support HTTP/1.1 >>> partially. I.e. declare them as HTTP/1.1 but

Re: svn commit: r1495419 - in /subversion/trunk/subversion/libsvn_ra_serf: options.c ra_serf.h serf.c util.c

2013-06-22 Thread Lieven Govaerts
On Sat, Jun 22, 2013 at 9:08 AM, Philip Martin wrote: > Ivan Zhakov writes: > >> Problem with this approach that some servers may support HTTP/1.1 >> partially. I.e. declare them as HTTP/1.1 but do not support chunked >> Transfer-Encoding. >> >> I wanted to avoid downgrade to HTTP/1.0 on later re

ssl tunnel with basic authentication currently broken (was Re: Another crash in ra_serf in 1.8.0)

2013-06-20 Thread Lieven Govaerts
Follow up also to serf-dev. On Thu, Jun 20, 2013 at 11:05 PM, Lieven Govaerts wrote: > On Thu, Jun 20, 2013 at 10:30 PM, Greg Stein wrote: >> On Thu, Jun 20, 2013 at 2:19 PM, Stefan Küng wrote: >>> Hi, >>> >>> Another crash that's climbing up in the cr

Re: Another crash in ra_serf in 1.8.0

2013-06-20 Thread Lieven Govaerts
On Thu, Jun 20, 2013 at 10:30 PM, Greg Stein wrote: > On Thu, Jun 20, 2013 at 2:19 PM, Stefan Küng wrote: >> Hi, >> >> Another crash that's climbing up in the crash report statistics for TSVN. >> Seems to be related to the previously discussed problem with checkouts in >> TSVN. Thanks Stefan. >

Re: [serf-dev] [Patch] Adding NTLM Support to Serf - Work in progress / Subversion regression

2013-06-20 Thread Lieven Govaerts
On Thu, Jun 20, 2013 at 5:27 PM, Ivan Zhakov wrote: > Mark, > .. > Also please believe that all my technical thoughts are fair and > related to technical issues only. My veto above is a technical veto. I personally never doubted this based on your first mail, but that's just because I know serf a

[PATCH] for 1.8 release notes (was Re: Missing info in release notes about http bulk updates config options)

2013-06-19 Thread Lieven Govaerts
Hi, Attached a patch to the site branch that tries to explain SVNAllowBulkUpdates and http-bulk-updates in the 1.8 release notes. Anyone care to review and fix where needed? I'm not a lot at my pc tonight and tomorrow so feel free to commit directly. regards, Lieven On Wed, Jun 19, 2013 at 3

Re: Commit error using 1.8.0 final

2013-06-18 Thread Lieven Govaerts
On Tue, Jun 18, 2013 at 3:20 PM, Mark Phippard wrote: > On Mon, Jun 17, 2013 at 4:00 PM, Lieven Govaerts wrote: >> On Mon, Jun 17, 2013 at 9:22 PM, Mark Phippard wrote: >>> I updated my Windows laptop to 1.8.0 final. I am trying to commit a >>> change for Subcli

Re: Commit error using 1.8.0 final

2013-06-17 Thread Lieven Govaerts
On Tue, Jun 18, 2013 at 1:33 AM, Mark Phippard wrote: > On Mon, Jun 17, 2013 at 4:00 PM, Lieven Govaerts wrote: >> On Mon, Jun 17, 2013 at 9:22 PM, Mark Phippard wrote: >>> I updated my Windows laptop to 1.8.0 final. I am trying to commit a >>> change for Subcli

Re: Commit error using 1.8.0 final

2013-06-17 Thread Lieven Govaerts
On Mon, Jun 17, 2013 at 9:22 PM, Mark Phippard wrote: > I updated my Windows laptop to 1.8.0 final. I am trying to commit a > change for Subclipse to tigris.org and it is failing. The error > message is not that helpful. Any ideas how to get better error > information? There might be a higher

Re: Missing info in release notes about http bulk updates config options

2013-06-14 Thread Lieven Govaerts
I'm working on a text to describe how these work, including two tables detailing all possible combinations. Lieven On Thu, Jun 13, 2013 at 1:20 AM, Johan Corveleyn wrote: > Before I forget, I just noticed that there is no mention anywhere in > the release notes about the http-bulk-updates or the

Re: svn commit: r1492168 - in /subversion/trunk/subversion: svn/cl.h svn/mergeinfo-cmd.c svn/svn.c tests/cmdline/mergeinfo_tests.py

2013-06-12 Thread Lieven Govaerts
On Wed, Jun 12, 2013 at 2:06 PM, i...@apache.org wrote: > Author: ivan > Date: Wed Jun 12 12:06:03 2013 > New Revision: 1492168 > > URL: http://svn.apache.org/r1492168 > Log: > Implement '--log' option for 'svn mergeinfo --show-revs' subcommand to print > revisions log message, author and date. T

Re: Should missing smart card support not be added in the release notes?

2013-06-05 Thread Lieven Govaerts
Hi, On Tue, Jun 4, 2013 at 8:21 PM, Stefan Küng wrote: > On 04.06.2013 13:56, Lieven Govaerts wrote: > >> You are referring to a configuration where OpenSSL uses MS's CryptoAPI >> to use the Windows certificate store. Never used it myself, but I see >> that TSVN

Re: Should missing smart card support not be added in the release notes?

2013-06-04 Thread Lieven Govaerts
On Tue, Jun 4, 2013 at 1:25 PM, Ivan Zhakov wrote: > On Tue, Jun 4, 2013 at 3:19 PM, Lieven Govaerts wrote: >> On Tue, Jun 4, 2013 at 12:55 PM, Ivan Zhakov wrote: >>> On Tue, Jun 4, 2013 at 2:51 PM, Lieven Govaerts wrote: >>>> Hi, >>>> >>>&g

Re: Should missing smart card support not be added in the release notes?

2013-06-04 Thread Lieven Govaerts
On Tue, Jun 4, 2013 at 1:25 PM, Ivan Zhakov wrote: > On Tue, Jun 4, 2013 at 3:19 PM, Lieven Govaerts wrote: >> On Tue, Jun 4, 2013 at 12:55 PM, Ivan Zhakov wrote: >>> On Tue, Jun 4, 2013 at 2:51 PM, Lieven Govaerts wrote: >>>> Hi, >>>> >>>&g

Re: Should missing smart card support not be added in the release notes?

2013-06-04 Thread Lieven Govaerts
On Tue, Jun 4, 2013 at 12:55 PM, Ivan Zhakov wrote: > On Tue, Jun 4, 2013 at 2:51 PM, Lieven Govaerts wrote: >> Hi, >> >> >> see subject. Serf and ra_serf don't have smart card support at this >> moment, unlike neon. >> >> I'd expected

Should missing smart card support not be added in the release notes?

2013-06-04 Thread Lieven Govaerts
Hi, see subject. Serf and ra_serf don't have smart card support at this moment, unlike neon. I'd expected this to be mentioned in the release notes for 1.8.0 as this is not new information (at least I hope so), but I can't find anything about it. Lieven.

Re: issues dump ?

2013-05-10 Thread Lieven Govaerts
On Fri, May 10, 2013 at 2:06 AM, Branko Čibej wrote: > On 10.05.2013 01:08, Ben Reser wrote: > > On Thu, May 9, 2013 at 4:04 PM, Daniel Shahaf > wrote: > >> Can we get a periodic data dump of subversion.tigris.org/issues/ onto > >> ASF hardware? > > Drive by comment here but we should probably s

Re: svn commit: r1464228 - in /subversion/trunk/subversion/libsvn_ra_serf: sb_bucket.c util.c

2013-04-04 Thread Lieven Govaerts
Hi Branko, On Thu, Apr 4, 2013 at 8:59 PM, Branko Čibej wrote: > On 04.04.2013 20:46, Lieven Govaerts wrote: >> On Thu, Apr 4, 2013 at 1:28 PM, Philip Martin >> wrote: >>> Lieven Govaerts writes: >>> >>>> On Thu, Apr 4, 2013 at 12:55 AM, phi...@

Re: svn commit: r1464228 - in /subversion/trunk/subversion/libsvn_ra_serf: sb_bucket.c util.c

2013-04-04 Thread Lieven Govaerts
On Thu, Apr 4, 2013 at 1:28 PM, Philip Martin wrote: > Lieven Govaerts writes: > >> On Thu, Apr 4, 2013 at 12:55 AM, phi...@apache.org wrote: >>> Author: philip >>> Date: Wed Apr 3 22:55:37 2013 >>> New Revision: 1464228 >>> >>> URL:

Re: svn commit: r1464228 - in /subversion/trunk/subversion/libsvn_ra_serf: sb_bucket.c util.c

2013-04-04 Thread Lieven Govaerts
Hi Philip, On Thu, Apr 4, 2013 at 12:55 AM, phi...@apache.org wrote: > Author: philip > Date: Wed Apr 3 22:55:37 2013 > New Revision: 1464228 > > URL: http://svn.apache.org/r1464228 > Log: > Remove (void) casts of ignored return values from ra_serf. > > * subversion/libsvn_ra_serf/util.c > (sv

Re: BDB vs FSFS - OMG!

2013-01-06 Thread Lieven Govaerts
Hi, On Sun, Jan 6, 2013 at 10:43 AM, Bert Huijben wrote: > The revprop and revision cache are in fsfs, not the repos layer... > > In what way are you then comparing the backends? > > You are now comparing a backend+caching to a backend without caching. > > I’m not against dropping support, but if

Re: What version of Serf will be the required minimum for Subversion 1.8?

2013-01-03 Thread Lieven Govaerts
Hi Mike, In short: for me serf 1.2.0 should be the minimum version. Its release blocks on gstein for now, but I'm ready to release it in time for svn 1.8.0. On Thu, Jan 3, 2013 at 5:02 PM, C. Michael Pilato wrote: > Hey, all. I'd like to get a sense of what we're targeting in terms of > minimum

Re: backgrounding 'svn log' over serf

2012-12-19 Thread Lieven Govaerts
On Wed, Dec 19, 2012 at 8:31 PM, Philip Martin wrote: > "C. Michael Pilato" writes: > >> On 12/19/2012 01:59 PM, C. Michael Pilato wrote: >>> On 12/19/2012 01:53 PM, Daniel Shahaf wrote: Actual result: backgrounding svn killed it. Expected result: after 'bg\n', keep fetching+filter

Re: svn commit: r1417639 - in /subversion/trunk/subversion/mod_dav_svn: dav_svn.h mod_dav_svn.c reports/update.c

2012-12-13 Thread Lieven Govaerts
Mike, On Thu, Dec 13, 2012 at 11:06 PM, C. Michael Pilato wrote: > On 12/13/2012 05:48 AM, Justin Erenkrantz wrote: >> On Wed, Dec 12, 2012 at 4:24 PM, C. Michael Pilato > > wrote: >> >> Here's a question I've been wondering for some time: should we expose to >>

Re: svn commit: r1417639 - in /subversion/trunk/subversion/mod_dav_svn: dav_svn.h mod_dav_svn.c reports/update.c

2012-12-12 Thread Lieven Govaerts
Mike, On Wed, Dec 12, 2012 at 10:15 PM, C. Michael Pilato wrote: > On 12/12/2012 03:30 PM, C. Michael Pilato wrote: >> On 12/12/2012 03:02 PM, Lieven Govaerts wrote: >>> On Tue, Dec 11, 2012 at 9:29 PM, C. Michael Pilato >>> wrote: >>>> Those technical

Re: svn commit: r1417639 - in /subversion/trunk/subversion/mod_dav_svn: dav_svn.h mod_dav_svn.c reports/update.c

2012-12-12 Thread Lieven Govaerts
On Wed, Dec 12, 2012 at 9:30 PM, C. Michael Pilato wrote: > On 12/12/2012 03:02 PM, Lieven Govaerts wrote: >> On Tue, Dec 11, 2012 at 9:29 PM, C. Michael Pilato >> wrote: >>> Those technical challenges aside, I've since started to doubt the wisdom of >>> ad

Re: svn commit: r1417639 - in /subversion/trunk/subversion/mod_dav_svn: dav_svn.h mod_dav_svn.c reports/update.c

2012-12-12 Thread Lieven Govaerts
On Tue, Dec 11, 2012 at 9:29 PM, C. Michael Pilato wrote: > On 12/10/2012 10:53 AM, C. Michael Pilato wrote: >> What if we revved the svn_ra_replay_range() API in such a way that it could >> now handle this "initial revision" scenario? We might add an 'incremental' >> flag that parallels what 'sv

Re: svn commit: r1415864 - /subversion/trunk/subversion/libsvn_ra_serf/update.c

2012-12-10 Thread Lieven Govaerts
Hi Markus, On Mon, Dec 3, 2012 at 10:43 AM, Markus Schaber wrote: > Hi, > > Just another crazy idea: > > The main problem with the parallelization in ra_serf seems to be the number > of http requests (which potentially causes a high number of authentications > and tcp connections). > > Maybe we

Re: svn commit: r1417642 - in /subversion/trunk/subversion: include/svn_config.h libsvn_ra_serf/ra_serf.h libsvn_ra_serf/serf.c libsvn_ra_serf/update.c libsvn_subr/config_file.c

2012-12-06 Thread Lieven Govaerts
Hi Peter, On Fri, Dec 7, 2012 at 6:25 AM, Peter Samuelson wrote: > >> > "### http-library Which library to use for >> > http/https" >> > >> >NL >> > "###

Re: svn commit: r1417639 - in /subversion/trunk/subversion/mod_dav_svn: dav_svn.h mod_dav_svn.c reports/update.c

2012-12-06 Thread Lieven Govaerts
On Wed, Dec 5, 2012 at 10:50 PM, Stefan Sperling wrote: > On Wed, Dec 05, 2012 at 03:54:00PM -0500, Mark Phippard wrote: >> On Wed, Dec 5, 2012 at 3:46 PM, wrote: >> > Author: lgo >> > Date: Wed Dec 5 20:46:33 2012 >> > New Revision: 1417639 >> > >> > URL: http://svn.apache.org/viewvc?rev=14176

Re: svn commit: r1417639 - in /subversion/trunk/subversion/mod_dav_svn: dav_svn.h mod_dav_svn.c reports/update.c

2012-12-06 Thread Lieven Govaerts
Hi, On Thu, Dec 6, 2012 at 11:04 PM, Mark Phippard wrote: > On Thu, Dec 6, 2012 at 4:40 PM, Lieven Govaerts wrote: > >> It is relevant because we already have a SVNAllowBulkUpdates flag that >> gives the admin some options to define this exact behavior. That flag >> gi

Re: svn commit: r1417639 - in /subversion/trunk/subversion/mod_dav_svn: dav_svn.h mod_dav_svn.c reports/update.c

2012-12-06 Thread Lieven Govaerts
Hi, On Thu, Dec 6, 2012 at 3:44 PM, Mark Phippard wrote: > On Wed, Dec 5, 2012 at 6:18 PM, Lieven Govaerts wrote: >> On Wed, Dec 5, 2012 at 9:54 PM, Mark Phippard wrote: >>> On Wed, Dec 5, 2012 at 3:46 PM, wrote: >>>> Author: lgo >>>> Date: Wed Dec

Re: svn commit: r1417639 - in /subversion/trunk/subversion/mod_dav_svn: dav_svn.h mod_dav_svn.c reports/update.c

2012-12-05 Thread Lieven Govaerts
On Wed, Dec 5, 2012 at 9:54 PM, Mark Phippard wrote: > On Wed, Dec 5, 2012 at 3:46 PM, wrote: >> Author: lgo >> Date: Wed Dec 5 20:46:33 2012 >> New Revision: 1417639 >> >> URL: http://svn.apache.org/viewvc?rev=1417639&view=rev >> Log: >> Add a Force option to SVNAllowBulkUpdates. This allows a

Re: non-skelta update editor mode in ra_serf (was Re: svn commit: r1415864 - /subversion/trunk/subversion/libsvn_ra_serf/update.c)

2012-12-05 Thread Lieven Govaerts
Hi, On Sat, Dec 1, 2012 at 10:18 PM, Ivan Zhakov wrote: > [ changing subject to make topic more visible] > > On Sat, Dec 1, 2012 at 9:00 PM, Mark Phippard wrote: >> On Sat, Dec 1, 2012 at 12:36 AM, Justin Erenkrantz >> wrote: >>> On Fri, Nov 30, 2012 at 4:54 PM, wrote: Author: cmpila

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-c

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 ho

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/

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: svn commit: r1415864 - /subversion/trunk/subversion/libsvn_ra_serf/update.c

2012-12-01 Thread Lieven Govaerts
On Sat, Dec 1, 2012 at 2:31 PM, Justin Erenkrantz wrote: > On Sat, Dec 1, 2012 at 5:59 AM, Johan Corveleyn wrote: >> >> I'm wondering whether your concerns apply to both internet-wide >> deployments and local (all on the same LAN) ones. > > > That line is certainly a fair one to draw in the sand.

Re: [serf-dev] Re: Double compression over HTTPS

2012-11-14 Thread Lieven Govaerts
On Thu, Nov 15, 2012 at 1:01 AM, Greg Stein wrote: > > On Nov 14, 2012 1:44 AM, "Lieven Govaerts" wrote: >>... > > >> Implemented in serf r1692, with an API for applications to enable SSL >> compression. Which to be clear, I don't propose svn to use.

  1   2   3   >