RE: [Patch] - Fix compilation of direct_uri-test.c on OS/2

2010-07-22 Thread Bert Huijben
> -Original Message- > From: Eric Zeitler [mailto:eric.zeit...@gmail.com] > Sent: vrijdag 23 juli 2010 1:48 > To: Bert Huijben > Cc: dev@subversion.apache.org > Subject: Re: [Patch] - Fix compilation of direct_uri-test.c on OS/2 > > On 7/22/2010 4:02 PM, Bert Huijben wrote: > > > >> ---

Re: [Patch] - Fix compilation of direct_uri-test.c on OS/2

2010-07-22 Thread Eric Zeitler
On 7/22/2010 4:02 PM, Bert Huijben wrote: > >> -Original Message- >> From: Eric Zeitler [mailto:eric.zeit...@gmail.com] >> Sent: donderdag 22 juli 2010 23:50 >> To: dev@subversion.apache.org >> Subject: Re: [Patch] - Fix compilation of direct_uri-test.c on OS/2 >> >> Matthew Bentham artvp

RE: [Patch] - Fix compilation of direct_uri-test.c on OS/2

2010-07-22 Thread Bert Huijben
> -Original Message- > From: Eric Zeitler [mailto:eric.zeit...@gmail.com] > Sent: donderdag 22 juli 2010 23:50 > To: dev@subversion.apache.org > Subject: Re: [Patch] - Fix compilation of direct_uri-test.c on OS/2 > > Matthew Bentham artvps.com> writes: > > > > > On 28/06/2010 10:29, Be

Re: svn commit: r966822 - in /subversion/trunk/subversion: libsvn_client/repos_diff.c tests/cmdline/merge_tests.py

2010-07-22 Thread Paul Burba
On Thu, Jul 22, 2010 at 5:04 PM, Bert Huijben wrote: > > >> -Original Message- >> From: pbu...@apache.org [mailto:pbu...@apache.org] >> Sent: donderdag 22 juli 2010 21:43 >> To: comm...@subversion.apache.org >> Subject: svn commit: r966822 - in /subversion/trunk/subversion: >> libsvn_clien

Re: [Patch] - Fix compilation of direct_uri-test.c on OS/2

2010-07-22 Thread Eric Zeitler
Matthew Bentham artvps.com> writes: > > On 28/06/2010 10:29, Bert Huijben wrote: > > > > > >> -Original Message- > >> From: Julian Foad [mailto:julian.foad wandisco.com] > >> Sent: maandag 28 juni 2010 10:48 > >> To: Paul Smedley > >> Cc: dev subversion.apache.org > >> Subject: Re: [Pa

RE: svn commit: r966822 - in /subversion/trunk/subversion: libsvn_client/repos_diff.c tests/cmdline/merge_tests.py

2010-07-22 Thread Bert Huijben
> -Original Message- > From: pbu...@apache.org [mailto:pbu...@apache.org] > Sent: donderdag 22 juli 2010 21:43 > To: comm...@subversion.apache.org > Subject: svn commit: r966822 - in /subversion/trunk/subversion: > libsvn_client/repos_diff.c tests/cmdline/merge_tests.py > > Author: pburb

RE: svn commit: r966770 - in /subversion/trunk/subversion/libsvn_wc: copy.c log.c log.h merge.c workqueue.c workqueue.h

2010-07-22 Thread Bert Huijben
> -Original Message- > From: Daniel Shahaf [mailto:d...@daniel.shahaf.name] > Sent: donderdag 22 juli 2010 20:58 > To: dev@subversion.apache.org > Cc: comm...@subversion.apache.org > Subject: Re: svn commit: r966770 - in > /subversion/trunk/subversion/libsvn_wc: copy.c log.c log.h merge.c

Re: svn commit: r966841 - /subversion/trunk/subversion/svnrdump/svnrdump.c

2010-07-22 Thread Stefan Sperling
On Thu, Jul 22, 2010 at 08:18:17PM -, artag...@apache.org wrote: > Author: artagnon > Date: Thu Jul 22 20:18:16 2010 > New Revision: 966841 > > URL: http://svn.apache.org/viewvc?rev=966841&view=rev > Log: > * subversion/svnrdump/svnrdump.c (replay_range, main, usage): Fake a > revision 0 lik

Re: svn commit: r966774 - /subversion/branches/issue-2779-dev/subversion/libsvn_client/update. c

2010-07-22 Thread C. Michael Pilato
On 07/22/2010 03:01 PM, Daniel Shahaf wrote: > C. Michael Pilato wrote on Thu, Jul 22, 2010 at 14:53:51 -0400: >> On 07/22/2010 02:12 PM, C. Michael Pilato wrote: >>> Hrm... 'svn relocate' just calls svn_client_relocate() as well. I don't >>> see any calls to svn_wc_is_wc_rootX() or anything. Do

Re: svn commit: r966774 - /subversion/branches/issue-2779-dev/subversion/libsvn_client/update. c

2010-07-22 Thread Daniel Shahaf
C. Michael Pilato wrote on Thu, Jul 22, 2010 at 14:53:51 -0400: > On 07/22/2010 02:12 PM, C. Michael Pilato wrote: > > Hrm... 'svn relocate' just calls svn_client_relocate() as well. I don't > > see any calls to svn_wc_is_wc_rootX() or anything. Does that allow partial > > relocations, too? > >

Re: svn commit: r966774 - /subversion/branches/issue-2779-dev/subversion/libsvn_client/update. c

2010-07-22 Thread Stefan Sperling
On Thu, Jul 22, 2010 at 09:53:02PM +0300, Daniel Shahaf wrote: > Stefan Sperling wrote on Thu, Jul 22, 2010 at 20:11:23 +0200: > > On Thu, Jul 22, 2010 at 07:59:42PM +0200, Bert Huijben wrote: > > > Shouldn't we relocate the complete working copy, starting by the working > > > copy root instead of

Re: svn commit: r966770 - in /subversion/trunk/subversion/libsvn_wc: copy.c log.c log.h merge.c workqueue.c workqueue.h

2010-07-22 Thread Daniel Shahaf
rhuij...@apache.org wrote on Thu, Jul 22, 2010 at 17:44:41 -: > Author: rhuijben > Date: Thu Jul 22 17:44:41 2010 > New Revision: 966770 > > URL: http://svn.apache.org/viewvc?rev=966770&view=rev > Log: > Replace the file-move loggy operation with a file-move wq operation. > > This to move awa

Re: svn commit: r966774 - /subversion/branches/issue-2779-dev/subversion/libsvn_client/update. c

2010-07-22 Thread Daniel Shahaf
Stefan Sperling wrote on Thu, Jul 22, 2010 at 20:11:23 +0200: > On Thu, Jul 22, 2010 at 07:59:42PM +0200, Bert Huijben wrote: > > Shouldn't we relocate the complete working copy, starting by the working > > copy root instead of just the operation anchor? > So "cd notes; svn up commit-access-temp

Re: svn commit: r966774 - /subversion/branches/issue-2779-dev/subversion/libsvn_client/update.c

2010-07-22 Thread C. Michael Pilato
On 07/22/2010 02:12 PM, C. Michael Pilato wrote: > Hrm... 'svn relocate' just calls svn_client_relocate() as well. I don't > see any calls to svn_wc_is_wc_rootX() or anything. Does that allow partial > relocations, too? To answer my own question: Yes. 'svn relocate', when run on a working cop

Re: svn commit: r966774 - /subversion/branches/issue-2779-dev/subversion/libsvn_client/update.c

2010-07-22 Thread C. Michael Pilato
On 07/22/2010 02:11 PM, Stefan Sperling wrote: > I'm thinking a callback mechanism to ask users if they really > want to relocate to the new URL would be nice. And I agree that > the entire working copy should be relocated (should be easy in WC-NG, > right? :) If you read the BRANCH-README, you'll

Re: svn commit: r966774 - /subversion/branches/issue-2779-dev/subversion/libsvn_client/update.c

2010-07-22 Thread C. Michael Pilato
On 07/22/2010 02:09 PM, C. Michael Pilato wrote: > On 07/22/2010 01:59 PM, Bert Huijben wrote: >>> -Original Message- >>> From: cmpil...@apache.org [mailto:cmpil...@apache.org] >>> Sent: donderdag 22 juli 2010 19:55 >>> To: comm...@subversion.apache.org >>> Subject: svn commit: r966774 - /s

Re: svn commit: r966774 - /subversion/branches/issue-2779-dev/subversion/libsvn_client/update.c

2010-07-22 Thread Stefan Sperling
On Thu, Jul 22, 2010 at 07:59:42PM +0200, Bert Huijben wrote: > > > > -Original Message- > > From: cmpil...@apache.org [mailto:cmpil...@apache.org] > > Sent: donderdag 22 juli 2010 19:55 > > To: comm...@subversion.apache.org > > Subject: svn commit: r966774 - /subversion/branches/issue-27

Re: svn commit: r966774 - /subversion/branches/issue-2779-dev/subversion/libsvn_client/update.c

2010-07-22 Thread C. Michael Pilato
On 07/22/2010 01:59 PM, Bert Huijben wrote: >> -Original Message- >> From: cmpil...@apache.org [mailto:cmpil...@apache.org] >> Sent: donderdag 22 juli 2010 19:55 >> To: comm...@subversion.apache.org >> Subject: svn commit: r966774 - /subversion/branches/issue-2779- >> dev/subversion/libsvn_

RE: svn commit: r966774 - /subversion/branches/issue-2779-dev/subversion/libsvn_client/update.c

2010-07-22 Thread Bert Huijben
> -Original Message- > From: cmpil...@apache.org [mailto:cmpil...@apache.org] > Sent: donderdag 22 juli 2010 19:55 > To: comm...@subversion.apache.org > Subject: svn commit: r966774 - /subversion/branches/issue-2779- > dev/subversion/libsvn_client/update.c > > Author: cmpilato > Date: Th

Re: --native-eol not applied to externals

2010-07-22 Thread Mark Hanson
On Thu, Jul 22, 2010 at 10:03 AM, Bert Huijben wrote: > Fixed this issue on trunk in r966747. > >>       Bert Excellent! Thanks, Bert!

RE: --native-eol not applied to externals

2010-07-22 Thread Bert Huijben
> -Original Message- > From: Bert Huijben [mailto:b...@qqmail.nl] > Sent: donderdag 22 juli 2010 17:54 > To: 'Mark Hanson'; dev@subversion.apache.org > Subject: RE: --native-eol not applied to externals > > > > > -Original Message- > > From: Mark Hanson [mailto:mbh...@gmail.com

RE: --native-eol not applied to externals

2010-07-22 Thread Bert Huijben
> -Original Message- > From: Mark Hanson [mailto:mbh...@gmail.com] > Sent: donderdag 22 juli 2010 17:25 > To: dev@subversion.apache.org > Subject: --native-eol not applied to externals > > The following commands demonstrate what seems to be a bug with > --native-eol and externals. Two r

Re: svn commit: r965892 - in /subversion/trunk: notes/dump-load-format.txt subversion/include/svn_repos.h subversion/libsvn_repos/dump.c

2010-07-22 Thread C. Michael Pilato
On 07/22/2010 11:33 AM, Stefan Sperling wrote: > On Thu, Jul 22, 2010 at 10:56:11AM -0400, C. Michael Pilato wrote: >> The error you show above happens deep inside the FS implementations' DAG >> layer, as a sanity check on a text-base before apply a delta to it. It's >> probably comparing the chec

Re: svn commit: r965892 - in /subversion/trunk: notes/dump-load-format.txt subversion/include/svn_repos.h subversion/libsvn_repos/dump.c

2010-07-22 Thread Stefan Sperling
On Thu, Jul 22, 2010 at 10:56:11AM -0400, C. Michael Pilato wrote: > svnsync doesn't do any checksum verification. That work is done by deeper > levels. But you provide no support for the implication that Subversion > didn't attempt to verify checksums at all the reasonable times. Looks like I n

--native-eol not applied to externals

2010-07-22 Thread Mark Hanson
The following commands demonstrate what seems to be a bug with --native-eol and externals. Two repositories are created. The first has an external link to the second. Text files with svn:eol-style=native are added in both repositories. When the first repository is exported with a non-default --

Re: svn commit: r965892 - in /subversion/trunk: notes/dump-load-format.txt subversion/include/svn_repos.h subversion/libsvn_repos/dump.c

2010-07-22 Thread C. Michael Pilato
On 07/22/2010 10:11 AM, Stefan Sperling wrote: > So we should not have added the additional checksums in the first place? > You might as well point fingers at the old backends for not storing checksums, > rather than at Hyrum trying to do his best with what he had. You have to define "his best" he

Re: BUG: when performing a checkout use the cased url as in the download URL

2010-07-22 Thread dave b
On 22 July 2010 23:59, Bert Huijben wrote: > > >> -Original Message- >> From: Matthew Bentham [mailto:mj...@artvps.com] >> Sent: donderdag 22 juli 2010 15:32 >> To: s...@stsp.name >> Cc: dave b; Stephen Butler; Bob Archer; Bolstridge, Andrew; >> dev@subversion.apache.org >> Subject: Re: BU

Re: svn commit: r965892 - in /subversion/trunk: notes/dump-load-format.txt subversion/include/svn_repos.h subversion/libsvn_repos/dump.c

2010-07-22 Thread Stefan Sperling
On Thu, Jul 22, 2010 at 09:00:25AM -0400, C. Michael Pilato wrote: > On 07/22/2010 06:15 AM, Stefan Sperling wrote: > > On Thu, Jul 22, 2010 at 09:53:35AM +0100, Philip Martin wrote: > >> Perhaps we could just have one checksum that includes all the property > >> names and values? > > > > I'd like

RE: BUG: when performing a checkout use the cased url as in the download URL

2010-07-22 Thread Bert Huijben
> -Original Message- > From: Matthew Bentham [mailto:mj...@artvps.com] > Sent: donderdag 22 juli 2010 15:32 > To: s...@stsp.name > Cc: dave b; Stephen Butler; Bob Archer; Bolstridge, Andrew; > dev@subversion.apache.org > Subject: Re: BUG: when performing a checkout use the cased url as in

Re: svn commit: r965892 - in /subversion/trunk: notes/dump-load-format.txt subversion/include/svn_repos.h subversion/libsvn_repos/dump.c

2010-07-22 Thread C. Michael Pilato
On 07/22/2010 09:40 AM, Daniel Shahaf wrote: > C. Michael Pilato wrote on Thu, Jul 22, 2010 at 09:00:25 -0400: >> Now for some nit-picking. This format: >> >>Header-Name: (PROPNAME) CHECKSUM >> >> is unnecessarily complex and, I bet, will catch at least person offguard >> when their dumpstream

Re: BUG: when performing a checkout use the cased url as in the download URL

2010-07-22 Thread dave b
On 22 July 2010 23:08, Stefan Sperling wrote: > On Thu, Jul 22, 2010 at 10:58:44PM +1000, dave b wrote: >> Can we at least agree that the current state is suboptimal ? > > Sure, but it's the state of windows that is suboptimal. > There simply isn't anything we can reasonably do about it. > > Stefa

Re: svn commit: r965892 - in /subversion/trunk: notes/dump-load-format.txt subversion/include/svn_repos.h subversion/libsvn_repos/dump.c

2010-07-22 Thread Daniel Shahaf
C. Michael Pilato wrote on Thu, Jul 22, 2010 at 09:00:25 -0400: > Now for some nit-picking. This format: > >Header-Name: (PROPNAME) CHECKSUM > > is unnecessarily complex and, I bet, will catch at least person offguard > when their dumpstream contains a property name with a ')' character in i

Re: BUG: when performing a checkout use the cased url as in the download URL

2010-07-22 Thread Matthew Bentham
On 22/07/2010 14:08, s...@stsp.name wrote: On Thu, Jul 22, 2010 at 10:58:44PM +1000, dave b wrote: Can we at least agree that the current state is suboptimal ? Sure, but it's the state of windows that is suboptimal. There simply isn't anything we can reasonably do about it. Stefan What if

Re: BUG: when performing a checkout use the cased url as in the download URL

2010-07-22 Thread Stefan Sperling
On Thu, Jul 22, 2010 at 10:58:44PM +1000, dave b wrote: > Can we at least agree that the current state is suboptimal ? Sure, but it's the state of windows that is suboptimal. There simply isn't anything we can reasonably do about it. Stefan

Re: svn commit: r965892 - in /subversion/trunk: notes/dump-load-format.txt subversion/include/svn_repos.h subversion/libsvn_repos/dump.c

2010-07-22 Thread C. Michael Pilato
On 07/22/2010 06:15 AM, Stefan Sperling wrote: > On Thu, Jul 22, 2010 at 09:53:35AM +0100, Philip Martin wrote: >> Perhaps we could just have one checksum that includes all the property >> names and values? > > I'd like loaders to be able to tell users which properties are > corrupted. This chang

Re: BUG: when performing a checkout use the cased url as in the download URL

2010-07-22 Thread dave b
On 22 July 2010 19:55, Stefan Sperling wrote: > On Thu, Jul 22, 2010 at 05:10:01PM +1000, dave b wrote: >> Perhaps if it is possible, svn on windows can use the posix api? - I >> don't know how that works :) > http://en.wikipedia.org/wiki/Microsoft_Windows_Services_for_UNIX#cite_note-6 > -> http:/

Re: svn commit: r964772 - /subversion/branches/1.6.x/STATUS

2010-07-22 Thread Senthil Kumaran S
Bert Huijben wrote: > I think this needs a backport branch. > svn_uri_canonicalize() is not publicly available in 1.6 and the private > variant that is in the 1.6 libsvn_subr didn't perform url encoding fixes. > That was only added on trunk a few months ago. I created a backport branch for this

Re: svn commit: r965892 - in /subversion/trunk: notes/dump-load-format.txt subversion/include/svn_repos.h subversion/libsvn_repos/dump.c

2010-07-22 Thread Stefan Sperling
On Thu, Jul 22, 2010 at 09:53:35AM +0100, Philip Martin wrote: > s...@apache.org writes: > > > Author: stsp > > Date: Tue Jul 20 16:14:53 2010 > > New Revision: 965892 > > > > URL: http://svn.apache.org/viewvc?rev=965892&view=rev > > Log: > > Make svnadmin dump print headers containing MD5 and SHA

Re: BUG: when performing a checkout use the cased url as in the download URL

2010-07-22 Thread Stefan Sperling
On Thu, Jul 22, 2010 at 05:10:01PM +1000, dave b wrote: > Perhaps if it is possible, svn on windows can use the posix api? - I > don't know how that works :) It doesn't work quite like that. Subversion uses APR (the apache portable runtime) for portability. This means that file i/o, memory handlin

Re: svn commit: r965892 - in /subversion/trunk: notes/dump-load-format.txt subversion/include/svn_repos.h subversion/libsvn_repos/dump.c

2010-07-22 Thread Philip Martin
s...@apache.org writes: > Author: stsp > Date: Tue Jul 20 16:14:53 2010 > New Revision: 965892 > > URL: http://svn.apache.org/viewvc?rev=965892&view=rev > Log: > Make svnadmin dump print headers containing MD5 and SHA1 checksums of > property content, as was already done for file content. Checksum

Re: svn commit: r966312 - in /subversion/trunk/subversion: include/ libsvn_client/ libsvn_wc/

2010-07-22 Thread Philip Martin
"Bert Huijben" writes: > It looks like this patch breaks the python bindings. Right. r966539 switched to using svn_stream_open_unique, the tests now pass on my Linux box. -- Philip

Re: BUG: when performing a checkout use the cased url as in the download URL

2010-07-22 Thread dave b
On 22 July 2010 16:49, Stephen Butler wrote: > > On Jul 22, 2010, at 7:10 , dave b wrote: > >> On 22 July 2010 05:47, Bob Archer wrote: >>> On 22 July 2010 03:12, dave b wrote: > On 22 July 2010 02:58, Bolstridge, Andrew > wrote: >>> -Original Message- >>> excuse to