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

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