On 03/04/2011 07:29 AM, Philip Martin wrote:
> "C. Michael Pilato" <cmpil...@collab.net> writes:
> 
>> Just a thought:  Have you considered expanding the scope of the private
>> resource space rather than using the magic prefix hack?  You could add
>> ".../!svn/vtxn/UUID" and ".../!svn/vtxr/UUID/..." to be alternate ways to
>> address transactions and transaction roots (the "v" there being a shortcut
>> for "virtual").  This is *effectively* the same approach as yours -- there's
>> a different prefix here.  But the prefix is a clearly defined piece of the
>> protocol, not just some magic bit buried in mod_dav_svn's codebase.
> 
> This looks promising, it's quite straightforward to implement.  New
> log/patch follows:

Oh, that's *so* much more satisfying.  Most of this kind of work is
copy-n-paste-n-tweak, I know, but I gave it a once-over anyway.  Looks good
to me!

-- 
C. Michael Pilato <cmpil...@collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to