Re: svn commit: r995478 [1/2] - in /subversion/branches/performance: ./ subversion/bindings/javahl/native/ subversion/bindings/javahl/src/org/apache/subversion/javahl/ subversion/bindings/javahl/src/o

2010-09-10 Thread Konstantin Kolinko
2010/9/10 Hyrum Wright : > On Thu, Sep 9, 2010 at 4:34 PM, Stefan Fuhrmann > wrote: >> Hyrum Wright wrote:  *identical_p = TRUE;  /* assume TRUE, until disproved below */ -  do +  while (! (done1 || done2))    { -      SVN_ERR(svn_io_file_read_full2(file1_h, buf1, -  

Re: [PATCH] allow svnsync to translate non-UTF-8 log messages to UTF-8

2010-09-10 Thread Daniel Trebbien
>> * subversion/svnsync/main.c >>   (svnsync_cmd_table): Added svnsync_opt_source_encoding to the list >> of acceptable options for the init, sync, and copy-revprops >> subcommands. > > Missing indentation on the non-first lines.  Should be in one of these forms: > > [[[ > * subversion/svnsync/main

Re: Wrong reply-to address in mails from issue tracker

2010-09-10 Thread Daniel Shahaf
Should be fixed now. Thanks. Johan Corveleyn wrote on Sat, Sep 11, 2010 at 00:06:19 +0200: > Hi, > > The mails coming from the SVN issue tracker (on the issues mailing > list) still contain the old d...@s.t.o address as reply-to. Could > someone change this to the d...@s.a.o address? > > Cheers

Re: [PATCH] allow svnsync to translate non-UTF-8 log messages to UTF-8

2010-09-10 Thread Daniel Shahaf
Daniel Trebbien wrote on Fri, Sep 10, 2010 at 12:24:29 -0700: > The test repository (attached) was created with Subversion version > 1.6.12 (r955767) on a Debian Squeeze AMD64 system. Since you're working on this code, let me point out that 'svnadmin load' works at a level below the UTF-8 validati

Re: [PATCH] allow svnsync to translate non-UTF-8 log messages to UTF-8

2010-09-10 Thread Daniel Shahaf
Daniel Trebbien wrote on Fri, Sep 10, 2010 at 11:12:20 -0700: > On Fri, Sep 10, 2010 at 9:16 AM, Daniel Shahaf > wrote: > > The patch didn't make it to the list (only to my personal copy), could > > you please re-send with the patch in "text/plain" MIME type? > > Hmmm. I wonder if adding a .txt

Wrong reply-to address in mails from issue tracker

2010-09-10 Thread Johan Corveleyn
Hi, The mails coming from the SVN issue tracker (on the issues mailing list) still contain the old d...@s.t.o address as reply-to. Could someone change this to the d...@s.a.o address? Cheers, -- Johan

Re: [Issue 3474] making a new subdir, moving files into it and then renaming the subdir, breaks history of the moved files

2010-09-10 Thread Johan Corveleyn
On Fri, Sep 10, 2010 at 11:45 PM, wrote: > http://subversion.tigris.org/issues/show_bug.cgi?id=3474 > > --- Additional comments from joha...@tigris.org Fri Sep 10 14:45:17 -0700 > 2010 --- > This issue seems to be fixed on trunk. The described scenario now goes as > follows: > > (assumi

Re: [PATCH] allow svnsync to translate non-UTF-8 log messages to UTF-8

2010-09-10 Thread Daniel Trebbien
Note: I was using the GNU Nano repository (svn://svn.sv.gnu.org/nano) to test my changes, but it's not ideal because the first commit where the --source-encoding feature is needed is revision 619. Also, as of now, 29 of the 4518 commits have non-ASCII log messages with 21 encoded in ISO-8859-1 and

Re: [PATCH] allow svnsync to translate non-UTF-8 log messages to UTF-8

2010-09-10 Thread Daniel Trebbien
On Fri, Sep 10, 2010 at 9:16 AM, Daniel Shahaf wrote: > The patch didn't make it to the list (only to my personal copy), could > you please re-send with the patch in "text/plain" MIME type? Hmmm. I wonder if adding a .txt extension will work... Index: subversion/svnsync/sync.h ===

Re: [PATCH] allow svnsync to translate non-UTF-8 log messages to UTF-8

2010-09-10 Thread Philip Martin
Daniel Trebbien writes: > [[[ > Add a command line option to the svnsync init, sync, and copy-revprops > subcommands (--source-encoding) that allows the user to specify the > character encoding of translatable properties from the source > repository. This is needed to allow svnsync to import some

Re: [PATCH] allow svnsync to translate non-UTF-8 log messages to UTF-8

2010-09-10 Thread Daniel Shahaf
The patch didn't make it to the list (only to my personal copy), could you please re-send with the patch in "text/plain" MIME type? Daniel Trebbien wrote on Fri, Sep 10, 2010 at 09:10:08 -0700: > [[[ > Add a command line option to the svnsync init, sync, and copy-revprops > subcommands (--source-e

[PATCH] allow svnsync to translate non-UTF-8 log messages to UTF-8

2010-09-10 Thread Daniel Trebbien
[[[ Add a command line option to the svnsync init, sync, and copy-revprops subcommands (--source-encoding) that allows the user to specify the character encoding of translatable properties from the source repository. This is needed to allow svnsync to import some older Subversion repositories that

Re: svn commit: r995475 - /subversion/trunk/subversion/libsvn_repos/load.c

2010-09-10 Thread Stefan Sperling
On Fri, Sep 10, 2010 at 04:47:39PM +0200, Bert Huijben wrote: > > > > -Original Message- > > From: Stefan Sperling [mailto:s...@elego.de] > > Sent: vrijdag 10 september 2010 15:26 > > To: Bert Huijben > > Cc: dev@subversion.apache.org > > Subject: Re: svn commit: r995475 - > > /subversion

Re: NODE_DATA / NODES status

2010-09-10 Thread Philip Martin
Erik Huelsmann writes: > There are however, a few queries in 'entries.c' which operate directly on > BASE_/WORKING_NODE. These queries will need to be migrated. However, in our > old entries, we don't have the concept of op_depths and op roots. That makes > it a bit hard to migrate the entries fi

Re: svn commit: r995475 - /subversion/trunk/subversion/libsvn_repos/load.c

2010-09-10 Thread Stefan Sperling
On Fri, Sep 10, 2010 at 04:02:52PM +0200, Branko Čibej wrote: > On 10.09.2010 15:26, Stefan Sperling wrote: > > I know we're using C89, but maybe it's time to move on and upgrade to C99 > > where the benefits are desirable? When Subversion was started, C89 was about > > a decade old, and C99 is ju

RE: svn commit: r995475 - /subversion/trunk/subversion/libsvn_repos/load.c

2010-09-10 Thread Bert Huijben
> -Original Message- > From: Stefan Sperling [mailto:s...@elego.de] > Sent: vrijdag 10 september 2010 15:26 > To: Bert Huijben > Cc: dev@subversion.apache.org > Subject: Re: svn commit: r995475 - > /subversion/trunk/subversion/libsvn_repos/load.c > > On Thu, Sep 09, 2010 at 08:58:10PM +0

NODE_DATA / NODES status

2010-09-10 Thread Erik Huelsmann
Today, I finished replacing all NODE_DATA queries (UPDATE/DELETE/INSERT) in wc_db.c by queries which operate on NODES. From here on, I'll start to write code to query BASE_NODE+NODES and WORKING_NODE+NODES, verifying that both tables return the same results. There are however, a few queries in 'en

Re: svn commit: r995475 - /subversion/trunk/subversion/libsvn_repos/load.c

2010-09-10 Thread Branko Čibej
On 10.09.2010 15:26, Stefan Sperling wrote: > I know we're using C89, but maybe it's time to move on and upgrade to C99 > where the benefits are desirable? When Subversion was started, C89 was about > a decade old, and C99 is just as old now... Microsoft's C compiler, to name only one, still does

Re: svn commit: r995603 - /subversion/branches/performance/subversion/libsvn_subr/svn_string.c

2010-09-10 Thread Stefan Sperling
On Thu, Sep 09, 2010 at 10:58:06PM -, stef...@apache.org wrote: > Author: stefan2 > Date: Thu Sep 9 22:58:06 2010 > New Revision: 995603 > > URL: http://svn.apache.org/viewvc?rev=995603&view=rev > Log: > Try really hard to help the compiler generate good code > for this frequently called func

Re: svn commit: r995475 - /subversion/trunk/subversion/libsvn_repos/load.c

2010-09-10 Thread Stefan Sperling
On Thu, Sep 09, 2010 at 08:58:10PM +0200, Bert Huijben wrote: > > @@ -524,9 +524,11 @@ parse_property_block(svn_stream_t *strea > >else if ((buf[0] == 'D') && (buf[1] == ' ')) > > { > >char *keybuf; > > + apr_int64_t len; > > > > + SVN_ERR(svn_cstring

Re: atomic-revprop: help tracking down ra_serf failures

2010-09-10 Thread Daniel Shahaf
Lieven Govaerts wrote on Fri, Sep 10, 2010 at 07:46:55 +0200: > On Fri, Sep 10, 2010 at 2:12 AM, Daniel Shahaf > wrote: > > [ I'm only talking about the ra_serf tests in this email ] > > > > When I build the atomic-revprop branch against serf r1408, I see that: > > > > * diff_tests 24 29 (and man

Re: svn commit: r995478 [1/2] - in /subversion/branches/performance: ./ subversion/bindings/javahl/native/ subversion/bindings/javahl/src/org/apache/subversion/javahl/ subversion/bindings/javahl/src/o

2010-09-10 Thread Branko Čibej
On 10.09.2010 01:45, Hyrum Wright wrote: > [1] Well, I've no way of knowing if it was infinite or just taking a > long time, but I'll defer a solution to the Halting Problem as an > exercise for the reader. :) Step 1: Pull plug from power socket. Step 2: Check if program is still running. If it is

RE: [RFC] 'base' is what in --accept=base?

2010-09-10 Thread Bert Huijben
> -Original Message- > From: Paul Burba [mailto:ptbu...@gmail.com] > Sent: donderdag 9 september 2010 18:56 > To: Subversion Development > Subject: [RFC] 'base' is what in --accept=base? > > What do we expect 'svn merge --accept=base' to do when encountering > text conflicts? (Guessing: