You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Ben Collins-Sussman <su...@collab.net> on 2004/03/03 22:16:38 UTC

Re: [PATCH] Add SVNParentPath support to mod_authz_svn

On Sun, 2004-02-29 at 10:08, Sander Striker wrote:
> Hi,
> 
> Completely untested, but throwing this out here since I ran out
> of time to work on this, and more specifically, to give it a good
> test run.  I'll try and pick it up later.

I'd love to review this, but I'm strapped for time right now.

Still, a log message might make it a bit easier to understand.  :-)



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: [PATCH] Add SVNParentPath support to mod_authz_svn

Posted by John Szakmeister <jo...@szakmeister.net>.
On Saturday 06 March 2004 03:52, Sander Striker wrote:
[snip]
> > I took a look at your patch and did some testing as well.  It was very
> > well behaved and did what it should after I added the missing user
> > parameter. This will definitely get a vote from me! :-)
>
> Care to give it a vote in STATUS? :)

Meant to do that last night... done.

-John

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: [PATCH] Add SVNParentPath support to mod_authz_svn

Posted by Sander Striker <st...@apache.org>.
On Sat, 2004-03-06 at 01:34, John Szakmeister wrote:
> On Friday 05 March 2004 07:54, Sander Striker wrote:
> > On Wed, 2004-03-03 at 23:16, Ben Collins-Sussman wrote:
> > > On Sun, 2004-02-29 at 10:08, Sander Striker wrote:
> > > > Hi,
> > > >
> > > > Completely untested, but throwing this out here since I ran out
> > > > of time to work on this, and more specifically, to give it a good
> > > > test run.  I'll try and pick it up later.
> > >
> > > I'd love to review this, but I'm strapped for time right now.
> > >
> > > Still, a log message might make it a bit easier to understand.  :-)
> >
> > Committed in r8892, after some testing ;).  While you're at it,
> > you may want to look at r8891 aswell :)
> 
> I took a look at your patch and did some testing as well.  It was very well 
> behaved and did what it should after I added the missing user parameter.  
> This will definitely get a vote from me! :-)

Care to give it a vote in STATUS? :)

Sander

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: [PATCH] Add SVNParentPath support to mod_authz_svn

Posted by John Szakmeister <jo...@szakmeister.net>.
On Friday 05 March 2004 07:54, Sander Striker wrote:
> On Wed, 2004-03-03 at 23:16, Ben Collins-Sussman wrote:
> > On Sun, 2004-02-29 at 10:08, Sander Striker wrote:
> > > Hi,
> > >
> > > Completely untested, but throwing this out here since I ran out
> > > of time to work on this, and more specifically, to give it a good
> > > test run.  I'll try and pick it up later.
> >
> > I'd love to review this, but I'm strapped for time right now.
> >
> > Still, a log message might make it a bit easier to understand.  :-)
>
> Committed in r8892, after some testing ;).  While you're at it,
> you may want to look at r8891 aswell :)

I took a look at your patch and did some testing as well.  It was very well 
behaved and did what it should after I added the missing user parameter.  
This will definitely get a vote from me! :-)

Thanks Sander!

-John

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: [PATCH] Add SVNParentPath support to mod_authz_svn

Posted by Sander Striker <st...@apache.org>.
On Wed, 2004-03-03 at 23:16, Ben Collins-Sussman wrote:
> On Sun, 2004-02-29 at 10:08, Sander Striker wrote:
> > Hi,
> > 
> > Completely untested, but throwing this out here since I ran out
> > of time to work on this, and more specifically, to give it a good
> > test run.  I'll try and pick it up later.
> 
> I'd love to review this, but I'm strapped for time right now.
> 
> Still, a log message might make it a bit easier to understand.  :-)

Committed in r8892, after some testing ;).  While you're at it,
you may want to look at r8891 aswell :)

Sander

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: [PATCH] Add SVNParentPath support to mod_authz_svn

Posted by John Szakmeister <jo...@szakmeister.net>.
On Wednesday 03 March 2004 17:16, Ben Collins-Sussman wrote:
> On Sun, 2004-02-29 at 10:08, Sander Striker wrote:
> > Hi,
> >
> > Completely untested, but throwing this out here since I ran out
> > of time to work on this, and more specifically, to give it a good
> > test run.  I'll try and pick it up later.
>
> I'd love to review this, but I'm strapped for time right now.
>
> Still, a log message might make it a bit easier to understand.  :-)

I've actually been meaning to pick this up and do some testing.  It looks like 
I'll have some time before the weekend to do this.  I'll post my results some 
time over the weekend.

-John

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org