bin/mdw-sbuild: Introduce a new `$signp' variable.
authorMark Wooding <mdw@distorted.org.uk>
Sat, 13 Feb 2016 11:09:31 +0000 (11:09 +0000)
committerMark Wooding <mdw@distorted.org.uk>
Sat, 13 Feb 2016 11:09:31 +0000 (11:09 +0000)
This ends up being rather cleaner than messing about with `$noactp' and
whether `$keyid' is defined.

bin/mdw-sbuild

index 11c6c16..be687f6 100755 (executable)
@@ -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