RE: svn commit: r1422706 - in /subversion/trunk/subversion: libsvn_subr/opt.c tests/cmdline/svntest/main.py

> -----Original Message-----
> From: brane_at_apache.org [mailto:brane_at_apache.org]
> Sent: zondag 16 december 2012 23:43
> To: commits_at_subversion.apache.org
> Subject: svn commit: r1422706 - in /subversion/trunk/subversion:
> libsvn_subr/opt.c tests/cmdline/svntest/main.py
>
> Author: brane
> Date: Sun Dec 16 22:43:25 2012
> New Revision: 1422706
>
> URL: http://svn.apache.org/viewvc?rev=1422706&view=rev
> Log:
> Print a warning in "svn --version" if plaintext password storage is enabled.

I tried to fix the getopt tests that this change broke, but the extra newline is impossible to parse in the current test setup.

Given the amount of user responses on the introduction of the earlier warnings for plain text password, I would recommend changing the output to show all the enabled password storage backends. That would also help in diagnosing problems with optional stores, while the current warning just produces a warning when the plaintext store is the last remaining option.

We have already revved most of the functions needed, so adding an optional config argument shouldn't be a problem

        Bert
Received on 2012-12-17 10:30:32 CET