Re: svn commit: r1390674 - /subversion/trunk/build/ac-macros/berkeley-db.m4

2012-09-26 Thread Greg Stein
On Wed, Sep 26, 2012 at 6:55 PM, Ben Reser wrote: > On Wed, Sep 26, 2012 at 3:29 PM, Peter Samuelson wrote: >> Interestingly, the case you tried to fix - libraries with >> Debian-derived packaging - is also a case where it _is_ safe to link to >> multiple versions of db in the same app. But you'

Re: General question on Serf

2012-09-26 Thread Greg Stein
On Wed, Sep 26, 2012 at 2:11 PM, Philip Martin wrote: > Lieven Govaerts writes: > >> If the intention is to instruct the proxy that a cached response is >> only valid for a certain X-SVN-VR-Base header, then that is what the >> "Vary" header is for. >> So with: >> Vary: X-SVN-VR-Base >> in the re

Re: [PATCH] Fix for issue #4234, svndumpfilter exclude --targets wants pathname to start with '/'

2012-09-26 Thread vijay
On Wednesday 26 September 2012 03:19 PM, Stefan Sperling wrote: On Wed, Sep 26, 2012 at 03:52:57AM +0530, vijay wrote: From the issue[1] description, When svndumpfilter exclude command accepts path prefixes starting with either / or just , I expect it to work the same way when we use with -

Re: svn commit: r1390674 - /subversion/trunk/build/ac-macros/berkeley-db.m4

2012-09-26 Thread Ben Reser
On Wed, Sep 26, 2012 at 3:29 PM, Peter Samuelson wrote: > Interestingly, the case you tried to fix - libraries with > Debian-derived packaging - is also a case where it _is_ safe to link to > multiple versions of db in the same app. But you're right, on many > other platforms, it is not safe. Ho

Re: svn commit: r1390674 - /subversion/trunk/build/ac-macros/berkeley-db.m4

2012-09-26 Thread Peter Samuelson
[Ben Reser] > Just confirming what brane said above is the reason why I did not try > to just build detection of BDB into our configure system. Figuring > out if it's safe to link to some BDB that we find ourselves is > actually really hard. If we could ask APR-Util what version of BDB it > was

Re: svn commit: r1390674 - /subversion/trunk/build/ac-macros/berkeley-db.m4

2012-09-26 Thread Ben Reser
On Wed, Sep 26, 2012 at 2:41 PM, Branko Čibej wrote: > In fact, there is. If mod_dav_svn is linked with a different version of > bdb and/or sqlite than the rest of httpd, it will break at best, or > corrupt your repo in the worst case. Just confirming what brane said above is the reason why I did

Re: svn commit: r1390674 - /subversion/trunk/build/ac-macros/berkeley-db.m4

2012-09-26 Thread Branko Čibej
On 26.09.2012 23:35, Peter Samuelson wrote: > [bre...@apache.org] >> + # We pass --dbm-libs here since Debian has modified apu-config not >> + # to return -ldb unless --dbm-libs is passed. > Better would be not to rely on apr-util to figure out where db is at > all. We're not using the a

Re: svn commit: r1390674 - /subversion/trunk/build/ac-macros/berkeley-db.m4

2012-09-26 Thread Peter Samuelson
[bre...@apache.org] > + # We pass --dbm-libs here since Debian has modified apu-config not > + # to return -ldb unless --dbm-libs is passed. Better would be not to rely on apr-util to figure out where db is at all. We're not using the apu wrapper for db, after all, we're calling the db

Re: General question on Serf

2012-09-26 Thread Philip Martin
Lieven Govaerts writes: > If the intention is to instruct the proxy that a cached response is > only valid for a certain X-SVN-VR-Base header, then that is what the > "Vary" header is for. > So with: > Vary: X-SVN-VR-Base > in the response, the proxy should only return a cached response if the >

Please vote for 1.7.x backports this week!

2012-09-26 Thread Ben Reser
stsp poked me on IRC to point out that we've got quite a few candidates piling up for 1.7.x. So I'd like to start the process for 1.7.7. I'm planning to roll tarballs on October 3rd which is one week from today. Given Subversion Live and that a number of us will be traveling for the next couple

Re: General question on Serf

2012-09-26 Thread Lieven Govaerts
On Wed, Sep 26, 2012 at 7:27 PM, C. Michael Pilato wrote: > On 09/26/2012 12:28 PM, Philip Martin wrote: >> Branko Čibej writes: >> >>> This is how the cache /should/ work, according to any number of HTTP >>> specs. Unfortunately, most proxies that I know about don't attempt >>> anything as "adva

Re: General question on Serf

2012-09-26 Thread C. Michael Pilato
On 09/26/2012 12:28 PM, Philip Martin wrote: > Branko Čibej writes: > >> This is how the cache /should/ work, according to any number of HTTP >> specs. Unfortunately, most proxies that I know about don't attempt >> anything as "advanced" as that. > > If the cache ignores X-SVN-VR-Base and simply

Re: General question on Serf

2012-09-26 Thread Philip Martin
Bert Huijben writes: > To what url was this request? > > I don't think it is to the public URL, or is it? > > This might be part of the answer. It's the response to the GET I posted earlier in the thread. It's a request for /f@2 with a base revision of /f@1: GET /obj/repo/!svn/rvr/2/f HTTP/1.1

RE: General question on Serf

2012-09-26 Thread Bert Huijben
To what url was this request? I don't think it is to the public URL, or is it? This might be part of the answer. Bert Huijben (Cell phone) From: Philip Martin Sent: 26-9-2012 18:29 To: Branko Čibej Cc: dev@subversion.apache.org Subject: Re: General question on Serf Branko Čibej writes: > This

Re: General question on Serf

2012-09-26 Thread Philip Martin
Branko Čibej writes: > This is how the cache /should/ work, according to any number of HTTP > specs. Unfortunately, most proxies that I know about don't attempt > anything as "advanced" as that. If the cache ignores X-SVN-VR-Base and simply caches the response that will break 1.8 clients. Looki

Re: svn commit: r1390435 - in /subversion/trunk: ./ subversion/include/private/ subversion/libsvn_delta/ subversion/libsvn_fs_fs/ subversion/libsvn_fs_util/ subversion/libsvn_repos/ subversion/libsvn_

2012-09-26 Thread Lieven Govaerts
Hi, On Wed, Sep 26, 2012 at 3:45 PM, Ivan Zhakov wrote: > On Wed, Sep 26, 2012 at 4:28 PM, wrote: >> Author: stefan2 >> Date: Wed Sep 26 12:28:26 2012 >> New Revision: 1390435 >> >> URL: http://svn.apache.org/viewvc?rev=1390435&view=rev >> Log: >> Merge first batch of changes from the 10Gb bran

Re: General question on Serf

2012-09-26 Thread Branko Čibej
On 26.09.2012 17:48, Philip Martin wrote: > Mark Phippard writes: > >> I have always been skeptical of the ability to put a cache in front of the >> SVN server. Wouldn't something like this keep a cache from working >> properly? How would a cache know about that header we added and that the >> c

Re: General question on Serf

2012-09-26 Thread Philip Martin
Mark Phippard writes: > I have always been skeptical of the ability to put a cache in front of the > SVN server. Wouldn't something like this keep a cache from working > properly? How would a cache know about that header we added and that the > content we returned to the client cannot be cached

Re: svn commit: r1390443 - in /subversion/trunk: ./ subversion/include/svn_ra_svn.h subversion/libsvn_ra_svn/client.c subversion/libsvn_ra_svn/editorp.c subversion/libsvn_ra_svn/marshal.c

2012-09-26 Thread Stefan Fuhrmann
On Wed, Sep 26, 2012 at 3:40 PM, C. Michael Pilato wrote: > On 09/26/2012 08:47 AM, stef...@apache.org wrote: > > Author: stefan2 > > Date: Wed Sep 26 12:47:23 2012 > > New Revision: 1390443 > > [...] > > > Modified: subversion/trunk/subversion/libsvn_ra_svn/client.c > > URL: > http://svn.apache.o

Re: General question on Serf

2012-09-26 Thread C. Michael Pilato
On 09/26/2012 09:51 AM, C. Michael Pilato wrote: > On 09/26/2012 09:40 AM, Philip Martin wrote: >> Serf sends the delta base to the server (what does TEMPORARILY mean >> here?): >> >> /** This header is *TEMPORARILY* used to transmit the delta base to the >> * server. It contains a version resourc

Re: General question on Serf

2012-09-26 Thread Mark Phippard
On Wed, Sep 26, 2012 at 9:44 AM, Mark Phippard wrote: > On Wed, Sep 26, 2012 at 9:40 AM, Philip Martin > wrote: > >> Mark Phippard writes: >> >> > What happens with Serf? From what I know, I would assume that Serf has >> to >> > do a GET of the entire file. Is this true or does it somehow onl

Re: General question on Serf

2012-09-26 Thread Philip Martin
Philip Martin writes: > GET /obj/repo/!svn/rvr/2/f HTTP/1.1 > Host: localhost: > User-Agent: SVN/1.8.0-dev serf/1.1.1 > DAV: http://subversion.tigris.org/xmlns/dav/svn/depth > DAV: http://subversion.tigris.org/xmlns/dav/svn/mergeinfo > DAV: http://subversion.tigris.org/xmlns/dav/svn/log-revpr

Re: General question on Serf

2012-09-26 Thread C. Michael Pilato
On 09/26/2012 09:40 AM, Philip Martin wrote: > Serf sends the delta base to the server (what does TEMPORARILY mean > here?): > > /** This header is *TEMPORARILY* used to transmit the delta base to the > * server. It contains a version resource URL for what is on the client. > */ > #define SVN_DA

Re: svn commit: r1390435 - in /subversion/trunk: ./ subversion/include/private/ subversion/libsvn_delta/ subversion/libsvn_fs_fs/ subversion/libsvn_fs_util/ subversion/libsvn_repos/ subversion/libsvn_

2012-09-26 Thread Ivan Zhakov
On Wed, Sep 26, 2012 at 4:28 PM, wrote: > Author: stefan2 > Date: Wed Sep 26 12:28:26 2012 > New Revision: 1390435 > > URL: http://svn.apache.org/viewvc?rev=1390435&view=rev > Log: > Merge first batch of changes from the 10Gb branch. These are all > internal optimizations, i.e. no UI or public AP

Re: General question on Serf

2012-09-26 Thread Mark Phippard
On Wed, Sep 26, 2012 at 9:40 AM, Philip Martin wrote: > Mark Phippard writes: > > > What happens with Serf? From what I know, I would assume that Serf has > to > > do a GET of the entire file. Is this true or does it somehow only > request > > a delta? > > Serf sends the delta base to the serve

Re: General question on Serf

2012-09-26 Thread Philip Martin
Mark Phippard writes: > What happens with Serf? From what I know, I would assume that Serf has to > do a GET of the entire file. Is this true or does it somehow only request > a delta? Serf sends the delta base to the server (what does TEMPORARILY mean here?): /** This header is *TEMPORARILY*

Re: svn commit: r1390443 - in /subversion/trunk: ./ subversion/include/svn_ra_svn.h subversion/libsvn_ra_svn/client.c subversion/libsvn_ra_svn/editorp.c subversion/libsvn_ra_svn/marshal.c

2012-09-26 Thread C. Michael Pilato
On 09/26/2012 08:47 AM, stef...@apache.org wrote: > Author: stefan2 > Date: Wed Sep 26 12:47:23 2012 > New Revision: 1390443 [...] > Modified: subversion/trunk/subversion/libsvn_ra_svn/client.c > URL: > http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_ra_svn/client.c?rev=1390443&r

RE: General question on Serf

2012-09-26 Thread Bert Huijben
I think I can answer this question, but maybe one of the ra_serf gurus can fill in the details. With the get request serf sends a header to tell the server which version of the file the client already knows. The server than usually sends the delta, just like neon or svnserve do/did. The server

General question on Serf

2012-09-26 Thread Mark Phippard
I have a question about how Serf works that I keep forgetting to send to this list. When you do an update with Serf, my understanding is that it does a REPORT request that gives it back a list of files that it needs to update and then it fires off a bunch of GET requests to retrieve those files.

Re: [PATCH] Fix for issue #4234, svndumpfilter exclude --targets wants pathname to start with '/'

2012-09-26 Thread Stefan Sperling
On Wed, Sep 26, 2012 at 03:52:57AM +0530, vijay wrote: > From the issue[1] description, > > > > When svndumpfilter exclude command accepts path prefixes starting with either > / > or just , I expect it to work the same way when we use with > --targets option. > > For Eg : > > Both of the foll