chiark / gitweb /
Do not spew diff output to terminal (by default). Print sensible message instead...
authorIan Jackson <ijackson@chiark.greenend.org.uk>
Sun, 3 Aug 2014 17:00:55 +0000 (18:00 +0100)
committerIan Jackson <ijackson@chiark.greenend.org.uk>
Sun, 3 Aug 2014 17:00:55 +0000 (18:00 +0100)
debian/changelog
dgit

index f0f2eb9..47ce6c2 100644 (file)
@@ -9,7 +9,8 @@ dgit (0.22~experimental1) experimental; urgency=low
   * Include canonicalised suite name in signed tag message.
   * Mention cross-version dgit rpush incompatibility in manpage.
   * New script tests/using-intree for running tests on the source tree.
-  * Do not spew diff output to terminal (by default).  Closes:#736526.
+  * Do not spew diff output to terminal (by default).  Print sensible
+    message instead.  Closes:#736526.
 
   Major new feature, currently stalled awaiting server infrastructure:
   * dgit-repos-server: New program for receiving signed-tag-based
diff --git a/dgit b/dgit
index 1fdca6c..843c7ec 100755 (executable)
--- a/dgit
+++ b/dgit
@@ -1395,8 +1395,9 @@ sub dopush () {
     my @diffcmd = (@git, qw(diff), $diffopt, $tree);
     printcmd \*DEBUG,$debugprefix."+",@diffcmd;
     $!=0; $?=0;
-    if (system @diffcmd) {
-       if ($! && $?==256) {
+    my $r = system @diffcmd;
+    if ($r) {
+       if ($r==256) {
            fail "$dscfn specifies a different tree to your HEAD commit;".
                " perhaps you forgot to build".
                ($diffopt eq '--exit-code' ? "" :