You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by David Glasser <gl...@davidglasser.net> on 2008/03/31 19:41:09 UTC
Re: svn commit: r29927 - trunk/subversion/svn
On Fri, Mar 14, 2008 at 9:34 PM, <kf...@tigris.org> wrote:
> Author: kfogel
> Date: Fri Mar 14 21:34:50 2008
> New Revision: 29927
>
> Log:
> * subversion/svn/conflict-callbacks.c
> (svn_cl__conflict_handler): Use "s" to "show all options", where
> doing so both shows more options and shows help for all options.
>
> See this message and its thread for more information:
>
> http://subversion.tigris.org/servlets/ReadMsg?list=dev&msgNo=136061
> From: Blair Zajac <bl...@orcaware.com>
> To: Subversion Developers <de...@subversion.tigris.org>
> Subject: Show all options for conflict resolution
> Date: Thu, 13 Mar 2008 15:59:44 -0700
> Message-ID: <47...@orcaware.com>
>
> Modified:
> trunk/subversion/svn/conflict-callbacks.c
>
> Modified: trunk/subversion/svn/conflict-callbacks.c
> URL: http://svn.collab.net/viewvc/svn/trunk/subversion/svn/conflict-callbacks.c?pathrev=29927&r1=29926&r2=29927
> ==============================================================================
> --- trunk/subversion/svn/conflict-callbacks.c Fri Mar 14 21:34:32 2008 (r29926)
> +++ trunk/subversion/svn/conflict-callbacks.c Fri Mar 14 21:34:50 2008 (r29927)
> @@ -402,12 +402,12 @@ svn_cl__conflict_handler(svn_wc_conflict
>
> prompt = apr_pstrcat(subpool, prompt, ",\n ", NULL);
> prompt = apr_pstrcat(subpool, prompt,
> - _("(h) help for more options: "),
> + _("(s) show all options: "),
> NULL);
>
> SVN_ERR(svn_cmdline_prompt_user2(&answer, prompt, b->pb, subpool));
>
> - if (strcmp(answer, "h") == 0 || strcmp(answer, "?") == 0)
> + if (strcmp(answer, "s") == 0)
> {
> SVN_ERR(svn_cmdline_fprintf(stderr, subpool,
> _(" (p) postpone - mark the conflict to be "
> @@ -421,7 +421,7 @@ svn_cl__conflict_handler(svn_wc_conflict
> "(lose my changes)\n"
> " (l) launch - launch external tool to "
> "resolve conflict\n"
> - " (h) help - show this list\n\n")));
> + " (s) show all - show this list\n\n")));
> }
> else if (strcmp(answer, "p") == 0)
> {
> @@ -553,7 +553,7 @@ svn_cl__conflict_handler(svn_wc_conflict
> "(ignore upstream addition)\n"
> " (tf) theirs-full - accept incoming item "
> "(overwrite pre-existing item)\n"
> - " (h) help - show this list\n\n")));
> + " (h) help - show this help\n\n")));
Hmm, so it looks like the help list is brought up by (s)how in some
prompts and (h)elp in other prompts? (The difference being "will it
show more options or just the same ones".) That seems pretty
confusing to me.
Also, do we really have to remove "?" ?
--dave
> }
> if (strcmp(answer, "p") == 0)
> {
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: svn-unsubscribe@subversion.tigris.org
> For additional commands, e-mail: svn-help@subversion.tigris.org
>
>
--
David Glasser | glasser@davidglasser.net | http://www.davidglasser.net/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Re: svn commit: r29927 - trunk/subversion/svn
Posted by Karl Fogel <kf...@red-bean.com>.
"David Glasser" <gl...@davidglasser.net> writes:
> Hmm, so it looks like the help list is brought up by (s)how in some
> prompts and (h)elp in other prompts? (The difference being "will it
> show more options or just the same ones".) That seems pretty
> confusing to me.
>
> Also, do we really have to remove "?" ?
Suggested solution:
We *advertise* "s" in the one case and "h" in the other ("s" when it
shows more options, "h" when it shows just the same ones). But in both
cases, we actually *accept* "s", "h", and "?", all synonymous.
Shall I?
-Karl
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org