Re: [WIP PATCH] Make svn_diff_diff skip identical prefix and suffix to make diff and blame faster

2010-10-03 Thread Johan Corveleyn
On Sun, Oct 3, 2010 at 1:46 AM, Johan Corveleyn wrote: > Hi, > > Here is a second iteration of the patch. It now passes make check. > > Differences from the previous version are: > - Support for \r eol-style (\n and \r\n was already ok). > - The number of prefix_lines is now passed to svn_diff__lc

Re: svn commit: r1002898 - in /subversion/trunk: ./ subversion/include/svn_string.h subversion/libsvn_subr/svn_string.c

2010-10-03 Thread Stefan Fuhrmann
On 01.10.2010 21:07, Greg Stein wrote: On Fri, Oct 1, 2010 at 04:36, Stefan Fuhrmann wrote: ... I mean calling convention. Where is the Subversion ABI defined? subversion/include/ It follows guidelines from: http://apr.apache.org/versioning.html With some additional clarifications/note

Re: Some tips on profiling

2010-10-03 Thread Stefan Fuhrmann
On 01.10.2010 15:56, Ramkumar Ramachandra wrote: Hi Stefan, Stefan Fuhrmann writes: You obviously don't have APR threads enabled. Thanks for finding this. Fixed in r1003430. I enabled it, but there's still some issue: subversion/svnadmin/main.c:1892: undefined reference to `svn_fs_get_cache

Re: svn commit: r1004003 - /subversion/site/publish/index.html

2010-10-03 Thread Mark Phippard
On Sun, Oct 3, 2010 at 5:58 PM, Mark Phippard wrote: > If we provided an RSS feed for our news items, which would be a good > idea but yet more work, than we could use something like this to > easily pull the most recent news items into the homepage: > > http://feed2js.org/ > > Maybe we could jus

Re: svn commit: r1002151 - in /subversion/trunk/subversion: libsvn_auth_kwallet/kwallet.cpp svn/main.c svnsync/main.c

2010-10-03 Thread Blair Zajac
On 10/3/10 2:45 PM, Arfrever Frehtes Taifersar Arahesis wrote: 2010-09-29 14:29:26 Stefan Sperling napisał(a): On Tue, Sep 28, 2010 at 11:21:26AM -0700, Blair Zajac wrote: On 9/28/2010 6:07 AM, s...@apache.org wrote: Author: stsp Date: Tue Sep 28 13:07:23 2010 New Revision: 1002151 URL: http:

Re: svn commit: r1004003 - /subversion/site/publish/index.html

2010-10-03 Thread Mark Phippard
On Sun, Oct 3, 2010 at 5:14 PM, Stefan Sperling wrote: > On Sun, Oct 03, 2010 at 05:03:25PM -0400, Greg Stein wrote: >> On Sun, Oct 3, 2010 at 13:12,   wrote: >> > Author: stsp >> > Date: Sun Oct  3 17:12:19 2010 >> > New Revision: 1004003 >> > >> > URL: http://svn.apache.org/viewvc?rev=1004003&vi

Re: svn commit: r1002151 - in /subversion/trunk/subversion: libsvn_auth_kwallet/kwallet.cpp svn/main.c svnsync/main.c

2010-10-03 Thread Arfrever Frehtes Taifersar Arahesis
2010-09-29 14:29:26 Stefan Sperling napisał(a): > On Tue, Sep 28, 2010 at 11:21:26AM -0700, Blair Zajac wrote: > > On 9/28/2010 6:07 AM, s...@apache.org wrote: > > >Author: stsp > > >Date: Tue Sep 28 13:07:23 2010 > > >New Revision: 1002151 > > > > > >URL: http://svn.apache.org/viewvc?rev=1002151&v

Re: svn commit: r1004003 - /subversion/site/publish/index.html

2010-10-03 Thread Stefan Sperling
On Sun, Oct 03, 2010 at 05:03:25PM -0400, Greg Stein wrote: > On Sun, Oct 3, 2010 at 13:12, wrote: > > Author: stsp > > Date: Sun Oct  3 17:12:19 2010 > > New Revision: 1004003 > > > > URL: http://svn.apache.org/viewvc?rev=1004003&view=rev > > Log: > > * site/publish/index.html: Fix 1.6.13 releas

Re: svn commit: r1004003 - /subversion/site/publish/index.html

2010-10-03 Thread Greg Stein
On Sun, Oct 3, 2010 at 13:12, wrote: > Author: stsp > Date: Sun Oct  3 17:12:19 2010 > New Revision: 1004003 > > URL: http://svn.apache.org/viewvc?rev=1004003&view=rev > Log: > * site/publish/index.html: Fix 1.6.13 release date, here, too. >   (Why are news items maintained in two separate places

[PATCH] extend svn_subst_translate_string() to record whether re-encoding and/or line ending translation were performed (v. 2)

2010-10-03 Thread Daniel Trebbien
On Fri, Oct 1, 2010 at 3:57 AM, Julian Foad wrote: >> Adds a public API function, svn_subst_translate_string2(), an extension of >> svn_subst_translate_string(), that has two, additional output parameters for >> determining whether re-encoding and/or line ending translation were >> performed. > >

Re: [Merge request] Merge r985477 from performance branch

2010-10-03 Thread Ramkumar Ramachandra
Hi Stefan, Stefan Sperling writes: > > Can I get an explicit +1 to commit this? I just want to get as many of > > Stefan's changes merged in quickly so that there's enough time before > > the 1.7 release to test them. > > > > Technical detail: How do I merge? > > cd svn-trunk-working-copy > svn

Re: [Merge request] Merge r985477 from performance branch

2010-10-03 Thread Stefan Sperling
On Sun, Oct 03, 2010 at 07:02:01PM +0530, Ramkumar Ramachandra wrote: > Hi Julian, > > Julian Foad writes: > > Looks good to me. > > > > I wondered if it is safe in a long-running Subversion process, like > > TortoiseSvn or a Linux equivalent. > > > > It seems to me that it won't really matter m

Re: [Merge request] Merge r985477 from performance branch

2010-10-03 Thread Ramkumar Ramachandra
Hi Julian, Julian Foad writes: > Looks good to me. > > I wondered if it is safe in a long-running Subversion process, like > TortoiseSvn or a Linux equivalent. > > It seems to me that it won't really matter much in practice. If someone > changes their umask and finds that Subversion carries on

Re: svn commit: r1003924 - /subversion/trunk/subversion/svnrdump/dump_editor.c

2010-10-03 Thread Ramkumar Ramachandra
Hi Bert, Bert Huijben writes: > Do you really need a 'named' file here? > > The named file algorithm of svn_io_open_uniquely_named() is very slow > compared to the 'just get me a temporary file' implemented by > svn_io_open_unique_file3(). Especially when you need more than a few > tempfiles w

Re: svn commit: r1003227 - /subversion/site/publish/docs/community-guide/releasing.part.html

2010-10-03 Thread Stefan Sperling
On Fri, Oct 01, 2010 at 01:41:42AM +0200, Daniel Shahaf wrote: > s...@apache.org wrote on Thu, Sep 30, 2010 at 20:05:54 -: > > +++ subversion/site/publish/docs/community-guide/releasing.part.html Thu > > Sep 30 20:05:53 2010 > > @@ -1096,7 +1096,7 @@ manager.) > > - > > + > > How not to make

RE: svn commit: r1003924 - /subversion/trunk/subversion/svnrdump/dump_editor.c

2010-10-03 Thread Bert Huijben
> -Original Message- > From: artag...@apache.org [mailto:artag...@apache.org] > Sent: zondag 3 oktober 2010 9:05 > To: comm...@subversion.apache.org > Subject: svn commit: r1003924 - > /subversion/trunk/subversion/svnrdump/dump_editor.c > > Author: artagnon > Date: Sun Oct 3 07:05:05 20

Re: [PATCH] -cM-N command line syntax

2010-10-03 Thread Gavin Beau Baumanis
Strictly from the view of the non-developer and user of SVN; I have only ever used the ":" for specifying a range. But I do see the sanity in having the input match the output too. If the ability to support both is indeed trivial - and if both cases are "expected" by users - then that would certai