"Mark J. Nelson" <Mark.J.Nelson at Sun.COM> writes:

> Reviewing #356:
>
> You're removing tags for active.revs, but you're only stripping
> active.bases(), right?  These don't necessarily match, do they?
>

If they don't, something is hellishly wrong.  strip strips changes
from the base (revision specified), on up to the head (this is why we
have to strip each local base, as otherwise we may still leave some
local change).

We remove tags that refer to a revision currently in the AL, as all
those revisions, by intent, are about to go away.  This would be
problematic if we allowed reci on branches, but we don't, your
workspace must have exactly one head and no uncommitted change to use
reci (so any branches, named or otherwise, you may have had have since
converged, and we want to remove any local change that isn't the
changeset we just added).

-- Rich  

Reply via email to