chiark / gitweb /
Split brain: TODOs
authorIan Jackson <ijackson@chiark.greenend.org.uk>
Sun, 31 Jul 2016 18:02:32 +0000 (19:02 +0100)
committerIan Jackson <ijackson@chiark.greenend.org.uk>
Mon, 5 Sep 2016 12:41:48 +0000 (13:41 +0100)
Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
TODO.BRANCH
dgit
tests/tests/quilt-gbp

index 7451476..b526d21 100644 (file)
@@ -13,6 +13,15 @@ Tag rename unfinished business
 
  - other tag should be [dgit Maintainer-Tag quilt=$quilt-mode]
 
+ - in dopush (?), we need to sort out synthetic fast forard
+    this should not depend entirely on quilt mode
+    but will always be needed if quiltmode_splitbrain
+
+ - there is some confusion between $split_brain (did we actually
+   generate any commits) and quiltmode_splitbrain (might we have done,
+   including in previous pushes); search for $split_brain and check
+   that use sites are correct
+
 
 Want tests for
  - start repo with old tag, push with new tag, run
diff --git a/dgit b/dgit
index 288fc78..53ccd72 100755 (executable)
--- a/dgit
+++ b/dgit
@@ -2148,8 +2148,6 @@ END
        }
     }
 
-    die 'xxx fast forward (should not depend on quilt mode, but will always be needed if we did $split_brain)' if $split_brain;
-
     check_not_dirty();
     changedir $ud;
     progress "checking that $dscfn corresponds to HEAD";
index d3624e9..da27d14 100755 (executable)
@@ -30,7 +30,8 @@ exit 0
 want to
 
  do push
- t-pushed-good
+ t-pushed-good XXX need to do something more subtle than check our branch is in repo now
+
  check that
    - we didn't update caller's branch
 
@@ -44,3 +45,8 @@ want to
   - has patches applied (reuse rules build magic)
   - has .gitignore edited
 
+make a new change
+
+push it
+
+check that the synthetic fast forward worked