Authz on Collection of Repositories (was: Expansion of authz policy name leak)

2012-10-18 Thread Thomas Åkesson
There was a discussion in April 2010 regarding the fix for issue 2753. http://svn.haxx.se/dev/archive-2010-04/0277.shtml Unfortunately the discussion died due to lack of other opinions. I think Mike had some very important input here and I believe that this concluding statement is incorrect:

RE: svn commit: r1399462 - in /subversion/trunk: subversion/libsvn_client/ subversion/libsvn_subr/ subversion/libsvn_wc/ subversion/mod_dav_svn/ subversion/svn/ subversion/svnserve/ subversion/tests/l

2012-10-18 Thread Bert Huijben
-Original Message- From: danie...@apache.org [mailto:danie...@apache.org] Sent: donderdag 18 oktober 2012 00:22 To: comm...@subversion.apache.org Subject: svn commit: r1399462 - in /subversion/trunk: subversion/libsvn_client/ subversion/libsvn_subr/ subversion/libsvn_wc/

Re: svn commit: r1399582 - /subversion/trunk/subversion/libsvn_ra_serf/util_error.c

2012-10-18 Thread Ivan Zhakov
On Thu, Oct 18, 2012 at 3:31 PM, s...@apache.org wrote: Author: stsp Date: Thu Oct 18 11:31:20 2012 New Revision: 1399582 URL: http://svn.apache.org/viewvc?rev=1399582view=rev Log: * subversion/libsvn_ra_serf/util_error.c: Include ra_serf.h which contains prototypes for functions

[PATCH] Implement svnadmin verify --force

2012-10-18 Thread Prabhu Gnana Sundar
Hi all, Currently svnadmin verify would stop verification process once an error/corruption is found in the repo. It does not go till the HEAD of the repo to see if there are further corruptions/errors. It would be helpful if --force switch would do this. Attaching a patch and the log

RE: [PATCH] Implement svnadmin verify --force

2012-10-18 Thread Bert Huijben
-Original Message- From: Prabhu Gnana Sundar [mailto:prabh...@collab.net] Sent: donderdag 18 oktober 2012 17:05 To: dev@subversion.apache.org Subject: [PATCH] Implement svnadmin verify --force Hi all, Currently svnadmin verify would stop verification process once an

Re: [PATCH] Implement svnadmin verify --force

2012-10-18 Thread Stefan Sperling
On Thu, Oct 18, 2012 at 08:35:03PM +0530, Prabhu Gnana Sundar wrote: Hi all, Currently svnadmin verify would stop verification process once an error/corruption is found in the repo. It does not go till the HEAD of the repo to see if there are further corruptions/errors. It would be

Re: [PATCH] Implement svnadmin verify --force

2012-10-18 Thread Julian Foad
Stefan Sperling wrote: Prabhu Gnana Sundar wrote: Currently svnadmin verify would stop verification process once an error/corruption is found in the repo. It does not go till the HEAD of the repo to see if there are further corruptions/errors. It would be helpful if --force switch

Re: svn commit: r1398389 - in /subversion/trunk/subversion: include/ libsvn_client/ libsvn_subr/ libsvn_wc/ svn/ tests/libsvn_wc/

2012-10-18 Thread Julian Foad
Would it be useful to copy this explanation somewhere more discoverable -- somewhere in the source tree, or linked from the release notes page? - Julian Philip Martin wrote: Johan Corveleyn jcor...@gmail.com writes: With this new exclusive mode it's probably quite clear (no

Re: renaming symmetric merge

2012-10-18 Thread Julian Foad
I (Julian Foad) wrote: Stefan Sperling wrote:   Hey Julian, are you still planning to rename these APIs from symmetric   to automatic merge? I think automatic is a much better term to use   because it is somewhat easier to understand from the user's perspective.   Especially for new users

Re: svn commit: r1399582 - /subversion/trunk/subversion/libsvn_ra_serf/util_error.c

2012-10-18 Thread Julian Foad
Ivan Zhakov wrote: Author: stsp Date: Thu Oct 18 11:31:20 2012 New Revision: 1399582 URL: http://svn.apache.org/viewvc?rev=1399582view=rev Log: * subversion/libsvn_ra_serf/util_error.c: Include ra_serf.h which contains     prototypes for functions implemented in this file. [...]

Re: Regular expressions in Subversion

2012-10-18 Thread Branko Čibej
On 17.10.2012 13:53, Stefan Küng wrote: On 17.10.2012 18:20, Hyrum K Wright wrote: Anyway, I was just wondering what folks feelings were about this possibility, and whether it's finally time to start thinking about proper regex support within Subversion. I'm wondering: if C++ is an option,

Re: svn commit: r1398983 - in /subversion/trunk: ./ subversion/include/ subversion/libsvn_ra_serf/ subversion/mod_dav_svn/

2012-10-18 Thread Julian Foad
Author: cmpilato URL: http://svn.apache.org/viewvc?rev=1398983view=rev Log: Reintegrate into trunk the 'http-dynamic-prop-namespaces' branch (using a simple 'svn merge BRANCH-URL' with a trunk@HEAD client!). Modified: subversion/trunk/subversion/mod_dav_svn/deadprops.c +static

Re: API simplification opportunity Re: svn commit: r1348666 - in /subversion/trunk/subversion: include/ libsvn_delta/ libsvn_fs_fs/ libsvn_ra/ tests/libsvn_delta/

2012-10-18 Thread Daniel Shahaf
Stefan Fuhrmann wrote on Thu, Oct 18, 2012 at 02:56:15 +0200: On Tue, Oct 16, 2012 at 12:55 AM, Daniel Shahaf d...@daniel.shahaf.namewrote: stef...@apache.org wrote on Sun, Jun 10, 2012 at 21:34:22 -: Author: stefan2 Date: Sun Jun 10 21:34:21 2012 New Revision: 1348666

Extensible property namespace work deemed insufficient; considering reversion.

2012-10-18 Thread C. Michael Pilato
Just added this comment to issue #1971[1] (aka certain property names cause non-wf XML responses, aka the thing that spun cmpilato off on this extensible property namespace business): {{{ Oh, man. Maybe this whole venture was ill-conceived. I just remembered the other restrictions on XML Names,

Re: svn commit: r1331883 - /subversion/trunk/subversion/svnadmin/main.c

2012-10-18 Thread Daniel Shahaf
Stefan Fuhrmann wrote on Thu, Oct 18, 2012 at 02:26:13 +0200: On Thu, Oct 18, 2012 at 1:57 AM, Daniel Shahaf d...@daniel.shahaf.namewrote: Branko Čibej wrote on Mon, Oct 15, 2012 at 23:51:43 -0400: On 15.10.2012 17:14, Stefan Fuhrmann wrote: However, if you have a long-running process

Re: svn commit: r1331883 - /subversion/trunk/subversion/svnadmin/main.c

2012-10-18 Thread Daniel Shahaf
Stefan Fuhrmann wrote on Thu, Oct 18, 2012 at 02:26:13 +0200: No need to go off on a hyperbole. Raise your concerns in a factual way and hold off meta- discussion until after those facts got confirmed. Still appreciate the review, though. And, yes, the coded logic may contain holes and I'm

Re: [PATCH] Implement svnadmin verify --force

2012-10-18 Thread Daniel Shahaf
Prabhu Gnana Sundar wrote on Thu, Oct 18, 2012 at 20:35:03 +0530: Hi all, Currently svnadmin verify would stop verification process once an error/corruption is found in the repo. It does not go till the HEAD of the repo to see if there are further corruptions/errors. Why is r=0

Re: svn commit: r1399462 - in /subversion/trunk: subversion/libsvn_client/ subversion/libsvn_subr/ subversion/libsvn_wc/ subversion/mod_dav_svn/ subversion/svn/ subversion/svnserve/ subversion/tests/l

2012-10-18 Thread Daniel Shahaf
Fixed all in r1399466 and r1399860. Thanks for the review. Bert Huijben wrote on Thu, Oct 18, 2012 at 13:45:46 +0200: + SVN_ERR(svn_stream_close(lsbinfo)); if (err) { svn_error_clear(err); This replaces one error leak by another one.

Re: [PATCH] Implement svnadmin verify --force

2012-10-18 Thread Stefan Fuhrmann
On Thu, Oct 18, 2012 at 11:00 PM, Daniel Shahaf d...@daniel.shahaf.namewrote: Prabhu Gnana Sundar wrote on Thu, Oct 18, 2012 at 20:35:03 +0530: Hi all, Currently svnadmin verify would stop verification process once an error/corruption is found in the repo. It does not go till the HEAD of

Re: svn commit: r1398983 - in /subversion/trunk: ./ subversion/include/ subversion/libsvn_ra_serf/ subversion/mod_dav_svn/

2012-10-18 Thread Daniel Shahaf
Julian Foad wrote on Thu, Oct 18, 2012 at 21:17:09 +0100: Author: cmpilato URL: http://svn.apache.org/viewvc?rev=1398983view=rev Log: Reintegrate into trunk the 'http-dynamic-prop-namespaces' branch (using a simple 'svn merge BRANCH-URL' with a trunk@HEAD client!). Modified:

Re: [PATCH] Implement svnadmin verify --force

2012-10-18 Thread Daniel Shahaf
Stefan Fuhrmann wrote on Thu, Oct 18, 2012 at 23:20:45 +0200: On Thu, Oct 18, 2012 at 11:00 PM, Daniel Shahaf d...@daniel.shahaf.namewrote: Prabhu Gnana Sundar wrote on Thu, Oct 18, 2012 at 20:35:03 +0530: Hi all, Currently svnadmin verify would stop verification process once an

Sparse Externals

2012-10-18 Thread Ben Reser
Regarding this issue: http://subversion.tigris.org/issues/show_bug.cgi?id=3311 We don't support using --depth options other than infinity with externals. Bert mentions that wc-ng should make it easier to implement this. However, given the way svn:externals can be defined I think that it's

Re: Sparse Externals

2012-10-18 Thread Hyrum K Wright
On Thu, Oct 18, 2012 at 5:30 PM, Ben Reser b...@reser.org wrote: Regarding this issue: http://subversion.tigris.org/issues/show_bug.cgi?id=3311 We don't support using --depth options other than infinity with externals. Bert mentions that wc-ng should make it easier to implement this.

Re: [PATCH] Implement svnadmin verify --force

2012-10-18 Thread Philip Martin
Stefan Fuhrmann stefan.fuhrm...@wandisco.com writes: Why is r=0 HEAD=$(svnlook youngest $repos) while [ $r -le $HEAD ]; do svnadmin verify -r$r $repos r=$(($r + 1)) done not an option? It will not report multiple issues with the same revision (e.g. predecessor

Re: [PATCH] Implement svnadmin verify --force

2012-10-18 Thread Daniel Shahaf
Philip Martin wrote on Thu, Oct 18, 2012 at 23:32:15 +0100: Stefan Fuhrmann stefan.fuhrm...@wandisco.com writes: Why is r=0 HEAD=$(svnlook youngest $repos) while [ $r -le $HEAD ]; do svnadmin verify -r$r $repos r=$(($r + 1)) done not an option? It will

Fwd: subversion: grammar and spelling errors in Polish translation

2012-10-18 Thread Peter Samuelson
Arfrever, Jaroslaw, Karol: you have been involved in po: pl translation for Subversion. Can you look at Jakub's report here and fix whatever needs to be fixed? (This is a fwd from http://bugs.debian.org/690815 .) Thanks, Peter [Jakub Wilk] po/pl.po has this message: msgstr --- Łączenie

RE: [PATCH v2] Remove NLS check for KWallet support and reorder private includes

2012-10-18 Thread Gavin Baumanis
Sorry for the delay, I have logged this in the issue tracker; http://subversion.tigris.org/issues/show_bug.cgi?id=4248 It is Issue # 4248 Gavin. -Original Message- From: Daniel Shahaf [mailto:d...@daniel.shahaf.name] Sent: Tuesday, 16 October 2012 11:40 To: Raphael Kubo da Costa

Re: Sparse Externals

2012-10-18 Thread Johan Corveleyn
On Thu, Oct 18, 2012 at 11:30 PM, Ben Reser b...@reser.org wrote: Regarding this issue: http://subversion.tigris.org/issues/show_bug.cgi?id=3311 We don't support using --depth options other than infinity with externals. Bert mentions that wc-ng should make it easier to implement this.

Re: Fwd: subversion: grammar and spelling errors in Polish translation

2012-10-18 Thread Arfrever Frehtes Taifersar Arahesis
2012-10-19 00:45:28 Peter Samuelson napisał(a): Arfrever, Jaroslaw, Karol: you have been involved in po: pl translation for Subversion. Can you look at Jakub's report here and fix whatever needs to be fixed? (This is a fwd from http://bugs.debian.org/690815 .) Thanks, Peter [Jakub