Re: svnrdump: The BIG update

2010-08-12 Thread Daniel Shahaf
Ramkumar Ramachandra wrote on Thu, Aug 12, 2010 at 12:17:34 +0530: > > > The dump functionality is also complete- thanks to Stefan's review and > > > MANY others for cleaning it up. It's however hit a brick wall now > > > because of missing headers in the RA layer. Until I (or someone else) > > > f

Re: NODE_DATA (2nd iteration)

2010-08-12 Thread Neels J Hofmeyr
I did draw some sort of chart of NODE_DATA in wc-metadata.sql... [[[ For illustration, with a scenario like this: # (0) svn rm foo svn cp ^/moo foo # (1) svn rm foo/bar touch foo/bar svn add foo/bar# (2) , these are the NODE_DATA for the path foo/bar (be

Re: svn_wc_translated_file3

2010-08-12 Thread Julian Foad
On Mon, 2010-08-09, Stefan Küng wrote: > On 09.08.2010 16:44, Hyrum K. Wright wrote: > > On Mon, Aug 9, 2010 at 7:46 AM, Julian Foad > > wrote: > >> On Mon, 2010-08-09, Julian Foad wrote: > >>> On Mon, 2010-08-09, Bert Huijben wrote: > Stefan Küng wrote: > > This leads me to another req

Re: Looking to improve performance of svn annotate

2010-08-12 Thread C. Michael Pilato
In times past, I've wondered if the server couldn't just store line-delta information -- as a comparison between each FS DAG node and its immediate predecessor -- similar to the way that CVS does (and in addition to the stuff it already stores, of course). The line-delta info could be populated po

[RFC] Change svn_wc_add_repos_file4(copyfrom_url) to (_root, _relpath)

2010-08-12 Thread Julian Foad
svn_wc_add_repos_file4(..., copyfrom_url, ...) has a check that says, "If copyfrom_url isn't in the same repository as the WC target parent dir, then SVN_ERR_UNSUPPORTED_FEATURE." Semantically, yes, copying from another repos isn't yet supported. But the check for that should be at a higher an

RE: [RFC] Change svn_wc_add_repos_file4(copyfrom_url) to (_root, _relpath)

2010-08-12 Thread Bert Huijben
> -Original Message- > From: Julian Foad [mailto:julian.f...@wandisco.com] > Sent: donderdag 12 augustus 2010 15:43 > To: dev@subversion.apache.org > Subject: [RFC] Change svn_wc_add_repos_file4(copyfrom_url) to (_root, > _relpath) > > svn_wc_add_repos_file4(..., copyfrom_url, ...) has a

Re: [PATCH] Revert to using zlib-provided compress bound

2010-08-12 Thread Gavin Beau Baumanis
Hi Michael, Just thought I would follow up your last email and see if you're still working on this patch? There's no rush... I just like to bring patch submissions back to the top of the mailing list after about 7 days or so - just to remind everyone. Gavin. On 07/08/2010, at 8:11 AM, Michae

Re: [RFC] Change svn_wc_add_repos_file4(copyfrom_url) to (_root, _relpath)

2010-08-12 Thread Ivan Zhakov
On Thu, Aug 12, 2010 at 17:43, Julian Foad wrote: > svn_wc_add_repos_file4(..., copyfrom_url, ...) has a check that says, > >  "If copyfrom_url isn't in the same repository as the WC target parent > dir, then SVN_ERR_UNSUPPORTED_FEATURE." > > Semantically, yes, copying from another repos isn't yet

Log HTTP header for autoversioning

2010-08-12 Thread Loren Cahlander
I ran across this in the subversion archives: http://thread.gmane.org/gmane.comp.version-control.subversion.devel/55794 I am also needing this feature. Could someone supply me with a patch or help tell me how to get a HTTP header property while in version.c svn_error_t * dav_svn__attach_auto_

Re: Looking to improve performance of svn annotate

2010-08-12 Thread Julian Foad
On Thu, 2010-08-12, C. Michael Pilato wrote: > In times past, I've wondered if the server couldn't just store line-delta > information -- as a comparison between each FS DAG node and its immediate > predecessor -- similar to the way that CVS does (and in addition to the > stuff it already stores, o

Re: NODE_DATA (2nd iteration)

2010-08-12 Thread Stefan Sperling
On Thu, Aug 12, 2010 at 02:28:39PM +0200, Neels J Hofmeyr wrote: > I've had so many family and summer events in the past weeks... soon I'll > start forgetting my passwords! Just type "yes" at the "really save in plaintext?" prompt :)

Re: Looking to improve performance of svn annotate

2010-08-12 Thread Greg Hudson
On Thu, 2010-08-12 at 10:57 -0400, Julian Foad wrote: > I'm wary of embedding any client functionality in the server, but I > guess it's worth considering if it would be that useful. If so, let's > take great care to ensure it's only lightly coupled to the core server > logic. Again, it's possibl

What C structure contains the HTTP Headers

2010-08-12 Thread Loren Cahlander
Hello there, I am trying to make a change to mod_dav_svn on my server and need to get a value out of the HTTP headers. I have not developed in C in quite a while and would appreciate any help by pointing me in the right direction. I can take it from there, but I do need some help. Thank you,

Re: What C structure contains the HTTP Headers

2010-08-12 Thread C. Michael Pilato
On 08/12/2010 12:04 PM, Loren Cahlander wrote: > Hello there, > > I am trying to make a change to mod_dav_svn on my server and need to get > a value out of the HTTP headers. I have not developed in C in quite a > while and would appreciate any help by pointing me in the right > direction. I can t

Re: What C structure contains the HTTP Headers

2010-08-12 Thread Hyrum K. Wright
On Thu, Aug 12, 2010 at 11:07 AM, C. Michael Pilato wrote: > On 08/12/2010 12:04 PM, Loren Cahlander wrote: >> Hello there, >> >> I am trying to make a change to mod_dav_svn on my server and need to get >> a value out of the HTTP headers. I have not developed in C in quite a >> while and would app

Re: [RFC] Change svn_wc_add_repos_file4(copyfrom_url) to (_root, _relpath)

2010-08-12 Thread Hyrum K. Wright
On Thu, Aug 12, 2010 at 8:43 AM, Julian Foad wrote: > svn_wc_add_repos_file4(..., copyfrom_url, ...) has a check that says, > >  "If copyfrom_url isn't in the same repository as the WC target parent > dir, then SVN_ERR_UNSUPPORTED_FEATURE." > > Semantically, yes, copying from another repos isn't y

Re: What C structure contains the HTTP Headers

2010-08-12 Thread C. Michael Pilato
On 08/12/2010 12:12 PM, Hyrum K. Wright wrote: > On Thu, Aug 12, 2010 at 11:07 AM, C. Michael Pilato > wrote: >> On 08/12/2010 12:04 PM, Loren Cahlander wrote: >>> Hello there, >>> >>> I am trying to make a change to mod_dav_svn on my server and need to get >>> a value out of the HTTP headers. I

Re: What C structure contains the HTTP Headers

2010-08-12 Thread Loren Cahlander
On Aug 12, 2010, at 11:12 AM, Hyrum K. Wright wrote: > On Thu, Aug 12, 2010 at 11:07 AM, C. Michael Pilato > wrote: >> On 08/12/2010 12:04 PM, Loren Cahlander wrote: >>> Hello there, >>> >>> I am trying to make a change to mod_dav_svn on my server and need to get >>> a value out of the HTTP hea

[PATCH] Tweak description of BASE_NODE table

2010-08-12 Thread Julian Foad
Can someone check this before I commit? [[[ Index: subversion/libsvn_wc/wc_db.h === --- subversion/libsvn_wc/wc_db.h(revision 984862) +++ subversion/libsvn_wc/wc_db.h(working copy) @@ -438,14 +438,19 @@ svn_wc__db_get_

Re: What C structure contains the HTTP Headers

2010-08-12 Thread C. Michael Pilato
On 08/12/2010 12:49 PM, Loren Cahlander wrote: > I want to change the following function in subversion/mod_dav_svn/version.c > to get those two headers values and set the values for SVN_PROP_REVISION_LOG > and SVN_PROP_REVISION_AUTHOR. Is there a simple way to get the request_rec > structure fr

Re: [PATCH] Revert to using zlib-provided compress bound

2010-08-12 Thread Daniel Shahaf
Welcome back Gavin :-) Gavin Beau Baumanis wrote on Fri, Aug 13, 2010 at 00:12:53 +1000: > Hi Michael, > > Just thought I would follow up your last email and see if you're still > working on this patch? > There's no rush... I just like to bring patch submissions back to the top of > the mailing

Re: RFC: WCNG and Issue #2915: Merge tracking and missing subtrees due to non-svn removal

2010-08-12 Thread Daniel Shahaf
Paul Burba wrote on Fri, Aug 06, 2010 at 10:30:50 -0400: > As described in issue #2915, in 1.6 when a merge target is a missing > subtree due to its removal by non-svn means, we try to record > mergeinfo such that the missing subtree doesn't have (or inherit) > mergeinfo describing the merge: > >

Re: [PATCH] Revert to using zlib-provided compress bound

2010-08-12 Thread Peter Samuelson
[Gavin Beau Baumanis] > Just thought I would follow up your last email and see if you're > still working on this patch? r984911. Thanks for the reminder, Peter

Re: [PATCH] neon capabilities exchange

2010-08-12 Thread Daniel Shahaf
Daniel Shahaf wrote on Fri, Jul 30, 2010 at 15:30:19 +0300: > So, here's an updated patch. I'll commit it in a few days if I don't hear > objections. 0:% svn ci --cl part Sendingsubversion/libsvn_ra_neon/options.c Sendingsubversion/libsvn_ra_serf/options.c Sendingsubversio

Re: svn commit: r983764 - /subversion/branches/performance/subversion/libsvn_subr/dirent_uri.c

2010-08-12 Thread Stefan Fuhrmann
Daniel Shahaf wrote: +1 to merge this to trunk. (Please specify "Approved by: danielsh" when doing so, since your commit access does not currently include /trunk. Thanks.) Done in r984926. -- Stefan^2. stef...@apache.org wrote on Mon, Aug 09, 2010 at 18:27:49 -: Author: stefan2 Da

Re: svn commit: r983766 - /subversion/branches/performance/subversion/libsvn_client/export.c

2010-08-12 Thread Stefan Fuhrmann
Daniel Shahaf wrote: stef...@apache.org wrote on Mon, Aug 09, 2010 at 18:33:53 -: Author: stefan2 Date: Mon Aug 9 18:33:53 2010 New Revision: 983766 URL: http://svn.apache.org/viewvc?rev=983766&view=rev Log: Fix the root cause of an assertion triggered by exporting KDE /trunk: File name

Re: svn commit: r983766 - /subversion/branches/performance/subversion/libsvn_client/export.c

2010-08-12 Thread Daniel Shahaf
Stefan Fuhrmann wrote on Thu, Aug 12, 2010 at 21:51:12 +0200: > Merged in r984928. Thanks. >> (actually, this may be a good change to backport to 1.6.x too) >> > It seems to work with the 1.6.x client (at least with the builds I used). > So, this bug might have gotten triggered by some canonic

Re: svn commit: r984926 - in /subversion/trunk: ./ subversion/libsvn_subr/dirent_uri.c

2010-08-12 Thread Daniel Shahaf
stef...@apache.org wrote on Thu, Aug 12, 2010 at 19:29:24 -: > Author: stefan2 > Date: Thu Aug 12 19:29:23 2010 > New Revision: 984926 > > URL: http://svn.apache.org/viewvc?rev=984926&view=rev > Log: > Merge r983764 from branches/performance. Approved by: danielsh > http://svn.haxx.se/dev/arch

Re: svn commit: r983766 - /subversion/branches/performance/subversion/libsvn_client/export.c

2010-08-12 Thread Stefan Fuhrmann
Hyrum K. Wright wrote: On Tue, Aug 10, 2010 at 5:56 AM, Stefan Sperling wrote: On Mon, Aug 09, 2010 at 09:45:18PM +0300, Daniel Shahaf wrote: Also, this isn't really related to performance; it belongs on /trunk. Next time, you could send this with a [PATCH] marker in the subject line,

Re: svn commit: r984926 - in /subversion/trunk: ./ subversion/libsvn_subr/dirent_uri.c

2010-08-12 Thread Lieven Govaerts
On Thu, Aug 12, 2010 at 9:59 PM, Daniel Shahaf wrote: > stef...@apache.org wrote on Thu, Aug 12, 2010 at 19:29:24 -: >> Author: stefan2 >> Date: Thu Aug 12 19:29:23 2010 >> New Revision: 984926 >> >> URL: http://svn.apache.org/viewvc?rev=984926&view=rev >> Log: >> Merge r983764 from branches/p

Returning revprops through commit info

2010-08-12 Thread Hyrum K. Wright
Recently, we've updated the various client APIs which do commits to return commit info back through a callback[1], since they may be extended to perform multiple commits (see issue #1199 for instance). In doing so, I've had the opportunity to take a look at the svn_commit_info_t struct. It's prett

Re: [PATCH] Tweak description of BASE_NODE table

2010-08-12 Thread Greg Stein
Looks good! On Aug 12, 2010 1:12 PM, "Julian Foad" wrote: > Can someone check this before I commit? > > [[[ > Index: subversion/libsvn_wc/wc_db.h > === > --- subversion/libsvn_wc/wc_db.h (revision 984862) > +++ subversion/libsvn_wc/w

Re: svn commit: r984926 - in /subversion/trunk: ./ subversion/libsvn_subr/dirent_uri.c

2010-08-12 Thread Lieven Govaerts
On Thu, Aug 12, 2010 at 10:08 PM, Lieven Govaerts wrote: > On Thu, Aug 12, 2010 at 9:59 PM, Daniel Shahaf > wrote: >> stef...@apache.org wrote on Thu, Aug 12, 2010 at 19:29:24 -: >>> Author: stefan2 >>> Date: Thu Aug 12 19:29:23 2010 >>> New Revision: 984926 >>> >>> URL: http://svn.apache.or

Re: What C structure contains the HTTP Headers

2010-08-12 Thread Loren Cahlander
After discussing things with Mike, I an going to be making the changes through the HTTP PROPPATCH method. Thank you Mike and Hyrum for you help and patience. Loren On Aug 12, 2010, at 12:33 PM, C. Michael Pilato wrote: > On 08/12/2010 12:49 PM, Loren Cahlander wrote: >> I want to change the

Re: Note to self (with all of dev@ peering over my shoulder...)

2010-08-12 Thread C. Michael Pilato
On 07/15/2010 04:04 PM, Daniel Shahaf wrote: > C. Michael Pilato wrote on Thu, Jul 15, 2010 at 15:43:10 -0400: >> [ Mailing this to the list so I don't forget about it -- I haven't >> time to dive in right now. ] >> >> I ran into a problem today with the SWIG Python bindings when doing some >> Vi

Re: svn commit: r983766 - /subversion/branches/performance/subversion/libsvn_client/export.c

2010-08-12 Thread Daniel Shahaf
Stefan Fuhrmann wrote on Thu, Aug 12, 2010 at 22:08:03 +0200: > Hyrum K. Wright wrote: >> On Tue, Aug 10, 2010 at 5:56 AM, Stefan Sperling wrote: >> >>> On Mon, Aug 09, 2010 at 09:45:18PM +0300, Daniel Shahaf wrote: >>> Also, this isn't really related to performance; it belongs on /tr

Re: svn commit: r984926 - in /subversion/trunk: ./ subversion/libsvn_subr/dirent_uri.c

2010-08-12 Thread Daniel Shahaf
Lieven Govaerts wrote on Thu, Aug 12, 2010 at 22:19:48 +0200: > Slightly off topic remark, but it would be nice if we could run > stefan's branch on the buildbots from time to time, would be > interesting to see the impact on the test run times. I'm pretty sure gmcdonald can arrange that, if we (c

Re: [patch] fsfs revprop packing paranoia

2010-08-12 Thread Daniel Shahaf
I figure this patch can't do any harm (except cost another file-read when a write-lock or txn-current-lock is being acquired), and it might as well help, so I committed it in r984990. Daniel Shahaf wrote on Tue, Aug 03, 2010 at 18:59:33 +0300: > Daniel Shahaf wrote on Mon, Aug 02, 2010 at 23:44:10

Re: Note to self (with all of dev@ peering over my shoulder...)

2010-08-12 Thread Роман Донченко
C. Michael Pilato писал в своём письме Fri, 13 Aug 2010 00:41:36 +0400: Simplest recipe I can find: $ python -c 'from svn import client, core; ctx = client.svn_client_ctx_t(); ctx.config = core.svn_config_get_config(None)' Traceback (most recent call last): File "", line 1, in File "/us

Re: svn commit: r966822 - in /subversion/trunk/subversion: libsvn_client/repos_diff.c tests/cmdline/merge_tests.py

2010-08-12 Thread Paul Burba
On Tue, Aug 3, 2010 at 12:40 PM, Paul Burba wrote: > Ok, I am waving the white flag as far as implementing a fix for issue > #3657 in the RA layer.  I simply don't see how it can be done outside > of this:  Put the DAV update report handler into send-all=TRUE > mode when creating a > svn_ra_repor

Re: svn commit: r983766 - /subversion/branches/performance/subversion/libsvn_client/export.c

2010-08-12 Thread Stefan Fuhrmann
Daniel Shahaf wrote: Stefan Fuhrmann wrote on Thu, Aug 12, 2010 at 22:08:03 +0200: Hyrum K. Wright wrote: On Tue, Aug 10, 2010 at 5:56 AM, Stefan Sperling wrote: On Mon, Aug 09, 2010 at 09:45:18PM +0300, Daniel Shahaf wrote: Also, this isn't really related to