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

Re: svn commit: r31506 - branches/1.5.x/subversion/svn

From: Lieven Govaerts <svnlgo_at_mobsol.be>
Date: Wed, 28 May 2008 23:22:13 +0200

jensseidel_at_tigris.org wrote:
> Author: jensseidel
> Date: Wed May 28 13:24:10 2008
> New Revision: 31506
>
> Log:
> On the '1.5.x' branch:
> Follow-up to r31501:
>
> * subversion/svn/main.c
> (svn_cl__options."accept"): Added a missing comma.
> This also brings some translations back to 100%.
>
> Modified:
> branches/1.5.x/subversion/svn/main.c
>
> Modified: branches/1.5.x/subversion/svn/main.c
> URL: http://svn.collab.net/viewvc/svn/branches/1.5.x/subversion/svn/main.c?pathrev=31506&r1=31505&r2=31506
> ==============================================================================
> --- branches/1.5.x/subversion/svn/main.c Wed May 28 13:05:49 2008 (r31505)
> +++ branches/1.5.x/subversion/svn/main.c Wed May 28 13:24:10 2008 (r31506)
> @@ -259,7 +259,7 @@ const apr_getopt_option_t svn_cl__option
> {"accept", opt_accept, 1,
> N_("specify automatic conflict resolution action\n"
> " "
> - "('postpone', 'base', 'mine-full', 'theirs-full'\n"
> + "('postpone', 'base', 'mine-full', 'theirs-full',\n"
> " "
> " 'edit', 'launch')")},
> /* 'mine-conflict' and 'theirs-conflict' are not
>
>
As simple as this change might be, it breaks one of the getopt tests. The getopt tests are one of the only tests really matching the actual string content character per character. I know these tests are overly strict, but while we have them I propose you run at least those before you commit a text change inside the code.

regards,

Lieven

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-05-28 23:22:27 CEST

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