Re: [PATCH] svn command - ls - Multiple targets

2011-02-10 Thread Noorul Islam K M
Daniel Shahaf writes: > Noorul Islam K M wrote on Thu, Feb 10, 2011 at 16:50:39 +0530: > >> Index: subversion/tests/cmdline/basic_tests.py >> +def ls_multiple_url_targets(sbox): >> + "ls multiple url targets" >> + >> + sbox.build() >> + >> + alpha = sbox.repo_url + '/A/B/E/alpha' >> + beta =

Re: [PATCH] svn command - ls - Multiple targets

2011-02-10 Thread Daniel Shahaf
Noorul Islam K M wrote on Thu, Feb 10, 2011 at 16:50:39 +0530: > Index: subversion/tests/cmdline/basic_tests.py > +def ls_multiple_url_targets(sbox): > + "ls multiple url targets" > + > + sbox.build() > + > + alpha = sbox.repo_url + '/A/B/E/alpha' > + beta = sbox.repo_url + '/A/B/E/beta' > +

Re: svn commit: r1068814 - /subversion/trunk/subversion/tests/libsvn_diff/diff-diff3-test.c

2011-02-10 Thread Daniel Shahaf
Thanks for the review; jcorvel fixed it in a subsequent commit.

Re: does 'svnadmin dump/verify' use the membuffer caches by default?

2011-02-10 Thread Daniel Shahaf
Stefan Fuhrmann wrote on Thu, Feb 10, 2011 at 00:08:46 +0100: > On 09.02.2011 03:43, Daniel Shahaf wrote: > >Stefan, > > > >> < wayita> stefan2: danielsh said: does 'svnadmin dump/verify' use the > >> caches by default? > >><@stefan2> danielsh: yes. However, the biggest performance improvement

Re: Combining public and private paths

2011-02-10 Thread Victor Sudakov
What the "anon-access = none" option does is remove the ANONYMOUS mech from the list of SASL mechs offered by svnserve (I see this in tcpflow). If this mech is present in the mech list, the svn client does not bother to authenticate even if a valid Kerberos ticket is available. If the svn client h

Re: Combining public and private paths

2011-02-10 Thread Victor Sudakov
The problem is probably in the following. When anon-access is other than "none", svnserve does not request authentication for some important operations like "svn log", and I have found no way to force it to request authentication. This effectively breaks path based authorization. I have found som

Re: svn commit: r1068863 - /subversion/trunk/build/generator/gen_msvc_dsp.py

2011-02-10 Thread Joe Swatosh
Hi Bert, On Thu, Feb 10, 2011 at 2:54 AM, Bert Huijben wrote: >> -Original Message- >> From: Joe Swatosh [mailto:joe.swat...@gmail.com] >> Sent: woensdag 9 februari 2011 23:43 >> To: dev@subversion.apache.org >> Cc: rhuij...@apache.org >> Subject: Re: svn commit: r1068863 - >> /subversion

Re: Integrating further changes from the performance branch

2011-02-10 Thread Stefan Fuhrmann
On 10.02.2011 22:15, Ivan Zhakov wrote: On Thu, Feb 10, 2011 at 03:25, Stefan Fuhrmann wrote: Hi there, Those among you following the commits will have noticed that I opened 5 new branches each containing a cherry- picked patch set from the performance branch (details below). These branches

Re: svn commit: r1069176 - in /subversion/branches/integrate-stream-api-extensions/subversion: include/svn_io.h libsvn_subr/stream.c libsvn_subr/subst.c

2011-02-10 Thread Stefan Fuhrmann
On 10.02.2011 11:03, Bert Huijben wrote: -Original Message- From: stef...@apache.org [mailto:stef...@apache.org] Sent: donderdag 10 februari 2011 1:36 To: comm...@subversion.apache.org Subject: svn commit: r1069176 - in /subversion/branches/integrate- stream-api-extensions/subversion: i

Re: svn commit: r879627 - /subversion/tests/cmdline/switch_tests.py

2011-02-10 Thread Paul Burba
> Author: rhuijben > Date: Thu Sep 24 08:38:55 2009 > New Revision: 879627 > > Add XFail test on how we handle 'svn cp' on directories containing > a switched subdirectory. > > * subversion/tests/cmdline/switch_tests.py > (copy_with_switched_subdir): New test. > (test_list): Add test as XFail > >

Re: Integrating further changes from the performance branch

2011-02-10 Thread Ivan Zhakov
On Thu, Feb 10, 2011 at 03:25, Stefan Fuhrmann wrote: > Hi there, > > Those among you following the commits will have noticed > that I opened 5 new branches each containing a cherry- > picked patch set from the performance branch (details below). > > These branches can be merged independent of eac

Re: Requesting a New Feature

2011-02-10 Thread C. Michael Pilato
On 02/10/2011 02:54 PM, Mark Phippard wrote: > On Thu, Feb 10, 2011 at 2:48 PM, C. Michael Pilato > wrote: >> On 02/10/2011 12:34 PM, Julian Foad wrote: >>> p.s. C-Mike, did I dream it, or did you implement something similar just >>> a few weeks ago? Maybe --parents for another subcommand? >> >>

Beehaviour change between 1.6 and 1.7 when reverting copies

2011-02-10 Thread Philip Martin
There has been a behaviour change reverting copied directories between 1.6 and the current 1.7. In 1.7 the depth of the revert must match the depth of the copy, so if the directory only has files it can be reverted using depth=files, if the directory has empty immediate subdirs it can be reverted

Re: svn commit: r1033709 - /subversion/tests/cmdline/patch_tests.py

2011-02-10 Thread Branko Čibej
On 10.02.2011 20:25, Paul Burba wrote: >> Author: danielsh >> Date: Wed Nov 10 21:02:46 2010 >> New Revision: 1033709 >> >> URL: http://svn.apache.org/viewvc?rev=1033709&view=rev >> Log: >> Add a regression test for 'svn patch' setting properties without >> trailing newlines. This could be useful,

Re: Requesting a New Feature

2011-02-10 Thread Mark Phippard
On Thu, Feb 10, 2011 at 2:48 PM, C. Michael Pilato wrote: > On 02/10/2011 12:34 PM, Julian Foad wrote: >> p.s. C-Mike, did I dream it, or did you implement something similar just >> a few weeks ago?  Maybe --parents for another subcommand? > > Wow.  I mean ... just ... wow. > > /me cries. That is

Re: Requesting a New Feature

2011-02-10 Thread C. Michael Pilato
On 02/10/2011 02:49 PM, Bob Archer wrote: >> On 02/10/2011 12:34 PM, Julian Foad wrote: >>> p.s. C-Mike, did I dream it, or did you implement something >> similar >>> just a few weeks ago? Maybe --parents for another subcommand? >> >> Wow. I mean ... just ... wow. >> >> /me cries. > > Is that a

RE: Requesting a New Feature

2011-02-10 Thread Bob Archer
> On 02/10/2011 12:34 PM, Julian Foad wrote: > > p.s. C-Mike, did I dream it, or did you implement something > similar > > just a few weeks ago? Maybe --parents for another subcommand? > > Wow. I mean ... just ... wow. > > /me cries. Is that a yea or nay? BOb

Re: Requesting a New Feature

2011-02-10 Thread C. Michael Pilato
On 02/10/2011 12:34 PM, Julian Foad wrote: > p.s. C-Mike, did I dream it, or did you implement something similar just > a few weeks ago? Maybe --parents for another subcommand? Wow. I mean ... just ... wow. /me cries. -- C. Michael Pilato CollabNet <> www.collab.net <> Distributed De

Re: s

2011-02-10 Thread C. Michael Pilato
On 02/06/2011 08:18 PM, Daniel Shahaf wrote: > I managed to cause 'svn' to lose my commit message when (for whatever > reason) Vim exited with error code 2 after I'd saved the file: I've read through the code a few times now, and I can't see a good reason not to commit this. (But I'll defer that

Re: svn commit: r1033709 - /subversion/tests/cmdline/patch_tests.py

2011-02-10 Thread Paul Burba
> Author: danielsh > Date: Wed Nov 10 21:02:46 2010 > New Revision: 1033709 > > URL: http://svn.apache.org/viewvc?rev=1033709&view=rev > Log: > Add a regression test for 'svn patch' setting properties without > trailing newlines. This could be useful, for example, to transmit > 'svn:eol-style' pro

RE: Partially recursive revert when node kind changes

2011-02-10 Thread Bob Archer
> Bob Archer writes: > > >> Suppose I have a directory tree: > >> > >> foo/ > >> bar/ > >> zig > >> > >> and I delete the tree foo/bar and replace it by a file foo/bar. > >> What > >> should "svn revert --depth=files foo" do? Should it revert > >> foo/bar? > >> How much of foo/bar

RE: Partially recursive revert when node kind changes

2011-02-10 Thread Bert Huijben
> -Original Message- > From: Bob Archer [mailto:bob.arc...@amsi.com] > Sent: donderdag 10 februari 2011 19:57 > To: Philip Martin > Cc: dev@subversion.apache.org > Subject: RE: Partially recursive revert when node kind changes > > > Bob Archer writes: > > > > >> Suppose I have a directo

RE: Partially recursive revert when node kind changes

2011-02-10 Thread Bob Archer
> Bob Archer writes: > > >> Suppose I have a directory tree: > >> > >> foo/ > >> bar/ > >> zig > >> > >> and I delete the tree foo/bar and replace it by a file foo/bar. > >> What > >> should "svn revert --depth=files foo" do? Should it revert > >> foo/bar? > >> How much of foo/bar

Re: UTF-8 NFC/NFD paths issue

2011-02-10 Thread C. Michael Pilato
On 09/20/2010 10:38 AM, Erik Huelsmann wrote: > To recap, the change I'm proposing is that we check pathnames with NFC/D > aware comparison routines upon add_file() / add_directory() inside > libsvn_repos or libsvn_fs_* - of which I suspect it's easier to handle > inside the latter. In my proposal,

Re: Partially recursive revert when node kind changes

2011-02-10 Thread Philip Martin
Bob Archer writes: >> Suppose I have a directory tree: >> >> foo/ >> bar/ >> zig >> >> and I delete the tree foo/bar and replace it by a file foo/bar. >> What >> should "svn revert --depth=files foo" do? Should it revert >> foo/bar? >> How much of foo/bar should it restore? > >

Re: Partially recursive revert when node kind changes

2011-02-10 Thread Philip Martin
"C. Michael Pilato" writes: > On 02/10/2011 11:28 AM, Philip Martin wrote: >> Suppose I have a directory tree: >> >> foo/ >> bar/ >> zig >> >> and I delete the tree foo/bar and replace it by a file foo/bar. What >> should "svn revert --depth=files foo" do? Should it revert foo/

RE: Partially recursive revert when node kind changes

2011-02-10 Thread Bob Archer
> Suppose I have a directory tree: > > foo/ > bar/ > zig > > and I delete the tree foo/bar and replace it by a file foo/bar. > What > should "svn revert --depth=files foo" do? Should it revert > foo/bar? > How much of foo/bar should it restore? You have done the following: 1. De

Re: Requesting a New Feature

2011-02-10 Thread Stefan Küng
On Thu, Feb 10, 2011 at 18:34, Julian Foad wrote: > On Thu, 2011-02-10, C. Michael Pilato wrote: >> Now tracked as http://subversion.tigris.org/issues/show_bug.cgi?id=3804 >> >> On 07/29/2010 10:14 AM, C. Michael Pilato wrote: >> > Sorta kinda.  Let's use our own tree as an example.  We start with

Re: Requesting a New Feature

2011-02-10 Thread Julian Foad
On Thu, 2011-02-10, C. Michael Pilato wrote: > Now tracked as http://subversion.tigris.org/issues/show_bug.cgi?id=3804 > > On 07/29/2010 10:14 AM, C. Michael Pilato wrote: > > Sorta kinda. Let's use our own tree as an example. We start with an empty > > checkout of the root of our project: > >

Re: Partially recursive revert when node kind changes

2011-02-10 Thread Philip Martin
Branko Čibej writes: > On 10.02.2011 17:43, C. Michael Pilato wrote: >> On 02/10/2011 11:28 AM, Philip Martin wrote: >>> Suppose I have a directory tree: >>> >>> foo/ >>> bar/ >>> zig >>> >>> and I delete the tree foo/bar and replace it by a file foo/bar. What >>> should "svn reve

What should this tree conflict test expect? (was: svn commit: r879162)

2011-02-10 Thread Paul Burba
Hi Neels (or any other tree conflict gurus), I know you don't have much time for Subversion these days, but if you find some time in the near future could you take a look at the tests you added in r879162 and combined in r879206 (this test currently resides at merge_tree_conflict_tests.py 23 'repl

Re: Partially recursive revert when node kind changes

2011-02-10 Thread Stefan Sperling
On Thu, Feb 10, 2011 at 05:47:04PM +0100, Branko Čibej wrote: > On 10.02.2011 17:43, C. Michael Pilato wrote: > > On 02/10/2011 11:28 AM, Philip Martin wrote: > >> Suppose I have a directory tree: > >> > >> foo/ > >> bar/ > >> zig > >> > >> and I delete the tree foo/bar and replace i

Re: Partially recursive revert when node kind changes

2011-02-10 Thread Branko Čibej
On 10.02.2011 17:43, C. Michael Pilato wrote: > On 02/10/2011 11:28 AM, Philip Martin wrote: >> Suppose I have a directory tree: >> >> foo/ >> bar/ >> zig >> >> and I delete the tree foo/bar and replace it by a file foo/bar. What >> should "svn revert --depth=files foo" do? Should

Re: Partially recursive revert when node kind changes

2011-02-10 Thread C. Michael Pilato
On 02/10/2011 11:28 AM, Philip Martin wrote: > Suppose I have a directory tree: > > foo/ > bar/ > zig > > and I delete the tree foo/bar and replace it by a file foo/bar. What > should "svn revert --depth=files foo" do? Should it revert foo/bar? > How much of foo/bar should it res

Re: Requesting a New Feature

2011-02-10 Thread C. Michael Pilato
Now tracked as http://subversion.tigris.org/issues/show_bug.cgi?id=3804 On 07/29/2010 10:14 AM, C. Michael Pilato wrote: > Sorta kinda. Let's use our own tree as an example. We start with an empty > checkout of the root of our project: > > $ svn co --depth empty http://svn.apache.org/repos/as

Partially recursive revert when node kind changes

2011-02-10 Thread Philip Martin
Suppose I have a directory tree: foo/ bar/ zig and I delete the tree foo/bar and replace it by a file foo/bar. What should "svn revert --depth=files foo" do? Should it revert foo/bar? How much of foo/bar should it restore? -- Philip

Re: svn info and svnversion show wrong last committed revision when switching branches.

2011-02-10 Thread Philip Martin
"C. Michael Pilato" writes: > On 02/10/2011 04:58 AM, Philip Martin wrote: >> I think this is fixed on trunk. I get your results with 1.6 and with >> 1.7 I get: >> >> $ svnversion -c svn-wc.12027/ >> 2:5 >> $ svn st -v svn-wc.12027/ >> 65 pm svn-wc.12027 >>

Re: svn info and svnversion show wrong last committed revision when switching branches.

2011-02-10 Thread C. Michael Pilato
On 02/10/2011 04:58 AM, Philip Martin wrote: > I think this is fixed on trunk. I get your results with 1.6 and with > 1.7 I get: > > $ svnversion -c svn-wc.12027/ > 2:5 > $ svn st -v svn-wc.12027/ > 65 pm svn-wc.12027 > 62 pm s

Re: svn commit: r1068977 - in /subversion/trunk/subversion: svn/cat-cmd.c tests/cmdline/cat_tests.py

2011-02-10 Thread C. Michael Pilato
On 02/10/2011 04:52 AM, Bert Huijben wrote: > FAIL: cat_tests.py 1: cat a local directory > FAIL: cat_tests.py 4: cat a nonexistent file > FAIL: cat_tests.py 5: cat should skip uncattable resources > FAIL: cat_tests.py 6: cat an unversioned file parent dir thinks exists > > These tests fail on

Re: [PATCH] Clarify that the issue tracker hasn't moved yet.

2011-02-10 Thread C. Michael Pilato
r1069422, after tweaks to minimize the diff. Thanks, Kevin. On 02/09/2011 09:25 PM, Kevin Benton wrote: > Index: issue-tracker.html > === > --- issue-tracker.html (revision 1069204) > +++ issue-tracker.html (working copy) > @@ -53,

Re: [PATCH] cat tests fails for http:// proto

2011-02-10 Thread Noorul Islam K M
Julian Foad writes: > On Thu, 2011-02-10, Noorul Islam K M wrote: > >> It looks like for file:// and http:// protocols cat prints different >> warning messages for non-existing target. >> >> For http:// >> >> svn: warning: W160013: >> '/svn-test-work/repositories/cat_tests-9/!svn/bc/1/non-exist

Re: [PATCH] cat tests fails for http:// proto

2011-02-10 Thread Julian Foad
On Thu, 2011-02-10, Noorul Islam K M wrote: > It looks like for file:// and http:// protocols cat prints different > warning messages for non-existing target. > > For http:// > > svn: warning: W160013: > '/svn-test-work/repositories/cat_tests-9/!svn/bc/1/non-existing' path > not found > > For fi

Re: svn commit: r1068695 - in /subversion/branches/integrate-stream-api-extensions: ./ subversion/include/svn_io.h subversion/libsvn_subr/stream.c subversion/libsvn_subr/subst.c

2011-02-10 Thread Hyrum K Wright
On Thu, Feb 10, 2011 at 1:02 AM, Stefan Fuhrmann wrote: > On 09.02.2011 02:07, Blair Zajac wrote: ... >> We have systems with APR 1.2 and it doesn't provide >> apr_file_buffer_size_get(). > > Is there anything in the 1.3 headers that would indicate > in which version a specific function got introd

Re: svn commit: r1068029 - /subversion/trunk/subversion/libsvn_subr/io.c

2011-02-10 Thread Stefan Küng
On Thu, Feb 10, 2011 at 10:47, Bert Huijben wrote: > > > > -Original Message- > > From: stevek...@apache.org [mailto:stevek...@apache.org] > > Sent: maandag 7 februari 2011 18:43 > > To: comm...@subversion.apache.org > > Subject: svn commit: r1068029 - > > /subversion/trunk/subversion/libs

Re: [PATCH] svn command - ls - Multiple targets

2011-02-10 Thread Noorul Islam K M
Noorul Islam K M writes: > Noorul Islam K M writes: > >> This patch is a followup of the following thread. All tests pass with >> this patch. >> >> http://svn.haxx.se/dev/archive-2011-01/0210.shtml >> >> Log >> [[[ >> >> Make 'svn ls' continue processing targets after printing warning if one >>

RE: svn commit: r1068863 - /subversion/trunk/build/generator/gen_msvc_dsp.py

2011-02-10 Thread Bert Huijben
> -Original Message- > From: Joe Swatosh [mailto:joe.swat...@gmail.com] > Sent: woensdag 9 februari 2011 23:43 > To: dev@subversion.apache.org > Cc: rhuij...@apache.org > Subject: Re: svn commit: r1068863 - > /subversion/trunk/build/generator/gen_msvc_dsp.py > > On Wed, Feb 9, 2011 at 4:32

Re: [PATCH] svn command - ls - Multiple targets

2011-02-10 Thread Noorul Islam K M
Noorul Islam K M writes: > This patch is a followup of the following thread. All tests pass with > this patch. > > http://svn.haxx.se/dev/archive-2011-01/0210.shtml > > Log > [[[ > > Make 'svn ls' continue processing targets after printing warning if one > or more of the targets is a non-existent

Re: svn commit: r1068977 - in /subversion/trunk/subversion: svn/cat-cmd.c tests/cmdline/cat_tests.py

2011-02-10 Thread Noorul Islam K M
"Bert Huijben" writes: >> -Original Message- >> From: cmpil...@apache.org [mailto:cmpil...@apache.org] >> Sent: woensdag 9 februari 2011 18:07 >> To: comm...@subversion.apache.org >> Subject: svn commit: r1068977 - in /subversion/trunk/subversion: >> svn/cat-cmd.c tests/cmdline/cat_tests.

Re: svn commit: r1034362 - in /subversion/trunk/build: generator/ generator/templates/ win32/

2011-02-10 Thread Julian Foad
On Tue, 2011-02-08, Joe Swatosh wrote: > On Fri, Nov 12, 2010 at 4:09 AM, wrote: > > Log: > > Move projects generated by the Windows build to the right locations. The > > necessary changes in the python scripting to handle this also allowed moving > > the remaining project templates .ezt files to

Re: svn info and svnversion show wrong last committed revision when switching branches.

2011-02-10 Thread Bert Wesarg
On Thu, Feb 10, 2011 at 10:58, Philip Martin wrote: > Bert Wesarg writes: > >> svn switch ^/branches/branch >> svn up >> svn merge ^/trunk >> svn ci -m "merging" >> svn up >> svnversion >> # 6 >> svnversion -c >> # 4:6 >> svn info >> # Revision: 6 >> # Last Changed Rev: 6 >> svn switch ^/trunk >>

RE: svn commit: r1068977 - in /subversion/trunk/subversion: svn/cat-cmd.c tests/cmdline/cat_tests.py

2011-02-10 Thread Bert Huijben
> -Original Message- > From: cmpil...@apache.org [mailto:cmpil...@apache.org] > Sent: woensdag 9 februari 2011 18:07 > To: comm...@subversion.apache.org > Subject: svn commit: r1068977 - in /subversion/trunk/subversion: > svn/cat-cmd.c tests/cmdline/cat_tests.py > > Author: cmpilato > Da

RE: svn commit: r1069176 - in /subversion/branches/integrate-stream-api-extensions/subversion: include/svn_io.h libsvn_subr/stream.c libsvn_subr/subst.c

2011-02-10 Thread Bert Huijben
> -Original Message- > From: stef...@apache.org [mailto:stef...@apache.org] > Sent: donderdag 10 februari 2011 1:36 > To: comm...@subversion.apache.org > Subject: svn commit: r1069176 - in /subversion/branches/integrate- > stream-api-extensions/subversion: include/svn_io.h libsvn_subr/str

Re: svn info and svnversion show wrong last committed revision when switching branches.

2011-02-10 Thread Philip Martin
Bert Wesarg writes: > svn switch ^/branches/branch > svn up > svn merge ^/trunk > svn ci -m "merging" > svn up > svnversion > # 6 > svnversion -c > # 4:6 > svn info > # Revision: 6 > # Last Changed Rev: 6 > svn switch ^/trunk > svn up > > # here it starts to show the wrong revision. > > svnversio

RE: svn commit: r1068029 - /subversion/trunk/subversion/libsvn_subr/io.c

2011-02-10 Thread Bert Huijben
> -Original Message- > From: stevek...@apache.org [mailto:stevek...@apache.org] > Sent: maandag 7 februari 2011 18:43 > To: comm...@subversion.apache.org > Subject: svn commit: r1068029 - > /subversion/trunk/subversion/libsvn_subr/io.c > svn_error_t * > svn_io_set_file_read_write_car

[PATCH] svn command - ls - Multiple targets

2011-02-10 Thread Noorul Islam K M
This patch is a followup of the following thread. All tests pass with this patch. http://svn.haxx.se/dev/archive-2011-01/0210.shtml Log [[[ Make 'svn ls' continue processing targets after printing warning if one or more of the targets is a non-existent URL or wc-entry. Also return a non-zero er