Re: [PATCH] add mod_authz_svndb to subversion

Greg, I haven't reviewed this patch in depth yet, but I did notice a
lot of similarities between it and our existing authz module.

I was wondering why you chose to implement it as a separate module?

Also, I was wondering if you were planning on integrating it into
Subversion's test suite.

Thanks, Dan

On Sat, 10 Feb 2007, Gregory Bartholomew wrote:

> Just thought I'd bump this patch to the top since I havn't heard any
> feedback and I'd like to point out that I think this module could handle the
> isse 2712 posted previously to the issues group (if the user were willing to
> use apache instead of svnserve).
>
> gb
>
> On 2/7/07, Gregory Bartholomew <gregory.lee.bartholomew@gmail.com> wrote:
> >
> >[[[
> >ADD mod_authz_svndb to subversion
> >[in subversion/mod_authz_svndb]
> >
> >* INSTALL
> >* mod_authz_svndb.c
> >* mod_authz_svndb.h
> >* special.h
> >* svndb.sh
> >* svndb.sql
> >
> >]]]
...
> >> On Tue, Feb 06, 2007 at 10:41:01PM -0600, Gregory Bartholomew wrote:
> >> > Hi,
> >> >
> >> > For my own uses, I have modified mod_authz_svn to get its permissions
> >> > from a database via apache 2's dbd api. I went all-out and
> >> > implemented a very rich feature
> >> > set and think that it would make a good addition to the subversion
> >> > project. Below,
> >> > I have pasted the modified INSTALL file with the feature discriptions
> >for
> >> > your
> >> > review. I have never contributed to an open source project and
> >> > request instructions
> >> > on how and to whom to send the source for review.