diff options
author | Richard Maw <richard.maw@gmail.com> | 2014-08-10 20:29:43 +0100 |
---|---|---|
committer | Richard Maw <richard.maw@codethink.co.uk> | 2014-08-12 18:07:41 +0100 |
commit | 0f9c9e2ff3c9afe00735fa986200ac5fdcc8f79e (patch) | |
tree | 95f93aea389a1988b3267d8d4af40d1ac78ec5b7 /tests.branching/show-system-branch-fails-outside-workspace.stderr | |
parent | 2e5d8664920453ede30b450c7b39ac3a0bc141bb (diff) | |
download | morph-0f9c9e2ff3c9afe00735fa986200ac5fdcc8f79e.tar.gz |
Fix `morph edit` when repo has the same ref as system branch
There was a check in it to see whether it needed to do the git branch
and git checkout based on whether the name of the branch matched that in
the morphology.
This had a couple of problems:
1. Now that we aren't always building from HEAD, we need to be able to
roll its commit back, so using the existing branch isn't always the
best idea.
2. It only checks the "ref" field, not "unpetrify-ref", so even though
we clone the right ref in there, it's checking the commit id against
the system branch name, so would always try to re-create the branch,
and fail when it already exists.
So now, we remove the original ref and re-create it with our preferred
HEAD.
A better solution might be to change the clone logic to not
automatically checkout HEAD, and instead require an explicit branch then
checkout, but the initial clone logic is shared with build, and I didn't
feel like tracking down all the different places that it was used.
Diffstat (limited to 'tests.branching/show-system-branch-fails-outside-workspace.stderr')
0 files changed, 0 insertions, 0 deletions