Fix 'repo sync' rebase logic on a published branch
If the current branch is published, but all published commits are
merged into the manifest revision, but there is also at least one
unpublished commit on the current branch, we should rebase the
unpublished commit, rather than creating a merge commit.
Signed-off-by: Shawn O. Pearce <sop@google.com>
diff --git a/project.py b/project.py
index 304480a..adb72b2 100644
--- a/project.py
+++ b/project.py
@@ -682,18 +682,15 @@
branch.name,
len(upstream_gain))
return
- elif upstream_gain:
- # We can fast-forward safely.
+ elif pub == head:
+ # All published commits are merged, and thus we are a
+ # strict subset. We can fast-forward safely.
#
def _doff():
self._FastForward(rev)
self._CopyFiles()
syncbuf.later1(self, _doff)
return
- else:
- # Trivially no changes in the upstream.
- #
- return
if merge == rev:
try: