Re: svn commit: r1486463 - in /subversion/trunk/tools/server-side/svnpubsub: revprop-change-hook.py svnpubsub/client.py svnpubsub/server.py watcher.py

2013-05-26 Thread Daniel Shahaf
On Sun, May 26, 2013 at 09:22:48PM +, Daniel Shahaf wrote: > On Sun, May 26, 2013 at 08:06:47PM -, br...@apache.org wrote: > > +def main(repo, revision, author, propname, action): > > +else: > > +sys.stderr.write('Unknown revprop change action "%s"\n' % action) > > +retu

Re: svn commit: r1486500 - in /subversion/trunk/tools/server-side/svnpubsub: commit-hook.py revprop-change-hook.py svnpubsub/util.py svnwcsub.py

2013-05-26 Thread Daniel Shahaf
On Mon, May 27, 2013 at 02:52:10AM -, br...@apache.org wrote: > Author: brane > Date: Mon May 27 02:52:10 2013 > New Revision: 1486500 > > URL: http://svn.apache.org/r1486500 > Log: > In SvnPubSub, extract wrapper for subprocess.check_output into a common > utility > module. And make problems

Re: svn commit: r1486397 - /subversion/branches/1.8.x/STATUS

2013-05-26 Thread Daniel Shahaf
Bert Huijben wrote on Sun, May 26, 2013 at 19:57:42 +: > You open an ra session to one canonical url and then get a session to another > one. > > > At that point you can't trust the relative path apis, nor reparent the > session to something using based on the other url. > In other words,

Re: svn commit: r1486457 - /subversion/trunk/tools/server-side/svnpubsub/commit-hook.py

2013-05-26 Thread Ben Reser
On Sun, May 26, 2013 at 1:12 PM, Branko Čibej wrote: > It would probably be a good idea to backport this to 1.8.x, ecause > currently the svnpubsub's commit-hook.py does not work with a 1.8 server. > > Opinions? +1

[svnbench] Revision: 1486495 compiled May 27 2013, 00:22:01 on x86_64-unknown-linux-gnu

2013-05-26 Thread neels
1.7.0@1181106 vs. trunk@1486467 Started at Mon May 27 00:26:23 UTC 2013 *DISCLAIMER* - This tests only file://-URL access on a GNU/Linux VM. This is intended to measure changes in performance of the local working copy layer, *only*. These results are *not* generally true for everyone. Charts of t

Re: svn commit: r1486463 - in /subversion/trunk/tools/server-side/svnpubsub: revprop-change-hook.py svnpubsub/client.py svnpubsub/server.py watcher.py

2013-05-26 Thread Branko Čibej
On 26.05.2013 23:22, Daniel Shahaf wrote: > On Sun, May 26, 2013 at 08:06:47PM -, br...@apache.org wrote: >> +def main(repo, revision, author, propname, action): >> +else: >> +sys.stderr.write('Unknown revprop change action "%s"\n' % action) >> +return > Maybe sys.exit(1)?

Re: svn commit: r1486463 - in /subversion/trunk/tools/server-side/svnpubsub: revprop-change-hook.py svnpubsub/client.py svnpubsub/server.py watcher.py

2013-05-26 Thread Daniel Shahaf
On Sun, May 26, 2013 at 08:06:47PM -, br...@apache.org wrote: > +def main(repo, revision, author, propname, action): > +else: > +sys.stderr.write('Unknown revprop change action "%s"\n' % action) > +return Maybe sys.exit(1)? Otherwise the stderr output will likely go unnoti

Re: svn commit: r1486457 - /subversion/trunk/tools/server-side/svnpubsub/commit-hook.py

2013-05-26 Thread Branko Čibej
On 26.05.2013 21:52, br...@apache.org wrote: > Author: brane > Date: Sun May 26 19:52:04 2013 > New Revision: 1486457 > > URL: http://svn.apache.org/r1486457 > Log: > In 1.8, post-commit hooks get a third argument (the transaction name). > Update the example svnpubsub post-commit handler to accept

Re: svn commit: r1486397 - /subversion/branches/1.8.x/STATUS

2013-05-26 Thread Bert Huijben
You open an ra session to one canonical url and then get a session to another one. At that point you can't trust the relative path apis, nor reparent the session to something using based on the other url. This breaks checkout, diff, or even simple ‘svn ls’ to these url forms. The biggest p

Re: svn commit: r1486397 - /subversion/branches/1.8.x/STATUS

2013-05-26 Thread Daniel Shahaf
Bert Huijben wrote on Sun, May 26, 2013 at 20:51:10 +0200: > > > > -Original Message- > > From: danie...@apache.org [mailto:danie...@apache.org] > > Sent: zondag 26 mei 2013 08:49 > > To: comm...@subversion.apache.org > > Subject: svn commit: r1486397 - /subversion/branches/1.8.x/STATUS >

RE: svn commit: r1486397 - /subversion/branches/1.8.x/STATUS

2013-05-26 Thread Bert Huijben
> -Original Message- > From: danie...@apache.org [mailto:danie...@apache.org] > Sent: zondag 26 mei 2013 08:49 > To: comm...@subversion.apache.org > Subject: svn commit: r1486397 - /subversion/branches/1.8.x/STATUS > > Author: danielsh > Date: Sun May 26 06:48:49 2013 > New Revision: 148

Re: Apache Subversion 1.8.0-rc2 Released

2013-05-26 Thread Daniel Shahaf
Branko Čibej wrote on Sun, May 26, 2013 at 20:00:01 +0200: > On 26.05.2013 16:41, Barry Scott wrote: > > I see the following warning when compiling on Mac OS X 10.8 with Xcode > > 4.6.2, > > "Apple clang version 4.0 (tags/Apple/clang-421.0.60) (based on LLVM > > 3.1svn)". > > > > Would you be wil

Re: Apache Subversion 1.8.0-rc2 Released

2013-05-26 Thread Branko Čibej
On 26.05.2013 16:41, Barry Scott wrote: > I see the following warning when compiling on Mac OS X 10.8 with Xcode 4.6.2, > "Apple clang version 4.0 (tags/Apple/clang-421.0.60) (based on LLVM 3.1svn)". > > Would you be willing to change the code to fix the warning? > > subversion/libsvn_delta/svndiff

Re: Apache Subversion 1.8.0-rc2 Released

2013-05-26 Thread Barry Scott
Oh and these warnings as well: subversion/libsvn_diff/diff_file.c:2211:49: warning: data argument not used by format string [-Wformat-extra-args] modified_start + 1, modified_length)); ^ ./subversion/include/svn_error.h:3

Re: Apache Subversion 1.8.0-rc2 Released

2013-05-26 Thread Barry Scott
I see the following warning when compiling on Mac OS X 10.8 with Xcode 4.6.2, "Apple clang version 4.0 (tags/Apple/clang-421.0.60) (based on LLVM 3.1svn)". Would you be willing to change the code to fix the warning? subversion/libsvn_delta/svndiff.c:791:34: warning: adding 'apr_size_t' (aka 'uns