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

On Mon, Apr 21, 2008 at 6:31 AM, C. Michael Pilato <cmpilato_at_collab.net> wrote:
> 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.

+1

--dave

-- 
David Glasser | glasser@davidglasser.net | http://www.davidglasser.net/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org