Re: state of union

On Sat, Sep 08, 2001 at 01:05:17AM +0100, Joe Orton wrote:
> On Fri, Sep 07, 2001 at 04:15:31PM -0700, Greg Stein wrote:
> > On Thu, Sep 06, 2001 at 07:12:04PM +0100, Joe Orton wrote:
> > >...
> > > ra_dav is doing a PROPFIND on /repos/svn/$svn/vcc/default asking for the
> > > baseline-collection and version-name properties.
> >
> > It also uses a Label: header to get those properties. In other words, don't
> > get them from the VCC, but from the baseline resource identified by the
> > label.
>...
> > > But in the response, mod_dav_svn is giving the properties for the URL
> > > /repos/svn/$svn/bln/1, rather than /repos/svn/$svn/vcc/default.
> >
> > As it should. :-)
>
> Okay... this is my understanding of why it gives the error it gives;
> doing a "co -r1"
>
> svn_ra_dav__get_props_resource(blah,
> url = "/repos/svn/$svn/vcc/default",
> label = "1"...)
>
> -> calls svn_ra_dav__get_props for that URL
> -> but because the href of the response was /repos/svn/$svn/bln/1, it's
> not finding the properties when it looks for them in the hash?

Urk... yah. That is the problem. I think if a label is present, then the
function should just look for the "first" URL which is returned and go with
that. IOW, be relaxed on the URL when a label is present, keep the hard
restriction when no label is there.

Cheers,
-g

-- 
Greg Stein, http://www.lyra.org/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org