Re: svn commit: rev 7541 - in trunk/subversion: include libsvn_client libsvn_wc

Greg Stein <gstein@lyra.org> writes:

> On Mon, Oct 27, 2003 at 02:18:48PM -0600, cmpilato@tigris.org wrote:
> >...
> > Log:
> > Finish issue #1557 - uuid's not assigned to root folder.
> >...
> > * subversion/libsvn_client/checkout.c
> > (svn_client__checkout_internal): Fetch the repository UUID, and
> > update calls to svn_wc_ensure_adm(), passing that UUID.
>
> I'm not familiar enough with the underlying code/control/data flow here,
> but is this special-casing the UUID too much? Are we going to miss out on
> other (entry) or revision properties?

Yes. Shoot. I, uh, forgot about that when I made this change.

I suppose we could go back to making dir_delta call open_root() no
matter what, oh, and then call change_dir_prop() even though no props
were changed. Which means we have to teach every editor to know the
difference between a real delta and a fake one.

> This is adding yet *another* turnaround to the server, isn't it?

Yes (unless the UUID is cached in the session baton, which is highly
unlikely).

I'll gladly revert, and I'm all ears for better suggestions.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Oct 27 23:43:29 2003