From: Mark Wooding <mdw@distorted.org.uk>
Date: Sat, 13 Feb 2016 11:09:31 +0000 (+0000)
Subject: bin/mdw-sbuild: Introduce a new	`$signp' variable.
X-Git-Url: https://www.chiark.greenend.org.uk/ucgi/~mdw/git/profile/commitdiff_plain/abfec7033192f94061714f1d9928c99f17d87ec1?ds=sidebyside

bin/mdw-sbuild: Introduce a new	`$signp' variable.

This ends up being rather cleaner than messing about with `$noactp' and
whether `$keyid' is defined.
---

diff --git a/bin/mdw-sbuild b/bin/mdw-sbuild
index 11c6c16..be687f6 100755
--- a/bin/mdw-sbuild
+++ b/bin/mdw-sbuild
@@ -33,8 +33,8 @@ fail_usage () { usage >&2; exit 1; }
 ###--------------------------------------------------------------------------
 ### Parse options.
 
-bogusp=nil noactp=nil
-unset buildopts keyid
+bogusp=nil noactp=nil signp=nil
+unset buildopts
 while getopts "haik:nt:" opt; do
   case $opt in
     h)
@@ -100,6 +100,6 @@ rsync -a "$host:$builddir/" ./
 case $rc in 0) ;; *) exit $rc ;; esac
 
 ## Maybe sign the result.
-case $?,${keyid+t},$noactp in
-  0,t,nil) debsign -k"$keyid" "${pkg}_${ver}_"*.changes ;;
+case $signp in
+  t) debsign -k"$keyid" "${pkg}_${ver}_"*.changes ;;
 esac