summaryrefslogtreecommitdiff
path: root/dgit
diff options
context:
space:
mode:
authorIan Jackson <ijackson@chiark.greenend.org.uk>2019-06-06 16:23:43 +0100
committerIan Jackson <ijackson@chiark.greenend.org.uk>2019-06-28 12:45:38 +0100
commitb4195a2465d892f7bf79fba55f169255a2768ada (patch)
tree85ec01b2464f8cd0dceb9aaedb700cc3da9564c8 /dgit
parent20d1d3d7ab35acd079f0ccf4127dcd74cb0265ac (diff)
dgit: split brain reorg: Check $split_brain against $do_split_brain
I think by now, $do_split_brain is always set early enough that $split_brain here implies $do_split_brain. And if $split_brain were not set but $do_split_brain was, that would mean we hadn't actually done the necessary work (constructing the and switching to the dgit-view branch in the playtree or computing $dgithead). Double check this with a couple of calls to confess. Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
Diffstat (limited to 'dgit')
-rwxr-xr-xdgit2
1 files changed, 2 insertions, 0 deletions
diff --git a/dgit b/dgit
index 4bc088e..2b1a293 100755
--- a/dgit
+++ b/dgit
@@ -4578,6 +4578,7 @@ END
my $r = system @diffcmd;
if ($r) {
if ($r==256) {
+ confess unless !!$split_brain == !!$do_split_brain;
my $referent = $split_brain ? $dgithead : 'HEAD';
my $diffs = cmdoutput @git, qw(diff --stat), $tree, $dgithead;
@@ -6778,6 +6779,7 @@ sub build_source {
# If we are in split brain, there is already a playtree with
# the thing we should package into a .dsc (thanks to quilt
# fixup). If not, make a playtree
+ confess unless !!$split_brain == !!$do_split_brain;
prep_ud() unless $split_brain;
changedir $playground;
unless ($split_brain) {