Re: svn commit: r31050 - trunk/subversion/libsvn_subr

On Tue, May 06, 2008 at 12:53:58PM -0700, cmpilato_at_tigris.org wrote:
> Author: cmpilato
> Date: Tue May 6 12:53:58 2008
> New Revision: 31050
>
> Log:
> * subversion/libsvn_subr/prompt.c
> (svn_cmdline_auth_plaintext_prompt): Rework the screen-splash text
> for improved readability (including better handling of strangely
> long or strangely short variable-length bits).
>
> Modified:
> trunk/subversion/libsvn_subr/prompt.c

Yay, this looks much better. I was a bit afraid of using
up that much vertical space, but it certainly makes the
prompt look even more like something important that should
be paid attention to :)

Thanks also for the provider-ordering fix, that issue
totally escaped me.

> Modified: trunk/subversion/libsvn_subr/prompt.c
> URL: http://svn.collab.net/viewvc/svn/trunk/subversion/libsvn_subr/prompt.c?pathrev=31050&r1=31049&r2=31050
> ==============================================================================
> --- trunk/subversion/libsvn_subr/prompt.c Tue May 6 10:22:17 2008 (r31049)
> +++ trunk/subversion/libsvn_subr/prompt.c Tue May 6 12:53:58 2008 (r31050)
> @@ -397,14 +397,17 @@ svn_cmdline_auth_plaintext_prompt(svn_bo
>
> SVN_ERR(svn_cmdline_fprintf(stderr, pool,
> _("-----------------------------------------------------------------------\n"
> - "ATTENTION! Your password for authentication realm\n"
> - "'%s'\n"
> - "can only be stored to disk unencrypted! It is recommended to configure\n"
> - "your system so that Subversion can store passwords encrypted, "
> - "if possible.\n"
> - "See the documentation for details. You can get rid of this warning by\n"
> - "editing '%s'\n"
> - "and setting 'store-plaintext-passwords' to either 'yes' or 'no'.\n"
> + "ATTENTION! Your password for authentication realm:\n"
> + "\n"
> + " %s\n"
> + "\n"
> + "can only be stored to disk unencrypted! You are advised to configure\n"
> + "your system so that Subversion can store passwords encrypted, if\n"
> + "possible. See the documentation for details.\n"
> + "\n"
> + "You can avoid future appearances of this warning by setting the value\n"
> + "of the 'store-plaintext-passwords' option to either 'yes' or 'no' in\n"
> + "'%s'.\n"
> "-----------------------------------------------------------------------\n"
> ), realmstring, config_path));
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: svn-unsubscribe_at_subversion.tigris.org
> For additional commands, e-mail: svn-help_at_subversion.tigris.org