On Thu, Jan 29, 2015 at 07:26:01PM +0100, Stefan Kueng wrote:
> TortoiseSVN has such a feature. Of course, it doesn't work in an ideal way.
> Here's what I do in TSVN:
> The branch/tag dialog (the one that's shown for an 'svn copy') scans the
> working copy for all external properties, and it does this recursively. It
What happens if one or more recursive externals definitions end up looping
back to an external that's already been traversed?
> also checks the uuid of the repository. All externals that point to the main
> repository (even the recursive ones) are then shown in a list at the bottom
> of the dialog. Each external item (file or folder) is also scanned for the
> highest revision. That's the revision the external is later tagged to.
> The list has a checkbox for each item, so users can chose which external
> they want to tag.
> See here for a screenshot:
> For copies made from a working copy, TSVN first modifies all external
> properties locally, then creates the copy from the working copy. For copies
> done directly on the repository, it first creates the copy, then modifies
> each external property separately (which leads to multiple 'commits' - as I
> said: not ideal).
> Looking at svn_client_commit6(), it has two bool values to include file
> and/or dir externals in the commit. So it seems to me that it might be
> possible to change externals (that reside in the same repository)
Well, I'd rather avoid having to make commits to multiple repositories.
If this feature only works for externals pointing at the same repository
then it's quite limited, isn't it?
> Also, the current implementation of this feature does not allow to select
> externals individually to tag. Would it be a good idea to extend this? Maybe
> one API that returns a list of externals and their revision which they would
> be tagged to, and then svn_client_copy6 would take that list (or a list with
> only those items the user wants to tag) and then only tag those externals in
> that list?
> This way, TSVN could make use of this new APIs and drop its own (not ideal)
If you need something like this in the API we can definitely make it happen.
But I'd rather not implement a recursive externals crawl in svn_client_copy().
Received on 2015-01-29 22:47:38 CET