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

feature request: commit warning for switched subtrees

From: Todd C. Gleason <tgleason_at_impac.com>
Date: Tue, 13 Oct 2009 09:06:39 -0700

I've seen users occasionally commit changes from multiple trees due to
the accidental presence of switched subtrees, and would like it if
TortoiseSVN gave a warning when you did this.


The way I think this most commonly happens is that the user has two
working copies, for instance:


C:\dev\A -> /branches/A

C:\dev\trunk -> /trunk


The user makes changes in A\1 and copies the whole "1" folder over under
C:\dev\trunk, overwriting C:\dev\trunk\1 .


Then maybe the user makes other changes in C:\dev\trunk\2, or copies
individual files from C:\dev\A\2 into C:\dev\trunk\2 .


Yes, the user doesn't necessarily understand that each individual .svn
folder knows where it goes, and is not a slave to the parent directory
except during switch operations. Or maybe the user knows it but isn't
paying that close attention, or forgot that their various WC folders
were not part of the same tree.


In any case, at this point C:\dev\trunk\1 is mapped to /branches/A/1,
while C:\dev\trunk\2 is mapped to /trunk/2 .


User commits, and changes end up in /branches/A as well as /trunk .


I think this is very rarely desired-similar to committing to a tag. Is
it appropriate to have a safeguard to check for committing from multiple
trees at once, and give a warning to the user? Or perhaps even from one
switched subtree?


For the multiple tree case (as above), when you commit changes from two
trees, TortoiseSVN could give a warning.


Potentially, if you commit from C:\dev\trunk and even if C:\dev\trunk\A
is a switched subtree, and contains all the changes to commit, you might
still give a warning, because the user may think that they're committing
to the trunk. They may even have "smartly" checked the Svn path for
C:\dev\trunk and assumed the whole WC was under /trunk .


Todd C. Gleason

Senior Software Development Engineer

Elekta Impac Software


www.impac.com <http://www.impac.com>


To unsubscribe from this discussion, e-mail: [users-unsubscribe_at_tortoisesvn.tigris.org].
Received on 2009-10-13 23:48:01 CEST

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