[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

RE: Fixing externals on branch/tag

From: Oddvar Kloster <Oddvar.Kloster_at_sintef.no>
Date: Fri, 30 Apr 2010 10:04:49 +0200

> > A related thought: Now that there are dedicated dialogs for editing
> > externals, would it make more sense to put the functionality for
> fixing
> > external revisions there, instead of in the branch/tag dialog? There
> > could be a button for updating the explicit revision of the externals
> > that are selected in the list, to the revision in the working copy.
> If
> > none are selected, update all. To me, this seems like a satisfactory
> > replacement, which is useful in more situations, e.g. also for normal
> > commits. The branch/tag dialog could simply give a warning if the
> user
> > is trying to branch/tag without fixed external revisions.
>
> I think the place for this really should be the branch/tag dialog.
> Because if it is in the properties dialog, it's not possible to fix the
> externals to a specific revisions if you branch/tag directly from urls.
> In that case, the branch/tag dialog tells the progress dialog to fix
> the
> external properties after the branch/tag is made, and the progress
> dialog then fixes those externals directly on the repository which
> unfortunately creates another revision.
>
> Stefan

Point taken. Although I still feel that an 'update external rev to rev in WC' button would be useful.
I assume that when branching from a URL at HEAD or a fixed revision, the revisions chosen are the latest revisions of the externals before the time of the chosen revision. In that case, the UI is somewhat confusing, since the revisions listed in the list are WC revisions no matter what radio button is chosen above. Is would be better to either show the correct revisions from the repos (probably complex...) or hide the revision column.

Oddvar

------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=757&dsMessageId=2599539

To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].
Received on 2010-04-30 10:27:03 CEST

This is an archived mail posted to the TortoiseSVN Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.