Yes yes, of course. Nothing to see here, please move along. (Thank
goodness for code review.)
On Tue, Jun 10, 2008 at 10:37 AM, C. Michael Pilato <cmpilato_at_collab.net> wrote:
> Ben Collins-Sussman wrote:
>
> [...]
>
>> @@ -404,13 +405,19 @@
>> "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));
>> + ), realmstring));
>>
>> + if (config_path)
>> + SVN_ERR(svn_cmdline_fprintf(stderr, pool,
>> + _("\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"
>> + ), realmstring));
>
> ^^^^ shouldn't this be 'config_path' instead of realm_string?
>
> --
> C. Michael Pilato <cmpilato_at_collab.net>
> CollabNet <> www.collab.net <> Distributed Development On Demand
>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-06-10 17:40:01 CEST