[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

RE: svn commit: r1066604 - /subversion/trunk/subversion/svn/main.c

From: Bert Huijben <bert_at_qqmail.nl>
Date: Wed, 2 Feb 2011 22:15:36 +0100

> -----Original Message-----
> From: stsp_at_apache.org [mailto:stsp_at_apache.org]
> Sent: woensdag 2 februari 2011 21:08
> To: commits_at_subversion.apache.org
> Subject: svn commit: r1066604 - /subversion/trunk/subversion/svn/main.c
>
> Author: stsp
> Date: Wed Feb 2 20:07:43 2011
> New Revision: 1066604
>
> URL: http://svn.apache.org/viewvc?rev=1066604&view=rev
> Log:
> * subversion/svn/main.c
> (svn_cl__cmd_table): On Windows, escape the caret in merge help text
> examples
> using the ^/ URL shortcut notation. This is required by cmd.exe.
> E.g. 'svn merge ^/trunk' becomes 'svn merge ^^/trunk'
> The same could be done for one example in 'svn help switch', but that
> would make getopt_test 7 fail and I have no way to run tests on Windows.

This patch most likely breaks localization of this text and I'm not sure if we should really change the examples to show the quoting. If this is a cmd.exe requirement I don't think we should perform the escaping as there are more ways to run svn than just from cmd.

Adding some note suggesting the quoting after the main help text might be more appropriate and this would be easier to localize.

        Bert
Received on 2011-02-02 22:16:25 CET

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.