diff options
author | Sam Thursfield <sam.thursfield@codethink.co.uk> | 2012-10-01 19:30:23 +0100 |
---|---|---|
committer | Sam Thursfield <sam.thursfield@codethink.co.uk> | 2012-10-03 14:40:37 +0100 |
commit | 8bcb622e182d49d143af417bd3d0a64d4188356b (patch) | |
tree | 917404b4d2108af6e175fb8e84eaea69548a7d6f /morphlib/plugins | |
parent | 28f8e387b8e5bf6ffcc0ca340bfc8717d5ffc9d5 (diff) | |
download | morph-8bcb622e182d49d143af417bd3d0a64d4188356b.tar.gz |
Use origin/ref for all refs other than the system branch ref itself
The rationale here is that inside a checkout of a system branch, the
user should only be committing to the refs for that system branch,
because those are the only ones that 'morph merge' will look at.
This removes a problem where we could be confused by a ref with a
name that would be sorted before 'master' in the result of 'git
show-ref'.
Diffstat (limited to 'morphlib/plugins')
-rw-r--r-- | morphlib/plugins/branch_and_merge_plugin.py | 16 |
1 files changed, 15 insertions, 1 deletions
diff --git a/morphlib/plugins/branch_and_merge_plugin.py b/morphlib/plugins/branch_and_merge_plugin.py index 279a9948..6f2e38fc 100644 --- a/morphlib/plugins/branch_and_merge_plugin.py +++ b/morphlib/plugins/branch_and_merge_plugin.py @@ -221,12 +221,26 @@ class BranchAndMergePlugin(cliapp.Plugin): self.app.runcmd(['git', 'remote', 'update'], cwd=dirname) def load_morphology(self, repo_dir, name, ref=None): + '''Returns a morphology, optionally retrieved from a specific ref + + The repositories that make up a system branch checkout should only + have modifications to the ref matching that system branch. Unless 'ref' + is a sha1, origin/ref will be used instead to ensure that no local + modifications are considered and no local tracking branch needs to be + created. Where the ref is that of the system branch, 'None' should be + passed because we should load directly from the working tree in this + case, honouring any uncommitted changes. + + ''' + if ref is None: filename = os.path.join(repo_dir, '%s.morph' % name) with open(filename) as f: text = f.read() else: - ref = morphlib.git.find_first_ref(self.app.runcmd, repo_dir, ref) + if not morphlib.git.is_valid_sha1(ref): + ref = morphlib.git.rev_parse(self.app.runcmd, repo_dir, + 'origin/' + ref) try: text = self.app.runcmd(['git', 'cat-file', 'blob', '%s:%s.morph' % (ref, name)], |