Re: [PATCH] Automate ra_svn + Cyrus SASL testing

On 9/25/06, Vlad Georgescu <vgeorgescu@gmail.com> wrote:
> With this patch, you can run 'make svncheck ENABLE_SASL=true' or 'make
> svnserveautocheck ENABLE_SASL=true' and all authentication requests
> will be handled by Cyrus SASL.

Vlad:

This looks good. I'm wondering though: I'm not too familiar with
SASL, but is it easy to set up non-global SASL configurations? Having
to run stuff as root and/or introduce fake users into a system
database during tests seems non-ideal. Could we possibly make
ENABLE_SASL=true automatically make a SASL configuration for you?
(I'm happy to do the legwork if you can point me at the appropriate
SASL commands and/or flags.)

(As a random code review note: shouldn't enable_sasl just be set to
True instead of 1? I guess the other options aren't doing that, but
it still seems better.)

--dave

-- 
David Glasser | glasser_at_mit.edu | http://www.davidglasser.net/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org