From b088a2a398b660f125ab01ae1352c8a094141f51 Mon Sep 17 00:00:00 2001
From: Ian Jackson <ijackson@chiark.greenend.org.uk>
Date: Thu, 6 Jun 2019 16:28:57 +0100
Subject: [PATCH 1/1] dgit: split brain reorg: Rename $made_split_brain

This variable is not technnically necessary any more.  But tracking
this means if we introduce bugs which mean that we didn't do the split
brain work, we will get much saner misbehaviour: a crash.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
---
 dgit | 22 +++++++++++-----------
 1 file changed, 11 insertions(+), 11 deletions(-)

diff --git a/dgit b/dgit
index 2b1a2930..7808fd82 100755
--- a/dgit
+++ b/dgit
@@ -175,7 +175,7 @@ our $keyid;
 autoflush STDOUT 1;
 
 our $supplementary_message = '';
-our $split_brain = 0;
+our $made_split_brain = 0;
 our $do_split_brain = 0;
 
 # Interactions between quilt mode and split brain
@@ -4531,7 +4531,7 @@ END
 	}
     }
     if ($do_split_brain) {
-	$split_brain = 1;
+	$made_split_brain = 1;
 	$dgithead = splitbrain_pseudomerge($clogp,
 					   $actualhead, $dgithead,
 					   $archive_hash);
@@ -4578,8 +4578,8 @@ END
     my $r = system @diffcmd;
     if ($r) {
 	if ($r==256) {
-	    confess unless !!$split_brain == !!$do_split_brain;
-	    my $referent = $split_brain ? $dgithead : 'HEAD';
+	    confess unless !!$made_split_brain == !!$do_split_brain;
+	    my $referent = $made_split_brain ? $dgithead : 'HEAD';
 	    my $diffs = cmdoutput @git, qw(diff --stat), $tree, $dgithead;
 
 	    my @mode_changes;
@@ -5837,8 +5837,8 @@ sub build_maybe_quilt_fixup () {
 	# remain dgit-view, rather than master, so subsequent calls to
 	#  unpack_playtree_need_cd_work
 	# will DTRT, resetting dgit-view.
-	die if $split_brain;
-	$split_brain = 1;
+	die if $made_split_brain;
+	$made_split_brain = 1;
     }
     chdir '..';
 
@@ -6064,7 +6064,7 @@ sub quilt_check_splitbrain_cache ($$) {
 	if ($cachehit ne $headref) {
 	    progress f_ "dgit view: found cached (%s)", $saved;
 	    runcmd @git, qw(checkout -q -b dgit-view), $cachehit;
-	    $split_brain = 1;
+	    $made_split_brain = 1;
 	    return ($cachehit, $splitbrain_cachekey);
 	}
 	progress __ "dgit view: found cached, no changes required";
@@ -6760,7 +6760,7 @@ sub building_source_in_playtree {
     #
     # Note that if we are building a source package in split brain
     # mode we do not support including uncommitted changes, because
-    # that makes quilt fixup too hard.  I.e. ($split_brain && (dgit is
+    # that makes quilt fixup too hard.  I.e. ($made_split_brain && (dgit is
     # building a source package)) => !$includedirty
     return !$includedirty;
 }
@@ -6779,10 +6779,10 @@ 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;
+	confess unless !!$made_split_brain == !!$do_split_brain;
+        prep_ud() unless $made_split_brain;
         changedir $playground;
-        unless ($split_brain) {
+        unless ($made_split_brain) {
             my $upstreamversion = upstreamversion $version;
             unpack_playtree_linkorigs($upstreamversion, sub { });
             unpack_playtree_need_cd_work($headref);
-- 
2.30.2