Re: svn commit: r997249 - in /subversion/trunk/subversion: include/svn_client.h libsvn_client/patch.c svn/cl.h svn/main.c svn/patch-cmd.c tests/cmdline/patch_tests.py tests/libsvn_client/client-test.c

2010-09-17 Thread Stefan Sperling
On Fri, Sep 17, 2010 at 02:01:27AM +0200, Daniel Shahaf wrote: Devil's advocate: It's a very narrow-scoped option, and might become obsolete if we extend this part of the code a bit. Yes, that's why I wasn't truely happy with it. The plan to detect the proper filename dynamically using the

Re: svn commit: r997249 - in /subversion/trunk/subversion: include/svn_client.h libsvn_client/patch.c svn/cl.h svn/main.c svn/patch-cmd.c tests/cmdline/patch_tests.py tests/libsvn_client/client-test.c

2010-09-16 Thread Daniel Shahaf
Stefan Sperling wrote on Wed, Sep 15, 2010 at 20:42:55 +0200: On Wed, Sep 15, 2010 at 04:51:15PM +0200, Daniel Shahaf wrote: s...@apache.org wrote on Wed, Sep 15, 2010 at 10:05:15 -: Author: stsp Date: Wed Sep 15 10:05:14 2010 New Revision: 997249 URL:

Re: svn commit: r997249 - in /subversion/trunk/subversion: include/svn_client.h libsvn_client/patch.c svn/cl.h svn/main.c svn/patch-cmd.c tests/cmdline/patch_tests.py tests/libsvn_client/client-test.c

2010-09-15 Thread Daniel Shahaf
s...@apache.org wrote on Wed, Sep 15, 2010 at 10:05:15 -: Author: stsp Date: Wed Sep 15 10:05:14 2010 New Revision: 997249 URL: http://svn.apache.org/viewvc?rev=997249view=rev Log: Add an --old-patch-target-names option to svn patch. This option is useful in two cases: 1) A patch

Re: svn commit: r997249 - in /subversion/trunk/subversion: include/svn_client.h libsvn_client/patch.c svn/cl.h svn/main.c svn/patch-cmd.c tests/cmdline/patch_tests.py tests/libsvn_client/client-test.c

2010-09-15 Thread Stefan Sperling
On Wed, Sep 15, 2010 at 04:51:15PM +0200, Daniel Shahaf wrote: s...@apache.org wrote on Wed, Sep 15, 2010 at 10:05:15 -: Author: stsp Date: Wed Sep 15 10:05:14 2010 New Revision: 997249 URL: http://svn.apache.org/viewvc?rev=997249view=rev Log: Add an --old-patch-target-names

Re: svn commit: r997249 - in /subversion/trunk/subversion: include/svn_client.h libsvn_client/patch.c svn/cl.h svn/main.c svn/patch-cmd.c tests/cmdline/patch_tests.py tests/libsvn_client/client-test.c

2010-09-15 Thread Hyrum K. Wright
On Wed, Sep 15, 2010 at 8:42 PM, Stefan Sperling s...@elego.de wrote: On Wed, Sep 15, 2010 at 04:51:15PM +0200, Daniel Shahaf wrote: s...@apache.org wrote on Wed, Sep 15, 2010 at 10:05:15 -: Author: stsp Date: Wed Sep 15 10:05:14 2010 New Revision: 997249 URL:

Re: svn commit: r997249 - in /subversion/trunk/subversion: include/svn_client.h libsvn_client/patch.c svn/cl.h svn/main.c svn/patch-cmd.c tests/cmdline/patch_tests.py tests/libsvn_client/client-test.c

2010-09-15 Thread Stefan Sperling
On Wed, Sep 15, 2010 at 08:53:17PM +0200, Hyrum K. Wright wrote: How about just adding '--patch-left' and '--patch-right' or something. The either both have to used, or neither, and if they are, use those names as the once in the patch. I don't like these names because it's not clear what