Re: svn commit: r1382764 - /subversion/trunk/subversion/libsvn_subr/opt.c

2012-09-10 Thread C. Michael Pilato
Did this just break getopt_tests.py 2? On 09/10/2012 07:37 AM, br...@apache.org wrote: > Author: brane > Date: Mon Sep 10 11:37:33 2012 > New Revision: 1382764 > > URL: http://svn.apache.org/viewvc?rev=1382764&view=rev > Log: > * subversion/libsvn_subr/opt.c (svn_opt__get_version_info): > Refor

Re: svn commit: r1382764 - /subversion/trunk/subversion/libsvn_subr/opt.c

2012-09-10 Thread Branko Čibej
It might have. I'm still running the tests. -- Brane On 10.09.2012 13:56, C. Michael Pilato wrote: > Did this just break getopt_tests.py 2? > > On 09/10/2012 07:37 AM, br...@apache.org wrote: >> Author: brane >> Date: Mon Sep 10 11:37:33 2012 >> New Revision: 1382764 >> >> URL: http://svn.apache.

Re: svn commit: r1382764 - /subversion/trunk/subversion/libsvn_subr/opt.c

2012-09-10 Thread C. Michael Pilato
(Answering my own question...) Yes, it did. I fixed the test expectations for you: Sendinggetopt_tests_data/svn--version_stdout Transmitting file data . Committed revision 1382782. On 09/10/2012 07:56 AM, C. Michael Pilato wrote: > Did this just break getopt_tests.py 2? > > On

Re: Please vote for 1.6.x backports this week!

2012-09-10 Thread Stefan Sperling
On Mon, Sep 03, 2012 at 11:24:37AM +0200, Stefan Sperling wrote: > As discussed in a different thread, I'd like to run the release > process for 1.6.19. > > There are a handful of nominations in 1.6.x STATUS and it would be > nice to get as many as possible included in the next (possibly final?) >

Re: svn commit: r1382764 - /subversion/trunk/subversion/libsvn_subr/opt.c

2012-09-10 Thread Branko Čibej
On 10.09.2012 14:04, C. Michael Pilato wrote: > (Answering my own question...) Yes, it did. I fixed the test expectations > for you: > >Sendinggetopt_tests_data/svn--version_stdout >Transmitting file data . >Committed revision 1382782. Thanks! I made the same change locally, b

Re: Please vote for 1.6.x backports this week!

2012-09-10 Thread C. Michael Pilato
On 09/10/2012 08:26 AM, Stefan Sperling wrote: > On Mon, Sep 03, 2012 at 11:24:37AM +0200, Stefan Sperling wrote: >> As discussed in a different thread, I'd like to run the release >> process for 1.6.19. >> >> There are a handful of nominations in 1.6.x STATUS and it would be >> nice to get as many

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

2012-09-10 Thread Daniel Shahaf
cmpil...@apache.org wrote on Mon, Sep 10, 2012 at 12:52:32 -: > Author: cmpilato > Date: Mon Sep 10 12:52:32 2012 > New Revision: 1382806 > > URL: http://svn.apache.org/viewvc?rev=1382806&view=rev > Log: > On the '1.6.x' branch: > > * STATUS > (r1338810 group): Add r1339154, resolving danie

Re: Please vote for 1.6.x backports this week!

2012-09-10 Thread Stefan Sperling
On Mon, Sep 10, 2012 at 08:56:11AM -0400, C. Michael Pilato wrote: > Sorry, Stefan. I'd intended to cruise through the backport suggestions last > week, but got my attention stuck on other tasks. I've just added the > necessary votes to backport three more changes. All that remains are > proposa

Re: Please vote for 1.6.x backports this week!

2012-09-10 Thread Stefan Sperling
On Mon, Sep 10, 2012 at 03:29:47PM +0200, Stefan Sperling wrote: > Thanks! > > I believe we're in good shape now for rolling the release :) One issue is that the 1.6.x test suite won't run properly with APR-1.4.6 :( I'm willing to wade through related commits backported to 1.7.x and create a cor

Re: svn commit: r1382864 - /subversion/trunk/tools/server-side/svnpubsub/svnwcsub.py

2012-09-10 Thread Greg Stein
On Sep 10, 2012 10:19 AM, wrote: > > Author: danielsh > Date: Mon Sep 10 14:18:51 2012 > New Revision: 1382864 > > URL: http://svn.apache.org/viewvc?rev=1382864&view=rev > Log: > [in tools/server-side/svnpubsub] > > Fix a bogus logging.critical(). > > Found by: joes > > * svnwcsub.py > (Backgrou

Re: Please vote for 1.6.x backports this week!

2012-09-10 Thread Philip Martin
Stefan Sperling writes: > One issue is that the 1.6.x test suite won't run properly > with APR-1.4.6 :( > > I'm willing to wade through related commits backported to 1.7.x > and create a corresponding 1.6.x backport branch. > This is a one-time effort, after all. I guess you are referring to the

Re: svn commit: r1381814 - /subversion/trunk/subversion/libsvn_fs_fs/fs_fs.c

2012-09-10 Thread Daniel Shahaf
stef...@apache.org wrote on Fri, Sep 07, 2012 at 00:14:28 -: > Author: stefan2 > Date: Fri Sep 7 00:14:28 2012 > New Revision: 1381814 > > URL: http://svn.apache.org/viewvc?rev=1381814&view=rev > Log: > Fix a performance issue created by directory deltification: rep streams > don't support ma

Re: svn commit: r1381817 - /subversion/trunk/subversion/libsvn_fs_fs/fs_fs.c

2012-09-10 Thread Daniel Shahaf
stef...@apache.org wrote on Fri, Sep 07, 2012 at 00:22:46 -: > Author: stefan2 > Date: Fri Sep 7 00:22:45 2012 > New Revision: 1381817 > > URL: http://svn.apache.org/viewvc?rev=1381817&view=rev > Log: > Follow-up to r1381814. > > * subversion/libsvn_fs_fs/fs_fs.c > (get_dir_contents): expa

Re: svn commit: r1382304 - /subversion/trunk/subversion/libsvn_fs_fs/fs_fs.c

2012-09-10 Thread Daniel Shahaf
stef...@apache.org wrote on Sat, Sep 08, 2012 at 13:51:39 -: > Author: stefan2 > Date: Sat Sep 8 13:51:39 2012 > New Revision: 1382304 > > URL: http://svn.apache.org/viewvc?rev=1382304&view=rev > Log: > Fix a minor issue with props deltification: Deltify against a props > representation inste

Re: svn commit: r1382990 - in /subversion/trunk/subversion: include/ libsvn_subr/ tests/cmdline/ tests/cmdline/getopt_tests_data/

2012-09-10 Thread Ivan Zhakov
On Mon, Sep 10, 2012 at 9:19 PM, wrote: > Author: brane > Date: Mon Sep 10 17:19:31 2012 > New Revision: 1382990 > > URL: http://svn.apache.org/viewvc?rev=1382990&view=rev > Log: > Make the extended version info retrieval a public API. > [...] > + * > + * @since New in 1.8. > + */ > +typedef stru

Re: svn commit: r1382990 - in /subversion/trunk/subversion: include/ libsvn_subr/ tests/cmdline/ tests/cmdline/getopt_tests_data/

2012-09-10 Thread Branko Čibej
On 10.09.2012 21:52, Ivan Zhakov wrote: > On Mon, Sep 10, 2012 at 9:19 PM, wrote: >> Author: brane >> Date: Mon Sep 10 17:19:31 2012 >> New Revision: 1382990 >> >> URL: http://svn.apache.org/viewvc?rev=1382990&view=rev >> Log: >> Make the extended version info retrieval a public API. >> > [...] >

Re: svn commit: r1381257 - sysinfo.c -

2012-09-10 Thread Julian Foad
Brane wrote: [...] > +#if LINUX > +static const char *linux_release_name(apr_pool_t *pool); > +#endif [...] On my Ubuntu GNU/Linux system: subversion/libsvn_subr/sysinfo.c:92:5: "LINUX" is not defined subversion/libsvn_subr/sysinfo.c:116:7: "LINUX" is not defined subversion/libsvn_subr/sysinfo.c

Re: svn commit: r1381257 - sysinfo.c -

2012-09-10 Thread Branko Čibej
On 10.09.2012 22:33, Julian Foad wrote: > Brane wrote: > [...] > >> +#if LINUX >> +static const char *linux_release_name(apr_pool_t *pool); >> +#endif > [...] > > On my Ubuntu GNU/Linux system: > > subversion/libsvn_subr/sysinfo.c:92:5: "LINUX" is not defined > subversion/libsvn_subr/sysinfo.c:116:

RE: svn commit: r1381257 - sysinfo.c -

2012-09-10 Thread Bert Huijben
> -Original Message- > From: Branko Čibej [mailto:br...@wandisco.com] > Sent: maandag 10 september 2012 22:48 > To: dev@subversion.apache.org > Subject: Re: svn commit: r1381257 - sysinfo.c - > > On 10.09.2012 22:33, Julian Foad wrote: > > Brane wrote: > > [...] > > > >> +#if LINUX > >>

Re: svn commit: r1381257 - sysinfo.c -

2012-09-10 Thread Julian Foad
> On 10.09.2012 22:33, Julian Foad wrote: >> Brane wrote: >> [...] >> >>> +#if LINUX >>> +static const char *linux_release_name(apr_pool_t *pool); >>> +#endif >> [...] >> >> On my Ubuntu GNU/Linux system: >> >> subversion/libsvn_subr/sysinfo.c:92:5: "LINUX" is not defined >> subversion

Re: svn commit: r1381257 - sysinfo.c -

2012-09-10 Thread Daniel Shahaf
Julian Foad wrote on Mon, Sep 10, 2012 at 22:16:26 +0100: > > On 10.09.2012 22:33, Julian Foad wrote: > > >> Brane wrote: > >> [...] > >> > >>> +#if LINUX > >>> +static const char *linux_release_name(apr_pool_t *pool); > >>> +#endif > >> [...] > >> > >> On my Ubuntu GNU/Linux system: > >>

Re: svn commit: r1382990 - in /subversion/trunk/subversion: include/ libsvn_subr/ tests/cmdline/ tests/cmdline/getopt_tests_data/

2012-09-10 Thread Daniel Shahaf
br...@apache.org wrote on Mon, Sep 10, 2012 at 17:19:32 -: > Author: brane > Date: Mon Sep 10 17:19:31 2012 > New Revision: 1382990 > > URL: http://svn.apache.org/viewvc?rev=1382990&view=rev > Log: > Make the extended version info retrieval a public API. > > +/** > + * Extended version infoma

Re: svn commit: r1381257 - sysinfo.c -

2012-09-10 Thread Philip Martin
Daniel Shahaf writes: > What about __LINUX__ ? $ echo | gcc -dM -E - | grep -i linux #define __linux 1 #define __linux__ 1 #define __gnu_linux__ 1 #define linux 1 -- Certified & Supported Apache Subversion Downloads: http://www.wandisco.com/subversion/download

Re: svn commit: r1381257 - sysinfo.c -

2012-09-10 Thread Branko Čibej
On 10.09.2012 23:01, Bert Huijben wrote: > I assume you already noticed, but all the buildbots are red after your > latest version info output tweaks. Eh. Bug in the test that didn't show up on my box. Fixed in r1383123, I think. -- Brane -- Certified & Supported Apache Subversion Downloads: h

Re: svn commit: r1381257 - sysinfo.c -

2012-09-10 Thread Branko Čibej
On 10.09.2012 23:16, Julian Foad wrote: >> On 10.09.2012 22:33, Julian Foad wrote: >>> Brane wrote: >>> [...] >>> +#if LINUX +static const char *linux_release_name(apr_pool_t *pool); +#endif >>> [...] >>> >>> On my Ubuntu GNU/Linux system: >>> >>> subversion/libsvn_subr/sysin

Re: svn commit: r1382990 - in /subversion/trunk/subversion: include/ libsvn_subr/ tests/cmdline/ tests/cmdline/getopt_tests_data/

2012-09-10 Thread Branko Čibej
On 10.09.2012 23:49, Daniel Shahaf wrote: > br...@apache.org wrote on Mon, Sep 10, 2012 at 17:19:32 -: >> Author: brane >> Date: Mon Sep 10 17:19:31 2012 >> New Revision: 1382990 >> >> URL: http://svn.apache.org/viewvc?rev=1382990&view=rev >> Log: >> Make the extended version info retrieval a p

Re: svn commit: r1382990 - in /subversion/trunk/subversion: include/ libsvn_subr/ tests/cmdline/ tests/cmdline/getopt_tests_data/

2012-09-10 Thread Daniel Shahaf
Branko Čibej wrote on Tue, Sep 11, 2012 at 00:12:11 +0200: > On 10.09.2012 23:49, Daniel Shahaf wrote: > >> + /** > >> + * Array of svn_version_loaded_lib_t describing loaded shared libraries. > >> + */ > >> + const apr_array_header_t *loaded_libs; > > Please document the type ('const char *'

Re: svn commit: r1382304 - /subversion/trunk/subversion/libsvn_fs_fs/fs_fs.c

2012-09-10 Thread Stefan Fuhrmann
On Mon, Sep 10, 2012 at 8:10 PM, Daniel Shahaf wrote: > stef...@apache.org wrote on Sat, Sep 08, 2012 at 13:51:39 -: > > Author: stefan2 > > Date: Sat Sep 8 13:51:39 2012 > > New Revision: 1382304 > > > > URL: http://svn.apache.org/viewvc?rev=1382304&view=rev > > Log: > > Fix a minor issue w

Re: svn commit: r1381817 - /subversion/trunk/subversion/libsvn_fs_fs/fs_fs.c

2012-09-10 Thread Stefan Fuhrmann
On Mon, Sep 10, 2012 at 8:04 PM, Daniel Shahaf wrote: > stef...@apache.org wrote on Fri, Sep 07, 2012 at 00:22:46 -: > > Author: stefan2 > > Date: Fri Sep 7 00:22:45 2012 > > New Revision: 1381817 > > > > URL: http://svn.apache.org/viewvc?rev=1381817&view=rev > > Log: > > Follow-up to r13818

Re: svn commit: r1381808 - in /subversion/trunk/subversion/libsvn_fs_fs: fs_fs.c fs_fs.h

2012-09-10 Thread Stefan Fuhrmann
On Sat, Sep 8, 2012 at 7:41 PM, Daniel Shahaf wrote: > Stefan Fuhrmann wrote on Sat, Sep 08, 2012 at 01:32:54 +0200: > > On Fri, Sep 7, 2012 at 9:36 PM, Daniel Shahaf wrote: > > > > > I think this fix is wrong. svn_fs_fs__read_noderev() should return the > > > parsed noderev as is --- if it's a

FSFS pack file reorganization tool

2012-09-10 Thread Stefan Fuhrmann
Hi all, Some of you may remember the ideas that I put down in writing 1.5 years ago: ^/notes/fsfs-improvements.txt Over the last two weeks, I got a very rough first prototype running and I will commit it after sending this post. Testing the results pointed me to the FSFS performance issues that

Re: svn commit: r1383214 - in /subversion/trunk: build.conf tools/fsfs/ tools/fsfs/reorg.c

2012-09-10 Thread Blair Zajac
On 9/10/12 6:42 PM, stef...@apache.org wrote: Author: stefan2 Date: Tue Sep 11 01:42:01 2012 New Revision: 1383214 URL: http://svn.apache.org/viewvc?rev=1383214&view=rev Log: Add a very early prototype of an FSFS pack optimization tool. * tools/fsfs (new folder) Suggest one of two things w

Re: svn commit: r1382990 - in /subversion/trunk/subversion: include/ libsvn_subr/ tests/cmdline/ tests/cmdline/getopt_tests_data/

2012-09-10 Thread Ivan Zhakov
On Tue, Sep 11, 2012 at 12:05 AM, Branko Čibej wrote: > On 10.09.2012 21:52, Ivan Zhakov wrote: >> On Mon, Sep 10, 2012 at 9:19 PM, wrote: >>> Author: brane >>> Date: Mon Sep 10 17:19:31 2012 >>> New Revision: 1382990 >>> >>> URL: http://svn.apache.org/viewvc?rev=1382990&view=rev >>> Log: >>> Ma