Re: r1592724: In 'svn propget --revprop', error out on non-existing properties.

2014-05-16 Thread Julian Foad
Hi Daniel. It looks like your change this morning created this warning: subversion/svn/propget-cmd.c:373:36: format '%ld' expects argument of type 'long int', but argument 5 has type 'svn_opt_revision_t' - Julian

Re: [PATCH] fix potentially undefined operation warning

2014-05-16 Thread Mattias Engdegård
16 maj 2014 kl. 19.24 skrev Andreas Stieger: I think he has. Further, the call is to svn__base36toui64() which, if I read correctly, uses the pointer as a return, e.g. in this case it does not matter whether the first parameter was incremented. Never versions of gcc might detect this and no

Re: Issues with bindings tests

2014-05-16 Thread James McCoy
On Wed, May 14, 2014 at 07:26:20AM -0400, James McCoy wrote: > On Wed, May 14, 2014 at 03:35:50AM +, Daniel Shahaf wrote: > > James McCoy wrote on Tue, May 13, 2014 at 22:00:55 -0400: > > > I had tried the below patch but ran into some other issues that I didn't > > > have the time/experience t

Re: [PATCH] fix potentially undefined operation warning

2014-05-16 Thread Stefan Fuhrmann
On Sat, May 10, 2014 at 2:26 PM, Andreas Stieger wrote: > Hello, > > I found a compiler warning in trunk on a potentially undefined operation > when using an increment operator on a variable referenced in the same > expression. This is on gcc 4.3.4, does not happen in gcc 4.7 and up. > Treated as

Re: upgrade_tests.py 29 spurious failure while testing 1.7.17

2014-05-16 Thread Johan Corveleyn
On Tue, May 13, 2014 at 5:58 PM, Ben Reser wrote: > On 5/13/14, 12:43 AM, Johan Corveleyn wrote: >> First, thanks a lot for taking a look and giving a plausible >> explanation. It's a possibility, but I'm not fully convinced yet :-). >> >> Pro: >> - It fits theoretically (the one bit off etc). >>

new svnfsfs tool (was: Re: svn commit: r1594860 [1/3] - in /subversion/trunk: ./ subversion/include/ tools/server-side/ tools/server-side/svnfsfs/)

2014-05-16 Thread Stefan Sperling
On Thu, May 15, 2014 at 10:48:37AM -, stef...@apache.org wrote: > Author: stefan2 > Date: Thu May 15 10:48:37 2014 > New Revision: 1594860 > > URL: http://svn.apache.org/r1594860 > Log: > Introduce FSFS expert tool 'svnfsfs'. It is intended to grow various > FSFS-specific commands in the futu

Re: 1.7.17 up for testing/signing

2014-05-16 Thread Mark Phippard
FWIW, this is the problem I am getting: http://svn.haxx.se/dev/archive-2011-04/0318.shtml And here is apparently someone else that has seen it as this includes a patch: http://svn.haxx.se/dev/archive-2013-08/0069.shtml Mark On Fri, May 16, 2014 at 1:41 PM, Mark Phippard wrote: > I added my

Re: [PATCH] generate subversion.pot in dist.sh

2014-05-16 Thread Mattias Engdegård
Sorry about the reminder, but are there any objections to the patch or could it be applied? I have received no comments on it, but perhaps the Apache mail server gremlins swallowed them. 6 maj 2014 kl. 23.52 skrev Mattias Engdegård: Translators will need the subversion.pot file; it should be

Re: 1.7.17 up for testing/signing

2014-05-16 Thread Ivan Zhakov
On 30 April 2014 22:00, Ben Reser wrote: > The 1.7.17 release artifacts are now available for testing/signing. > Please get the tarballs from > https://dist.apache.org/repos/dist/dev/subversion > and add your signatures there. I plan to try and release on May 7th so please > try and get your vo

Re: 1.7.17 up for testing/signing

2014-05-16 Thread Paul Burba
On Thu, May 15, 2014 at 5:12 PM, Ben Reser wrote: > On 4/30/14, 11:00 AM, Ben Reser wrote: >> The 1.7.17 release artifacts are now available for testing/signing. >> Please get the tarballs from >> https://dist.apache.org/repos/dist/dev/subversion >> and add your signatures there. I plan to try

Re: [PATCH] fix potentially undefined operation warning

2014-05-16 Thread Andreas Stieger
Hi, > On 15 May 2014, at 15:49, Branko Čibej wrote: > > Good catch, the result of these operations is indeed undefined, according to > the C standard. > However, I don't know if your fix is correct ... the other interpretation is: > > foo(&data, data + 1); > ++data; > > The au

Re: 1.7.17 up for testing/signing

2014-05-16 Thread Mark Phippard
I added my signature to Windows build and give it a +1. I will leave it to you to decide if you want to count it. I was able to verify the zip file, build successfully as well as run the JavaHL tests successfully. The build has been failing for me for a while when linking the C tests. It is jus

Moving towards a 1.9.x branch

2014-05-16 Thread Ben Reser
We planned to release a year after 1.8.0, it doesn't look like we're going to make that. However, we can still manage to a release out with some delay after that. To that end I suggest that we need to start moving towards a 1.9.x branch. We'd planned to defer new development and concentrate on

Re: deleting locked file fails on 1.8.x

2014-05-16 Thread Sergey Raevskiy
> The problem can be fixed by merging subversion/trunk:r1553501,1553556,1559197 > with --accept=theirs-conflict (+ adding a missing svn_error_t *err; > declaration). This worked for me, all tests passed, including new one (Windows 8, httpd 2.2.27, svn 1.8.8, http x fsfs). BTW, should this be fil

Re: 1.7.17 up for testing/signing

2014-05-16 Thread Ben Reser
On 4/30/14, 11:00 AM, Ben Reser wrote: > The 1.7.17 release artifacts are now available for testing/signing. > Please get the tarballs from > https://dist.apache.org/repos/dist/dev/subversion > and add your signatures there. I plan to try and release on May 7th so please > try and get your votes

Re: deleting locked file fails on 1.8.x

2014-05-16 Thread Stefan Sperling
On Thu, May 15, 2014 at 01:05:53PM +0400, Sergey Raevskiy wrote: > > The problem can be fixed by merging > > subversion/trunk:r1553501,1553556,1559197 > > with --accept=theirs-conflict (+ adding a missing svn_error_t *err; > > declaration). > > This worked for me, all tests passed, including new

svn doesn't cope well with read-only auth cache

2014-05-16 Thread Johan Corveleyn
If the file with the cached credentials (the file under /auth/svn.simple) is read-only, 'svn' 1.8 (on Windows) will behave as follows after a password change: [[[ C:\>svn ls https://svn.example.com/svn Authentication realm: SVN Password for 'myself': ### dela

Re: [PATCH] fix potentially undefined operation warning

2014-05-16 Thread Branko Čibej
On 10.05.2014 05:26, Andreas Stieger wrote: > Hello, > > I found a compiler warning in trunk on a potentially undefined operation > when using an increment operator on a variable referenced in the same > expression. This is on gcc 4.3.4, does not happen in gcc 4.7 and up. > Treated as an error in s

Re: svn doesn't cope well with read-only auth cache

2014-05-16 Thread Johan Corveleyn
On Thu, May 15, 2014 at 4:16 PM, Johan Corveleyn wrote: > If the file with the cached credentials (the file under config dir>/auth/svn.simple) is read-only, 'svn' 1.8 (on Windows) will > behave as follows after a password change: > > [[[ > C:\>svn ls https://svn.example.com/svn > Authentication r

RE: svn commit: r1593015 - in /subversion/trunk/subversion/libsvn_fs_fs: fs.c fs.h

2014-05-16 Thread Bert Huijben
> -Original Message- > From: stef...@apache.org [mailto:stef...@apache.org] > Sent: woensdag 7 mei 2014 15:44 > To: comm...@subversion.apache.org > Subject: svn commit: r1593015 - in > /subversion/trunk/subversion/libsvn_fs_fs: fs.c fs.h > > Author: stefan2 > Date: Wed May 7 13:43:55 20