chiark / gitweb /
dgit: When propagating pseudomerge error, strip leading \n
authorIan Jackson <ijackson@chiark.greenend.org.uk>
Sat, 15 Jun 2019 15:13:14 +0000 (16:13 +0100)
committerIan Jackson <ijackson@chiark.greenend.org.uk>
Fri, 28 Jun 2019 11:45:38 +0000 (12:45 +0100)
fail adds an initial \n to provide a blank line separating the error
message from previous output.  We want to strip any such thing away
again.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
dgit

diff --git a/dgit b/dgit
index 769e48ee6ee50c7c7924cd9f0bbec02cec3d5840..f0a1e76cfad8f7fba10dfe48f9221c6c6730b7d8 100755 (executable)
--- a/dgit
+++ b/dgit
@@ -4134,6 +4134,7 @@ sub pseudomerge_version_check ($$) {
                $cd = $gf->('Distribution');
            };
            if ($@) {
+                $@ =~ s/^\n//s;
                $@ =~ s/^dgit: //gm;
                fail "$@".
                    f_ "Perhaps debian/changelog does not mention %s ?", $v;
@@ -6786,7 +6787,7 @@ sub build_source {
        unlink "$buildproductsdir/$sourcechanges" or $!==ENOENT
            or fail f_ "remove %s: %s", $sourcechanges, $!;
     }
-    confess unless !!$made_split_brain == !!$do_split_brain;
+#    confess unless !!$made_split_brain == !!$do_split_brain;
 
     my @cmd = (@dpkgsource, qw(-b --));
     my $leafdir;