On Mon, Apr 21, 2008 at 10:44:19AM -0700, David Glasser wrote:
> 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
Right, so if we stored 'store-plaintext-passwords' in 'servers' only,
we should also move 'store-passwords' and 'store-auth-creds' to 'servers'.
That is, the whole [auth] section of the 'config' file will go to the
'servers' file, with a comment in 'config' informing users about the move.
Agreed?
--
Stefan Sperling <stsp_at_elego.de> Software Monkey
German law requires the following banner :(
elego Software Solutions GmbH HRB 77719
Gustav-Meyer-Allee 25, Gebaeude 12 Tel: +49 30 23 45 86 96
13355 Berlin Fax: +49 30 23 45 86 95
http://www.elego.de CEO: Olaf Wagner
Store password unencrypted (yes/no)? No
- application/pgp-signature attachment: stored
Received on 2008-04-22 07:50:34 CEST