On Thu, Apr 30, 2009 at 13:33, Hyrum K. Wright <hyrum_at_hyrumwright.org> wrote:
>...
>>> + * move wcprops into wc.db (as "dav_cache")
>>> + * the wcprops have already been switched to live in wc.db. no
>>> proactive shift has occurred for format 12 working copies. formats
>>> less
>>> than 12 will have their properties shifting during the upgrade
>>> process.
>>> when we bump to 13, we will force a shift of wcprops, and ensure that
>>> the
>>> `.svn` subdirs are cleaned of old wcprops files
>>
>> Since we already cache properties of format 12 can't move the
>> dav_cache from
>> the files into the db in the 12->13 step. This would break 12
>> working copies
>> that already have the properties in the database.
>>
>> We would overwrite valid cached properties in the database with a
>> possible
>> out of date version in the working copy. (Of course we can move these
>> properties for working copies <= 11).
>
> Sure, but it's pretty easy to check and see if there's something in
> wc.db before we blindly overwrite it. Do you see any problems with
> that approach?
Sorry. Should have been more clear...
Yes, I meant "if there is nothing present, then force the shift."
Bert is absolutely right that we don't want to grab old wcprops, if
there is something in wc.db.
Cheers,
-g
------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1995055
Received on 2009-04-30 15:16:05 CEST