A few quick comments:
I) Setting the property on the parent of the collection rather than on the
collection itself scares me. It means that things like 'svn cp collection
another-collection' will fail to remember that this item is a collection.
'Collectionness' is a property of the collection, not its parent.
II) I don't think you need two types of collections (although I guess I have
no strong objections). I'd just make all collections opaque collections.
III) The fact that .rtfd collections can rename all the files in the
collection is going to wreak havok with incremental storage if that is
handled as a series of adds and removes. I wonder whether it might not be
better to store collections on the server in some combined format. E.g. tar
the directory up before sending it to the server. That might make
incremental storage more efficient.
Question: How does the diffy storage handle transpositions? Does the change
from AABBBBBBCCCCCCDD to AACCCCCBBBBBDD get stored efficiently?
Of course, taring things up in this way has a number of down-sides too. It
is a much larger change (it sounds a little to me like the module support -
instead of 'read this property and check out these other files', it becomes
'read this property and untar these other files'). It requires either a
dependency on tar or adding an equivalent library to svn. It is
significantly less efficient.
IV) update/merge: I think you need to treat the collection like any other
opaque file type - if you get a diff anywhere in the collection then you end
up with a three versions of the entire collection on disc - the base, the
local version and the head version. You then use plug-in diff/merge tools
on the entire thing. I don't think you want to start producing conflicts on
files within the collection.
Yay for bundle support :),
\x/ill :-}
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu May 23 23:44:26 2002