Re: Missing reported recursively; and a tree-conflict upon checkout

2010-10-25 Thread Julian Foad
On Sat, 2010-10-23, Daniel Shahaf wrote: > My shell session: Using exactly what build of svn? > % $svn checkout --depth empty > https://svn.apache.org/repos/infra/infrastructure/ ./ > % $svn update --set-depth infinity trunk > % cd trunk > % rm -rf machines > % rm -rf tools >

Re: [PATCH] for building subversion 1.6.12 for haiku

2010-10-25 Thread Julian Foad
On Fri, 2010-10-22, scott mc wrote: > We've run into another issue on Haiku, perhaps it's just something I > did wrong when configuring it, seems the subversion port on Haiku > thinks the system uses ISO-8859-15 encoding: > http://ports.haiku-files.org/ticket/422 > Haiku's locale kit is still evolv

Re: Missing reported recursively; and a tree-conflict upon checkout

2010-10-25 Thread Daniel Shahaf
Julian Foad wrote on Mon, Oct 25, 2010 at 09:11:20 +0100: > On Sat, 2010-10-23, Daniel Shahaf wrote: > > My shell session: > > Using exactly what build of svn? > tr...@r1026650, probably. It was not more than a day old at the time I wrote the email.

Re: svn commit: r1026475 - in /subversion/trunk/subversion: svn/ tests/cmdline/ tests/cmdline/getopt_tests_data/

2010-10-25 Thread Julian Foad
> URL: http://svn.apache.org/viewvc?rev=1026475&view=rev > Log: > Add an 'svn relocate' first-class subcommand. For too long has relocate > functionality has been awkwardly tacked onto the back of 'svn switch'. Lovely. The help makes it so much clearer because no options are listed in the help f

Re: [PATCH] for building subversion 1.6.12 for haiku

2010-10-25 Thread Philip Martin
Julian Foad writes: > On Fri, 2010-10-22, scott mc wrote: >> We've run into another issue on Haiku, perhaps it's just something I >> did wrong when configuring it, seems the subversion port on Haiku >> thinks the system uses ISO-8859-15 encoding: >> http://ports.haiku-files.org/ticket/422 >> Haik

svn patch --strip-count=N or svn patch --strip=N?

2010-10-25 Thread Julian Foad
Do we need both "--strip-count" and "--strip" as aliases? I think "--strip" is sufficient; the word "count" doesn't add much. "--strip" matches GNU patch. Fewer aliases -> better. ? - Julian

Re: svn patch --strip-count=N or svn patch --strip=N?

2010-10-25 Thread Stefan Sperling
On Mon, Oct 25, 2010 at 01:16:29PM +0100, Julian Foad wrote: > Do we need both "--strip-count" and "--strip" as aliases? I think > "--strip" is sufficient; the word "count" doesn't add much. "--strip" > matches GNU patch. Fewer aliases -> better. The word "count" adds something when you're look

Re: svn patch --strip-count=N or svn patch --strip=N?

2010-10-25 Thread C. Michael Pilato
On 10/25/2010 09:39 AM, Stefan Sperling wrote: > On Mon, Oct 25, 2010 at 01:16:29PM +0100, Julian Foad wrote: >> Do we need both "--strip-count" and "--strip" as aliases? I think >> "--strip" is sufficient; the word "count" doesn't add much. "--strip" >> matches GNU patch. Fewer aliases -> bette

Re: svn commit: r1027029 - /subversion/trunk/subversion/svn/main.c

2010-10-25 Thread Stefan Sperling
On Mon, Oct 25, 2010 at 10:21:27AM -, julianf...@apache.org wrote: > Author: julianfoad > Date: Mon Oct 25 10:21:27 2010 > New Revision: 1027029 > > URL: http://svn.apache.org/viewvc?rev=1027029&view=rev > Log: > * subversion/svn/main.c > (svn_cl__options): Remove the alias '--nul' (for --no

Re: svn patch --strip-count=N or svn patch --strip=N?

2010-10-25 Thread Stefan Sperling
On Mon, Oct 25, 2010 at 09:50:41AM -0400, C. Michael Pilato wrote: > On 10/25/2010 09:39 AM, Stefan Sperling wrote: > > On Mon, Oct 25, 2010 at 01:16:29PM +0100, Julian Foad wrote: > >> Do we need both "--strip-count" and "--strip" as aliases? I think > >> "--strip" is sufficient; the word "count"

Re: svn patch --strip-count=N or svn patch --strip=N?

2010-10-25 Thread C. Michael Pilato
On 10/25/2010 10:13 AM, Stefan Sperling wrote: >>> The --strip-count name matches the variable name in the code. >> >> Er... This is a non-reason. :-) > > It's not a non-reason because there is no good reason for not naming > things in a consistent way. > > Consistency between the API and the CL

Re: svn patch --strip-count=N or svn patch --strip=N?

2010-10-25 Thread Stefan Sperling
On Mon, Oct 25, 2010 at 10:46:52AM -0400, C. Michael Pilato wrote: > We can always rename an API variable. :-) But not without having a huge bikeshed discussion about the new name first! :)

Re: svn commit: r1027029 - /subversion/trunk/subversion/svn/main.c

2010-10-25 Thread Julian Foad
On Mon, 2010-10-25 at 15:54 +0200, Stefan Sperling wrote: > On Mon, Oct 25, 2010 at 10:21:27AM -, julianf...@apache.org wrote: > > Author: julianfoad > > Date: Mon Oct 25 10:21:27 2010 > > New Revision: 1027029 > > > > URL: http://svn.apache.org/viewvc?rev=1027029&view=rev > > Log: > > * subve

Re: svn patch --strip-count=N or svn patch --strip=N?

2010-10-25 Thread Hyrum K. Wright
On Mon, Oct 25, 2010 at 9:13 AM, Stefan Sperling wrote: ... > It's not a non-reason because there is no good reason for not naming > things in a consistent way. ENEGOVERFLOW :) -Hyrum

Re: svn patch --strip-count=N or svn patch --strip=N?

2010-10-25 Thread Beware of Bikesheds
Stefan Sperling wrote on Mon, Oct 25, 2010 at 16:13:58 +0200: > On Mon, Oct 25, 2010 at 09:50:41AM -0400, C. Michael Pilato wrote: > > I don't think it's so terribly important to match the 'patch' command's > > option name with our option's primary name, but it's a nice-to-have > > achievable via t

Re: svn commit: r985606 - /subversion/branches/performance/subversion/libsvn_ra_svn/marshal.c

2010-10-25 Thread Hyrum K. Wright
On Sat, Aug 14, 2010 at 7:20 PM, wrote: > Author: stefan2 > Date: Sun Aug 15 00:20:34 2010 > New Revision: 985606 > > URL: http://svn.apache.org/viewvc?rev=985606&view=rev > Log: > Increase the RA_SVN throughput by reducing the overhead in read_item for > packing received data into various struct

wc-ng code organisation

2010-10-25 Thread Philip Martin
I've been looking at a delete bug: delete can remove a directory NODES row and leave rows for children, thus creating an invalid metadata state. The code is structured as svn_wc_delete4(path): various checks on path for each child of path: various checks on child

Conflict storage

2010-10-25 Thread Hyrum K. Wright
As I understand it, the conflict storage handling updates are one of the blocking items for 1.7[1]. In an effort to better understand the problem, and the proposed solution, I'm writing this mail with the intent to start some discussion. The content of notes/wc-ng/conflict-storage seem to describ

Re: Conflict storage

2010-10-25 Thread Stefan Sperling
On Mon, Oct 25, 2010 at 03:42:16PM -0500, Hyrum K. Wright wrote: > [1] I don't fully understand *why* this is true, just that it is. My > guess is that people have thought "hey, while we're changing > everything in the working copy, let's change this, too." If that is > the sole reason for making