Re: Real problem is r88099

2010-01-26 Thread Philip Martin
Philip Martin writes: > ==10949== Invalid read of size 8 > ==10949==at 0x5991011: range_contains (mergeinfo.c:780) > ==10949==by 0x59913F3: rangelist_intersect_or_remove (mergeinfo.c:912) > ==10949==by 0x5991903: svn_rangelist_intersect (mergeinfo.c:1043) > ==10949==by 0x4E76FF3:

Re: svn commit: r902303 - in /subversion/trunk/subversion: include/private/svn_wc_private.h libsvn_wc/lock.c

2010-01-26 Thread Philip Martin
s...@apache.org writes: > Author: stsp > Date: Fri Jan 22 22:50:42 2010 > New Revision: 902303 > +svn_error_t * > +svn_wc__call_with_write_lock(svn_wc__with_write_lock_func_t func, > + void *baton, > + svn_wc_context_t *wc_ctx, > +

Re: Real problem is r88099

2010-01-26 Thread Philip Martin
Philip Martin writes: > ==13758== Invalid read of size 4 > ==13758==at 0x50FA05B: close_directory (update_editor.c:3166) > ==13758==by 0x50B6D6E: close_directory (ambient_depth_filter_editor.c:567) > ==13758==by 0x5541477: close_directory (cancel.c:261) > ==13758==by 0x5541477: cl

Real problem is r88099

2010-01-26 Thread Philip Martin
Philip Martin writes: > s...@apache.org writes: > >> Author: stsp >> Date: Tue Jan 26 15:46:59 2010 >> New Revision: 903284 >> >> URL: http://svn.apache.org/viewvc?rev=903284&view=rev >> Log: >> Make svn patch provide notifications about individual hunks. >> Make the CLI client print them. > > I

Re: svn_wc__db_base_get_info() vs. svn_wc__db_read_info() ?

2010-01-26 Thread Greg Stein
base_get_info returns information about the BASE tree. read_info returns information about the WORKING tree, or if a specified node is not present in WORKING, then info from the BASE tree. and recall that BASE == what you checked out from the repository. WORKING corresponds to added/removed/copie

Re: svn commit: r903507 - /subversion/trunk/tools/dev/unix-build/Makefile.svn

2010-01-26 Thread Hyrum K. Wright
On Jan 26, 2010, at 4:04 PM, ne...@apache.org wrote: > Author: neels > Date: Wed Jan 27 00:04:16 2010 > New Revision: 903507 > > URL: http://svn.apache.org/viewvc?rev=903507&view=rev > Log: > * tools/dev/unix-build/Makefile.svn: Add missing line ending chars. I'm curious: why do we need to make

svn_wc__db_base_get_info() vs. svn_wc__db_read_info() ?

2010-01-26 Thread Neels J Hofmeyr
How does err = svn_wc__db_base_get_info(&status, NULL, &revision, &repos_relpath, &repos_root_url, NULL, NULL, NULL, NULL, NULL, NULL,

Re: svn merge - repeat merge is not a no-op

2010-01-26 Thread Paul Burba
On Mon, Jan 25, 2010 at 12:45 PM, Paul Burba wrote: > On Mon, Jan 25, 2010 at 9:32 AM, Julian Foad wrote: >> After merging all changes from a branch into the WC, a second attempt of >> the same merge brings in a different file. >> >> Full Details >> >> I tried a merge in a clean WC of the branch

Re: svn commit: r892189 - in /subversion/branches/1.6.x-r891672/subversion: libsvn_client/commit_util.c tests/cmdline/externals_tests.py

2010-01-26 Thread Paul Burba
On Mon, Jan 25, 2010 at 7:16 PM, Senthil Kumaran S wrote: > Hi Paul, > > Paul Burba wrote: >> >> On Fri, Dec 18, 2009 at 4:16 AM,   wrote: >>> >>> Author: stylesen >>> Date: Fri Dec 18 09:16:18 2009 >>> New Revision: 892189 >>> >>> URL: http://svn.apache.org/viewvc?rev=892189&view=rev >>> Log: >>>

Re: svn commit: r903284 - in /subversion/trunk/subversion: include/svn_wc.h libsvn_client/patch.c svn/notify.c tests/cmdline/patch_tests.py

2010-01-26 Thread Philip Martin
s...@apache.org writes: > Author: stsp > Date: Tue Jan 26 15:46:59 2010 > New Revision: 903284 > > URL: http://svn.apache.org/viewvc?rev=903284&view=rev > Log: > Make svn patch provide notifications about individual hunks. > Make the CLI client print them. I don't know why yet, but this patch is

Re: JavaHL package name? (was: Discussion: graduating Subversion)

2010-01-26 Thread Martin Furter
On Mon, 25 Jan 2010, Justin Erenkrantz wrote: My take is deprecate the old package names (org.tigris.*), add the new package name (org.apache.subversion.*), and remove the deprecated names whenever we go to 2.0. I'm moderately ambivalent on whether this happens for 1.7 or if can wait for 1.8.

Re: svn commit: r903342 - /subversion/trunk/subversion/libsvn_diff/parse-diff.c

2010-01-26 Thread Stefan Sperling
On Tue, Jan 26, 2010 at 11:23:18AM -0800, Blair Zajac wrote: > On 01/26/2010 10:32 AM, Stefan Sperling wrote: > >I've been told some time ago, by long-time unix hackers, > >that clearning errno is usually not necessary, one of the few > >exceptions to this rule being the strtol() family of function

Re: svn commit: r903342 - /subversion/trunk/subversion/libsvn_diff/parse-diff.c

2010-01-26 Thread Blair Zajac
On 01/26/2010 10:32 AM, Stefan Sperling wrote: On Tue, Jan 26, 2010 at 10:02:02AM -0800, Blair Zajac wrote: On 01/26/2010 09:54 AM, s...@apache.org wrote: Author: stsp Date: Tue Jan 26 17:54:14 2010 New Revision: 903342 URL: http://svn.apache.org/viewvc?rev=903342&view=rev Log: * subversion/li

Re: svn commit: r903342 - /subversion/trunk/subversion/libsvn_diff/parse-diff.c

2010-01-26 Thread Stefan Sperling
On Tue, Jan 26, 2010 at 10:02:02AM -0800, Blair Zajac wrote: > On 01/26/2010 09:54 AM, s...@apache.org wrote: > >Author: stsp > >Date: Tue Jan 26 17:54:14 2010 > >New Revision: 903342 > > > >URL: http://svn.apache.org/viewvc?rev=903342&view=rev > >Log: > >* subversion/libsvn_diff/parse-diff.c > >

Re: svn commit: r903342 - /subversion/trunk/subversion/libsvn_diff/parse-diff.c

2010-01-26 Thread Blair Zajac
On 01/26/2010 09:54 AM, s...@apache.org wrote: Author: stsp Date: Tue Jan 26 17:54:14 2010 New Revision: 903342 URL: http://svn.apache.org/viewvc?rev=903342&view=rev Log: * subversion/libsvn_diff/parse-diff.c (parse_offset): Clearing errno is never necessary, so don't. Why are you claiming

Re: svn commit: r903006 - in /subversion/site/publish: getting.html packages.html

2010-01-26 Thread Mark Phippard
On Tue, Jan 26, 2010 at 12:00 PM, Mark Phippard wrote: > On Tue, Jan 26, 2010 at 11:23 AM, Bert Huijben wrote: > >> Maybe the Windows packages need a more prominent location instead of 5 pages >> down >> on a more than average screensize. >> I don't think the Aix packages which are the first on

Re: svn commit: r903006 - in /subversion/site/publish: getting.html packages.html

2010-01-26 Thread Mark Phippard
On Tue, Jan 26, 2010 at 11:23 AM, Bert Huijben wrote: > Maybe the Windows packages need a more prominent location instead of 5 pages > down > on a more than average screensize. > I don't think the Aix packages which are the first on screen, come anything > near the > number of downloads of the

Re: svn commit: r903006 - in /subversion/site/publish: getting.html packages.html

2010-01-26 Thread C. Michael Pilato
Yeah, I made similar murmurings the other day. In general, it seems we should favor the OSes for whom Subversion doesn't come via a semi-automated package management system. Bert Huijben wrote: > >> -Original Message- >> From: cmpil...@apache.org [mailto:cmpil...@apache.org] >> Sent: maa

Re: New website -- what do we lack?

2010-01-26 Thread Julian Foad
On Fri, 2010-01-22, C. Michael Pilato wrote: > Julian Foad wrote: > > I've been thinking about how to make entry easy for both newcomers and > > regulars. Here's the resulting ideas. > > > > > > * On the main page: > > > > Let's tell the visitor... > > > > - what the project is (and what it i

RE: svn commit: r903006 - in /subversion/site/publish: getting.html packages.html

2010-01-26 Thread Bert Huijben
> -Original Message- > From: cmpil...@apache.org [mailto:cmpil...@apache.org] > Sent: maandag 25 januari 2010 23:57 > To: comm...@subversion.apache.org > Subject: svn commit: r903006 - in /subversion/site/publish: getting.html > packages.html > > Author: cmpilato > Date: Mon Jan 25 22:57

Re: svn commit: r903006 - in /subversion/site/publish: getting.html packages.html

2010-01-26 Thread C. Michael Pilato
Mark Phippard wrote: > On Mon, Jan 25, 2010 at 5:57 PM, wrote: >> Author: cmpilato >> Date: Mon Jan 25 22:57:07 2010 >> New Revision: 903006 >> >> URL: http://svn.apache.org/viewvc?rev=903006&view=rev >> Log: >> Move binary package listing to their own page. >> >> * site/publish/packages.html >>

Re: svn commit: r903006 - in /subversion/site/publish: getting.html packages.html

2010-01-26 Thread Mark Phippard
On Mon, Jan 25, 2010 at 5:57 PM, wrote: > Author: cmpilato > Date: Mon Jan 25 22:57:07 2010 > New Revision: 903006 > > URL: http://svn.apache.org/viewvc?rev=903006&view=rev > Log: > Move binary package listing to their own page. > > * site/publish/packages.html >  Fork from "getting.html", heavil

Re: svn commit: r903006 - in /subversion/site/publish: getting.html packages.html

2010-01-26 Thread Ivan Zhakov
On Tue, Jan 26, 2010 at 1:57 AM, wrote: > Author: cmpilato > Date: Mon Jan 25 22:57:07 2010 > New Revision: 903006 > > URL: http://svn.apache.org/viewvc?rev=903006&view=rev > Log: > Move binary package listing to their own page. > > * site/publish/packages.html >  Fork from "getting.html", heavil

Re: svn commit: r902886 - /subversion/trunk/subversion/tests/cmdline/copy_tests.py

2010-01-26 Thread Daniel Shahaf
Julian Foad wrote on Tue, 26 Jan 2010 at 09:35 -: > I tested the behaviour and confirmed that the specified pattern had to > match a whole line, so it would not be a "match anything" pattern. Thanks for the confirmation. (I checked myself whether it should be a whole/partial line match, but t

Re: Probable bug with svn copy

2010-01-26 Thread Noorul Islam K M
Julian Foad writes: > Paul Burba wrote: >> On Mon, Jan 25, 2010 at 2:17 AM, Noorul Islam K M wrote: >> > + # Move new added file to another one and commit. >> > + second_path = os.path.join(new_path, 'second') >> > + rav_svn(None, None, [], 'move', first_path, second_path) >> > + rav_svn(Non

Re: davautocheck - "(98)Address already in use: make_sock: could not bind to address 127.0.0.1:xxxx"

2010-01-26 Thread Julian Foad
On Fri, 2010-01-15, Julian Foad wrote: > On Fri, 2010-01-15 at 12:22 +, Philip Martin wrote: > > Julian Foad writes: > > > I'm unable to run DAV tests this week. I'm getting: > > > > > > [[[ > > > (98)Address already in use: make_sock: could not bind to address > > > 127.0.0.1:3961 > > > no l

Re: svn commit: r903153 - /subversion/trunk/subversion/libsvn_ra_neon/util.c

2010-01-26 Thread Greg Stein
On Tue, Jan 26, 2010 at 07:13, Ivan Zhakov wrote: > On Tue, Jan 26, 2010 at 3:01 PM, Greg Stein wrote: >> On Tue, Jan 26, 2010 at 06:59, Ivan Zhakov wrote: >>> On Tue, Jan 26, 2010 at 2:41 PM, Greg Stein wrote: On Tue, Jan 26, 2010 at 05:12,   wrote: > Author: ivan > Date: Tue Jan

Re: svn commit: r903153 - /subversion/trunk/subversion/libsvn_ra_neon/util.c

2010-01-26 Thread Ivan Zhakov
On Tue, Jan 26, 2010 at 3:01 PM, Greg Stein wrote: > On Tue, Jan 26, 2010 at 06:59, Ivan Zhakov wrote: >> On Tue, Jan 26, 2010 at 2:41 PM, Greg Stein wrote: >>> On Tue, Jan 26, 2010 at 05:12,   wrote: Author: ivan Date: Tue Jan 26 10:12:05 2010 New Revision: 903153 URL:

Re: svn commit: r903153 - /subversion/trunk/subversion/libsvn_ra_neon/util.c

2010-01-26 Thread Greg Stein
On Tue, Jan 26, 2010 at 06:59, Ivan Zhakov wrote: > On Tue, Jan 26, 2010 at 2:41 PM, Greg Stein wrote: >> On Tue, Jan 26, 2010 at 05:12,   wrote: >>> Author: ivan >>> Date: Tue Jan 26 10:12:05 2010 >>> New Revision: 903153 >>> >>> URL: http://svn.apache.org/viewvc?rev=903153&view=rev >>> Log: >>>

Re: svn commit: r903153 - /subversion/trunk/subversion/libsvn_ra_neon/util.c

2010-01-26 Thread Ivan Zhakov
On Tue, Jan 26, 2010 at 2:41 PM, Greg Stein wrote: > On Tue, Jan 26, 2010 at 05:12,   wrote: >> Author: ivan >> Date: Tue Jan 26 10:12:05 2010 >> New Revision: 903153 >> >> URL: http://svn.apache.org/viewvc?rev=903153&view=rev >> Log: >> Follow up to r902836. >> >> * subversion/libsvn_ra_neon/util

Re: svn commit: r903153 - /subversion/trunk/subversion/libsvn_ra_neon/util.c

2010-01-26 Thread Greg Stein
On Tue, Jan 26, 2010 at 05:12, wrote: > Author: ivan > Date: Tue Jan 26 10:12:05 2010 > New Revision: 903153 > > URL: http://svn.apache.org/viewvc?rev=903153&view=rev > Log: > Follow up to r902836. > > * subversion/libsvn_ra_neon/util.c >  (wrapper_reader_cb): Use SVN_RA_NEON__REQ_ERR macro inste

Re: Multiple matching lines in authz file

2010-01-26 Thread Eric Lee
On Fri, Jan 15, 2010 at 6:16 AM, Philip Martin wrote: > What should happen when a username matches multiple lines in an authz > file? The svnbook states that the first matching line applies: > > [groups] > paint-developers = frank, sally, jane > > [paint:/projects/paint] > jane = r > @p

Re: [PATCH] Fix the awful "200 OK" error message when can't connect to repository

2010-01-26 Thread Ivan Zhakov
On Mon, Jan 25, 2010 at 6:21 PM, Julian Foad wrote: > On Fri, 2010-01-22, Ivan Zhakov wrote: >> On Wed, Jan 13, 2010 at 6:04 PM, Julian Foad >> wrote: >> > On Mon, 2010-01-11, Ivan Zhakov wrote: >> > Your new patch now has a more descriptive error message, that is the >> > same as the one produc

Re: Multiple matching lines in authz file

2010-01-26 Thread Stefan Sperling
On Tue, Jan 26, 2010 at 11:36:13AM +0200, Mark Cammidge wrote: > Danil Shopyrin wrote: > > >> Do they? Perhaps they write them by trial-and-error, testing to see > >> if they get the behaviour they want. In that case we will break authz > >> files that work. > > > > Agree on this. From our expe

Re: Multiple matching lines in authz file

2010-01-26 Thread Mark Cammidge
Danil Shopyrin wrote: >> Do they? Perhaps they write them by trial-and-error, testing to see >> if they get the behaviour they want. In that case we will break authz >> files that work. > > Agree on this. From our experience, people configure access > permissions by trial-and-error approach. At

Re: svn commit: r902886 - /subversion/trunk/subversion/tests/cmdline/copy_tests.py

2010-01-26 Thread Julian Foad
Daniel Shahaf wrote: > julianf...@apache.org wrote on Mon, 25 Jan 2010 at 17:29 -: > > Fix the expected output of an XFail test added in r902841, so that when the > > bug is fixed the test will have at least a chance of passing. > > > > + rav_svn(None, "Adding.*New|Adding.*first||Committed re