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

Re: svn commit: r30723 - in branches/dont-save-plaintext-passwords-by-default: . subversion/include subversion/libsvn_subr subversion/svn

From: C. Michael Pilato <cmpilato_at_collab.net>
Date: Mon, 21 Apr 2008 09:31:29 -0400

Stefan Sperling wrote:
> On Sun, Apr 20, 2008 at 04:42:56PM -0700, David Glasser wrote:
>> Hmm, why not just have the "global" servers section be the normal
>> place to configure this? Why use the other config file at all?
> Because most users will look at the 'config' file first, I guess.
> Also, there, it's right next to 'store-passwords'.
> The docstrings reference each other, so I think it's sorta OK...
> The other idea I had was to have the option be valid in both
> the config file and the servers file [global] section, and
> have svn print a warning when the two disagree and fall back
> to 'prompt'. But I discarded that as overkill.

That we ever had the store-passwords option in the 'config' file instead of
in the 'servers' file (bound to RA-related things) might have been a
mistake. Let's evaluate the correct location for this new option without
concern for the dubious decisions of the past.

C. Michael Pilato <cmpilato_at_collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand

Received on 2008-04-22 09:26:52 CEST

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.