Re: svn commit: r1499423 - /subversion/trunk/subversion/libsvn_ra_serf/options.c

2013-07-03 Thread Greg Stein
On Wed, Jul 3, 2013 at 10:40 AM, wrote: > Author: philip > Date: Wed Jul 3 14:40:21 2013 > New Revision: 1499423 > > URL: http://svn.apache.org/r1499423 > Log: > Fix busted-proxy detection: any 411 will be received without a > Subversion error. Tested by tweaking the testsuite code to set > bus

Re: svn commit: r1499541 - in /subversion/trunk: build.conf subversion/svnauth/ subversion/svnauth/svnauth.1 subversion/svnauth/svnauth.c

2013-07-03 Thread Daniel Shahaf
On Wed, Jul 03, 2013 at 08:02:02PM -, s...@apache.org wrote: > + SVN_ERR(svn_config_get_user_config_path(&config_path, > + opt_state->config_dir, NULL, > + pool)); > + footer = _("Available authe

Re: backport.pl feature requests

2013-07-03 Thread Daniel Shahaf
Daniel Shahaf wrote on Wed, Jul 03, 2013 at 17:32:46 +0300: > Are there features others would like to see in backport.pl that would > make it useful for their workflows? For example, if I added a 'show log > messages of these revisions' feature, or a 'Add my +1 to this > nomination' feature, I we

Re: subversion client having issues talking to subversion repository over HTTPS via proxy

2013-07-03 Thread Ashish SHUKLA
On Wed, 3 Jul 2013 15:26:16 +0200, Lieven Govaerts said: > Hi, > On Tue, Jun 25, 2013 at 11:17 PM, Lieven Govaerts > wrote: >> Hi, >> >> On Tue, Jun 25, 2013 at 9:37 PM, Greg Stein wrote: >>> Ashish, >>> >>> This is normally a question for users@, but I think Lieven was working >>> on somethi

Re: [PATCH] Block BDB 6

2013-07-03 Thread Daniel Shahaf
Ben Reser wrote on Wed, Jul 03, 2013 at 08:05:43 -0700: > On Wed, Jul 3, 2013 at 7:38 AM, Daniel Shahaf wrote: > > We could do the same in configure.ac: permit BDB 6 to go through if it > > was explicitly specified in the argument of --with-berkeley-db. > > I'm less confident that this is a good

Re: [PATCH] Block BDB 6

2013-07-03 Thread Ben Reser
On Wed, Jul 3, 2013 at 7:38 AM, Daniel Shahaf wrote: > We could do the same in configure.ac: permit BDB 6 to go through if it > was explicitly specified in the argument of --with-berkeley-db. I'm less confident that this is a good solution for configure because certain platforms peculiarities hav

Re: [PATCH] Block BDB 6

2013-07-03 Thread Daniel Shahaf
Ben Reser wrote on Wed, Jul 03, 2013 at 07:28:56 -0700: > On Wed, Jul 3, 2013 at 7:20 AM, Daniel Shahaf wrote: > > Ben Reser wrote on Wed, Jul 03, 2013 at 07:10:46 -0700: > >> Not sure how to really handle the Windows build. I guess we can just > >> add "60" if we test with it because we have no

RE: svn commit: r1499315 - in /subversion/branches/verify-keep-going/subversion: libsvn_repos/dump.c svnadmin/svnadmin.c

2013-07-03 Thread Bert Huijben
> -Original Message- > From: Daniel Shahaf [mailto:danie...@elego.de] > Sent: woensdag 3 juli 2013 13:38 > To: dev@subversion.apache.org > Cc: comm...@subversion.apache.org; prabh...@apache.org > Subject: Re: svn commit: r1499315 - in /subversion/branches/verify-keep- > going/subversion:

backport.pl feature requests

2013-07-03 Thread Daniel Shahaf
My workflow for reviewing backports is to run backport.pl with no arguments. It shows me each entry and allows me to merge it (real merge, not a dry-run) and view the resulting diff. When I'm done I revert the last merge, edit STATUS to add my votes, and commit. Are there features others would l

Re: [PATCH] Block BDB 6

2013-07-03 Thread Ben Reser
On Wed, Jul 3, 2013 at 7:20 AM, Daniel Shahaf wrote: > Ben Reser wrote on Wed, Jul 03, 2013 at 07:10:46 -0700: >> Not sure how to really handle the Windows build. I guess we can just >> add "60" if we test with it because we have no auto-detection, so >> you're making a deliberate choice to selec

Re: [PATCH] Block BDB 6

2013-07-03 Thread Daniel Shahaf
Ben Reser wrote on Wed, Jul 03, 2013 at 07:10:46 -0700: > On Wed, Jul 3, 2013 at 5:55 AM, Daniel Shahaf wrote: > > BDB's relicense to AGPL sparked some discussion about potentially > > blacklisting it in configure or making it opt-in. Attached a patch that > > blacklists it. (We can convert it t

Re: Subversion and KeepAlive

2013-07-03 Thread Branko Čibej
On 03.07.2013 16:07, Branko Čibej wrote: > I do have a comment; it specifically assumes the clients are browsers, ... and that the servers host web sites; important distinction there, too. -- Branko Čibej | Director of Subversion WANdisco // Non-Stop Data e. br...@wandisco.com

Re: [PATCH] Block BDB 6

2013-07-03 Thread Ben Reser
On Wed, Jul 3, 2013 at 5:55 AM, Daniel Shahaf wrote: > BDB's relicense to AGPL sparked some discussion about potentially > blacklisting it in configure or making it opt-in. Attached a patch that > blacklists it. (We can convert it to opt-in after we test with it, or > if someone asks, etc.) > >

Re: Subversion and KeepAlive

2013-07-03 Thread Branko Čibej
On 03.07.2013 15:35, Mark Phippard wrote: > I have noted KeepAlive coming up in a few of the Serf threads. > > I have been looking at setting up a load-balanced Subversion cluster, > somewhat like what Blair describes in this wiki: > > http://www.orcaware.com/svn/wiki/Server_performance_tuning_for_

Re: svn commit: r1495419 - in /subversion/trunk/subversion/libsvn_ra_serf: options.c ra_serf.h serf.c util.c

2013-07-03 Thread Philip Martin
Greg Stein writes: > Could you try svn trunk? Set busted-proxy=yes in your config to enable > the new behavior. > > Without the knob turned out, you should get a 411 error. As Stefan > pointed out, we may want to consider detecting 411 and providing a > better error message. > > With the knob ena

Subversion and KeepAlive

2013-07-03 Thread Mark Phippard
I have noted KeepAlive coming up in a few of the Serf threads. I have been looking at setting up a load-balanced Subversion cluster, somewhat like what Blair describes in this wiki: http://www.orcaware.com/svn/wiki/Server_performance_tuning_for_Linux_and_Unix Anyway, in looking at load balancers

Re: subversion client having issues talking to subversion repository over HTTPS via proxy

2013-07-03 Thread Lieven Govaerts
Hi, On Tue, Jun 25, 2013 at 11:17 PM, Lieven Govaerts wrote: > Hi, > > On Tue, Jun 25, 2013 at 9:37 PM, Greg Stein wrote: >> Ashish, >> >> This is normally a question for users@, but I think Lieven was working >> on something exactly like this already. I've cc'd him for more detail. > > Ashish a

Re: [PATCH] Block BDB 6

2013-07-03 Thread Branko Čibej
On 03.07.2013 14:55, Daniel Shahaf wrote: > BDB's relicense to AGPL sparked some discussion about potentially > blacklisting it in configure or making it opt-in. Attached a patch that > blacklists it. (We can convert it to opt-in after we test with it, or > if someone asks, etc.) > > Thoughts? L

[PATCH] Block BDB 6

2013-07-03 Thread Daniel Shahaf
BDB's relicense to AGPL sparked some discussion about potentially blacklisting it in configure or making it opt-in. Attached a patch that blacklists it. (We can convert it to opt-in after we test with it, or if someone asks, etc.) Thoughts? [[[ * build/ac-macros/berkeley-db.m4 (SVN_LIB_BERKE

Re: svn commit: r1499315 - in /subversion/branches/verify-keep-going/subversion: libsvn_repos/dump.c svnadmin/svnadmin.c

2013-07-03 Thread Prabhu
On 07/03/2013 05:07 PM, Daniel Shahaf wrote: prabh...@apache.org wrote on Wed, Jul 03, 2013 at 10:49:02 -: + notify_failure->warning_str = apr_psprintf(pool, + _("E%06d: %s"), Would it make sense to push the E%06d: part down to the implementation

Re: svn commit: r1499315 - in /subversion/branches/verify-keep-going/subversion: libsvn_repos/dump.c svnadmin/svnadmin.c

2013-07-03 Thread Daniel Shahaf
Branko Čibej wrote on Wed, Jul 03, 2013 at 13:48:57 +0200: > On 03.07.2013 13:37, Daniel Shahaf wrote: > > prabh...@apache.org wrote on Wed, Jul 03, 2013 at 10:49:02 -: > >> + notify_failure->warning_str = apr_psprintf(pool, > >> + _("E%06d: %s"), >

Re: svn commit: r1499315 - in /subversion/branches/verify-keep-going/subversion: libsvn_repos/dump.c svnadmin/svnadmin.c

2013-07-03 Thread Branko Čibej
On 03.07.2013 13:37, Daniel Shahaf wrote: > prabh...@apache.org wrote on Wed, Jul 03, 2013 at 10:49:02 -: >> + notify_failure->warning_str = apr_psprintf(pool, >> + _("E%06d: %s"), > Would it make sense to push the E%06d: part down to the implementat

Re: --rm-I alias to --remove-ignored

2013-07-03 Thread Daniel Shahaf
Konstantin Kolinko wrote on Wed, Jul 03, 2013 at 02:22:54 +0400: > 2013/7/2 Daniel Shahaf : > > How about adding 'svn cleanup --rm-I' as a short option for 'svn cleanup > > --remove-ignored'? > > > > Currently we have --cl as a short option for --changelist, and in the > > past we decided not to ad

Re: svn commit: r1499315 - in /subversion/branches/verify-keep-going/subversion: libsvn_repos/dump.c svnadmin/svnadmin.c

2013-07-03 Thread Daniel Shahaf
prabh...@apache.org wrote on Wed, Jul 03, 2013 at 10:49:02 -: > + notify_failure->warning_str = apr_psprintf(pool, > + _("E%06d: %s"), Would it make sense to push the E%06d: part down to the implementation of svn_err_best_message()? I.e., if the in

Re: svn commit: r1498394 - in /subversion/branches/verify-keep-going/subversion: libsvn_repos/dump.c svnadmin/svnadmin.c

2013-07-03 Thread Daniel Shahaf
Prabhu wrote on Wed, Jul 03, 2013 at 15:11:36 +0530: > Do you want it to look like, > > < > $ svnadmin verify /tmp/testrepo/ --keep-going > * Verifying repository metadata ... > * Verified revision 0. > * Verified revision 1. > * Error verifying revision 2. > svnadmin: E160004: Final line in revisi

Re: svn commit: r1498947 - /subversion/trunk/tools/dev/po-merge.py

2013-07-03 Thread Dongsheng Song
于 2013/7/3 18:11, Konstantin Kolinko 写道: > 2013/7/3 Dongsheng Song : >> On 2013/7/3 6:55, Konstantin Kolinko wrote: >>> 2013/7/3 Andreas Stieger : Hi There, On 02/07/13 16:00, Dongsheng Song wrote: > Today, when I merge zh_CN.po from trunk to 1.8.x, I had encounter the > foll

Re: svn commit: r1498394 - in /subversion/branches/verify-keep-going/subversion: libsvn_repos/dump.c svnadmin/svnadmin.c

2013-07-03 Thread Stefan Sperling
On Wed, Jul 03, 2013 at 11:56:16AM +0200, Branko Čibej wrote: > svnadmin: E165011: Repository '/tmp/testrepo' failed to verify > Summary of corrupt revisions: > Revision 2: E160004: Final line in revision file r2 missing space > Revision 4: E160004: Invalid change kind in rev file > Revision 6: E16

Re: svn commit: r1498947 - /subversion/trunk/tools/dev/po-merge.py

2013-07-03 Thread Konstantin Kolinko
2013/7/3 Dongsheng Song : > On 2013/7/3 6:55, Konstantin Kolinko wrote: >> 2013/7/3 Andreas Stieger : >>> Hi There, >>> >>> On 02/07/13 16:00, Dongsheng Song wrote: Today, when I merge zh_CN.po from trunk to 1.8.x, I had encounter the following error: $ python ../../../../trunk/

Re: svn commit: r1498394 - in /subversion/branches/verify-keep-going/subversion: libsvn_repos/dump.c svnadmin/svnadmin.c

2013-07-03 Thread Branko Čibej
On 03.07.2013 11:41, Prabhu wrote: > On 07/01/2013 05:35 PM, Daniel Shahaf wrote: >> >>> +++ >>> subversion/branches/verify-keep-going/subversion/svnadmin/svnadmin.c >>> Mon Jul 1 12:01:38 2013 >>> @@ -850,13 +850,16 @@ repos_notify_handler(void *baton, >>> case svn_repos_notify_failure_summ

Re: svn commit: r1498394 - in /subversion/branches/verify-keep-going/subversion: libsvn_repos/dump.c svnadmin/svnadmin.c

2013-07-03 Thread Prabhu
On 07/03/2013 03:11 PM, Prabhu wrote: On 07/01/2013 05:35 PM, Daniel Shahaf wrote: +++ subversion/branches/verify-keep-going/subversion/svnadmin/svnadmin.c Mon Jul 1 12:01:38 2013 @@ -850,13 +850,16 @@ repos_notify_handler(void *baton, case svn_repos_notify_failure_summary: if

Re: svn commit: r1498394 - in /subversion/branches/verify-keep-going/subversion: libsvn_repos/dump.c svnadmin/svnadmin.c

2013-07-03 Thread Prabhu
On 07/01/2013 05:35 PM, Daniel Shahaf wrote: +++ subversion/branches/verify-keep-going/subversion/svnadmin/svnadmin.c Mon Jul 1 12:01:38 2013 @@ -850,13 +850,16 @@ repos_notify_handler(void *baton, case svn_repos_notify_failure_summary: if (notify->revision != SVN_INVALID_REVNUM

Re: fsfs-format7 integration plan

2013-07-03 Thread Ivan Zhakov
On Tue, Jul 2, 2013 at 9:00 PM, Ben Reser wrote: > On Tue, Jul 2, 2013 at 5:07 AM, Greg Stein wrote: >> As noted on IRC earlier, we just deprecated BDB so that we wouldn't >> have to continue supporting multiple backends. But it seems you have >> just created a third/new backend. > > I think that