Re: svn commit: r1058308 - /subversion/site/publish/docs/release-notes/1.7.html

2011-01-12 Thread Daniel Shahaf
s...@apache.org wrote on Wed, Jan 12, 2011 at 20:36:34 -: > Author: stsp > Date: Wed Jan 12 20:36:33 2011 > New Revision: 1058308 > > URL: http://svn.apache.org/viewvc?rev=1058308&view=rev > Log: > * publish/docs/release-notes/1.7.html > (atomic-revprops): Mention that this is also a client-

Re: issue 3486 should be reopened?

2011-01-12 Thread Gavin Beau Baumanis
Thanks Stefan, I will remove it from my watch list. On 13/01/2011, at 6:07 AM, Stefan Sperling wrote: > On Mon, Nov 29, 2010 at 09:14:09AM +1100, Gavin Beau Baumanis wrote: >> On 27/11/2010, at 12:46 AM, Stefan Sperling wrote: >>> On Fri, Nov 26, 2010 at 02:45:07PM +0100, Stefan Sperling wrote:

Re: [PATCH] Add include of "svn_dirent_uri.h" to mod_authz_svn.c

2011-01-12 Thread Stefan Sperling
On Wed, Jan 12, 2011 at 07:50:11PM +, Piper, Nick wrote: > This is a follow up to a previously submitted patch which was applied, see > http://mail-archives.apache.org/mod_mbox/subversion-dev/201011.mbox/%3c87fwvigwgr@stat.home.lan%3e > and r1030536 r1030540. > > It seems that patch was

[PATCH] Add include of "svn_dirent_uri.h" to mod_authz_svn.c

2011-01-12 Thread Piper, Nick
This is a follow up to a previously submitted patch which was applied, see http://mail-archives.apache.org/mod_mbox/subversion-dev/201011.mbox/%3c87fwvigwgr@stat.home.lan%3e and r1030536 r1030540. It seems that patch was missing: #include "svn_dirent_uri.h" Without this, we get a warning:

Re: issue 3486 should be reopened?

2011-01-12 Thread Stefan Sperling
On Mon, Nov 29, 2010 at 09:14:09AM +1100, Gavin Beau Baumanis wrote: > On 27/11/2010, at 12:46 AM, Stefan Sperling wrote: > > On Fri, Nov 26, 2010 at 02:45:07PM +0100, Stefan Sperling wrote: > >> On Fri, Nov 26, 2010 at 02:40:07PM +0100, Peter Parker wrote: > >>> On 13.11.2010 16:07, Daniel Shahaf

Re: headers in the body Re: [l10n] Translation status report for trunk r1057984

2011-01-12 Thread Hyrum K Wright
On Wed, Jan 12, 2011 at 12:01 PM, Branko Čibej wrote: > On 12.01.2011 18:43, Branko Čibej wrote: >> On 12.01.2011 16:47, Hyrum K Wright wrote: >>> On Wed, Jan 12, 2011 at 8:14 AM, Branko Čibej wrote: On 12.01.2011 15:01, Daniel Shahaf wrote: > SVN DEV wrote on Wed, Jan 12, 2011 at 04:21:

Re: headers in the body Re: [l10n] Translation status report for trunk r1057984

2011-01-12 Thread Branko Čibej
On 12.01.2011 18:43, Branko Čibej wrote: > On 12.01.2011 16:47, Hyrum K Wright wrote: >> On Wed, Jan 12, 2011 at 8:14 AM, Branko Čibej wrote: >>> On 12.01.2011 15:01, Daniel Shahaf wrote: SVN DEV wrote on Wed, Jan 12, 2011 at 04:21:52 +: > X-Mailer: l10n-report.py r1055713 > Reply

Re: headers in the body Re: [l10n] Translation status report for trunk r1057984

2011-01-12 Thread Branko Čibej
On 12.01.2011 16:47, Hyrum K Wright wrote: > On Wed, Jan 12, 2011 at 8:14 AM, Branko Čibej wrote: >> On 12.01.2011 15:01, Daniel Shahaf wrote: >>> SVN DEV wrote on Wed, Jan 12, 2011 at 04:21:52 +: X-Mailer: l10n-report.py r1055713 Reply-To: dev@subversion.apache.org Mail-Followu

Re: [PATCH] Force scratch pool usage in the calls to svn_uri_condense_targets/svn_dirent_condense_targets

2011-01-12 Thread Stefan Sperling
On Mon, Dec 27, 2010 at 04:05:02PM +0530, vijay wrote: > Hi, > > We can use SCRATCH_POOL in the calls to > svn_uri_condense_targets/svn_dirent_condense_targets wherever it is > applicable. It will save few bits of memory. > > The attached patch forces SCRATCH_POOL usage in few places. > > Thank

Re: headers in the body Re: [l10n] Translation status report for trunk r1057984

2011-01-12 Thread Hyrum K Wright
On Wed, Jan 12, 2011 at 8:14 AM, Branko Čibej wrote: > On 12.01.2011 15:01, Daniel Shahaf wrote: >> SVN DEV wrote on Wed, Jan 12, 2011 at 04:21:52 +: >>> X-Mailer: l10n-report.py r1055713 >>> Reply-To: dev@subversion.apache.org >>> Mail-Followup-To: dev@subversion.apache.org >>> Auto-Submitted

Re: svn commit: r1057088 - in /subversion/trunk/subversion: libsvn_fs_base/bdb/env.c libsvn_ra_svn/cyrus_auth.c libsvn_subr/io.c libsvn_subr/sqlite.c

2011-01-12 Thread C. Michael Pilato
On 01/12/2011 09:54 AM, Branko Čibej wrote: > 's true that static (and in fact all global) storage is default-inited > to 0 ... but there's no harm in putting the initializer there. IMO it's > better to have it, if only for the sake of clarity. You took the words right outta my ... Outbox. I take

Re: svn commit: r1057088 - in /subversion/trunk/subversion: libsvn_fs_base/bdb/env.c libsvn_ra_svn/cyrus_auth.c libsvn_subr/io.c libsvn_subr/sqlite.c

2011-01-12 Thread Branko Čibej
On 12.01.2011 15:12, Daniel Shahaf wrote: > Philip Martin wrote on Wed, Jan 12, 2011 at 11:01:12 +: >> danie...@apache.org writes: >> >>> Author: danielsh >>> Date: Mon Jan 10 06:03:30 2011 >>> New Revision: 1057088 >>> >>> URL: http://svn.apache.org/viewvc?rev=1057088&view=rev >>> Log: >>> Ini

Re: Canonicalizing relative URLs

2011-01-12 Thread C. Michael Pilato
On 01/11/2011 11:18 PM, Blair Zajac wrote: > Mike, don't you want to canonicalize the resolved URI, not what's in the > external definition? I think I called svn_uri_canonicalize() just so I > could use the rest of the svn_path_*() API without core dumping. Blair, I'm not composing the code snipp

Re: SQLite and callbacks

2011-01-12 Thread C. Michael Pilato
Nothing to add to the immediate discussion at this time. Just wanted to note that we have essentially the same problems to deal with in the BDB backend. The solution we've taken there is "don't drive public API callbacks from inside BDB transactions". It's not always the best for performance, bu

Re: svn commit: r1057088 - in /subversion/trunk/subversion: libsvn_fs_base/bdb/env.c libsvn_ra_svn/cyrus_auth.c libsvn_subr/io.c libsvn_subr/sqlite.c

2011-01-12 Thread Daniel Shahaf
Philip Martin wrote on Wed, Jan 12, 2011 at 11:01:12 +: > danie...@apache.org writes: > > > Author: danielsh > > Date: Mon Jan 10 06:03:30 2011 > > New Revision: 1057088 > > > > URL: http://svn.apache.org/viewvc?rev=1057088&view=rev > > Log: > > Initialize svn_atomic_t's to zero, per svn_atomi

Re: headers in the body Re: [l10n] Translation status report for trunk r1057984

2011-01-12 Thread Branko Čibej
On 12.01.2011 15:01, Daniel Shahaf wrote: > SVN DEV wrote on Wed, Jan 12, 2011 at 04:21:52 +: >> X-Mailer: l10n-report.py r1055713 >> Reply-To: dev@subversion.apache.org >> Mail-Followup-To: dev@subversion.apache.org >> Auto-Submitted: auto-generated >> Translation status report for tr...@r1057

headers in the body Re: [l10n] Translation status report for trunk r1057984

2011-01-12 Thread Daniel Shahaf
SVN DEV wrote on Wed, Jan 12, 2011 at 04:21:52 +: > X-Mailer: l10n-report.py r1055713 > Reply-To: dev@subversion.apache.org > Mail-Followup-To: dev@subversion.apache.org > Auto-Submitted: auto-generated > Translation status report for tr...@r1057984 Looks like something inserts a spurious blan

Re: [l10n] Translation status report for trunk r1055292

2011-01-12 Thread Hyrum K Wright
On Wed, Jan 12, 2011 at 7:50 AM, Ramkumar Ramachandra wrote: > Hi, > > Hyrum K Wright writes: >> I'm working on getting a web-based translation system installed on ASF >> hardware for use by Subversion, and possibly other projects, and will >> report progress here as it happens.  I am in the unfor

Re: [l10n] Translation status report for trunk r1055292

2011-01-12 Thread Ramkumar Ramachandra
Hi, Hyrum K Wright writes: > I'm working on getting a web-based translation system installed on ASF > hardware for use by Subversion, and possibly other projects, and will > report progress here as it happens. I am in the unfortunate position > of being a monoglot, and hence unable to actually do

Re: SQLite and callbacks

2011-01-12 Thread Stefan Sperling
On Tue, Nov 30, 2010 at 12:33:38PM +, Philip Martin wrote: > "Hyrum K. Wright" writes: > > > Stefan's patch to make a recursive proplist much more performant > > highlights the great benefit that our sqlite-backed storage can have. > > However, he reverted it due to concerns about the potenti

Re: svn commit: r1057088 - in /subversion/trunk/subversion: libsvn_fs_base/bdb/env.c libsvn_ra_svn/cyrus_auth.c libsvn_subr/io.c libsvn_subr/sqlite.c

2011-01-12 Thread Philip Martin
danie...@apache.org writes: > Author: danielsh > Date: Mon Jan 10 06:03:30 2011 > New Revision: 1057088 > > URL: http://svn.apache.org/viewvc?rev=1057088&view=rev > Log: > Initialize svn_atomic_t's to zero, per svn_atomic__init_once(). That is redundant, strict speaking, because static variables