> -----Original Message-----
> From: br...@apache.org [mailto:br...@apache.org]
> Sent: zondag 16 december 2012 23:43
> To: comm...@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

Reply via email to