[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

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

From: Bert Huijben <bert_at_qqmail.nl>
Date: Mon, 17 Dec 2012 10:29:50 +0100

> -----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

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.