Commit ad48e93b authored by Mattia Rizzolo's avatar Mattia Rizzolo Committed by Holger Levsen

reproducible debian: simplify conditions

* at that point we already know that SAVE_ARTIFACTS=0 (well we know != 1, but
  0 is the only other value)
* avoid nested if while still checkin for the same value

pondering a `case` statement here…
Signed-off-by: Mattia Rizzolo's avatarMattia Rizzolo <mattia@debian.org>
Signed-off-by: Holger Levsen's avatarHolger Levsen <holger@layer-acht.org>
parent 5649bf35
......@@ -101,17 +101,16 @@ save_artifacts() {
cleanup_all() {
echo "Starting cleanup."
if [ "$SAVE_ARTIFACTS" = "1" ] ; then
save_artifacts
elif [ "$NOTIFY" = "2" ] && [ "$SAVE_ARTIFACTS" = "0" ] ; then
save_artifacts # this will also notify IRC as needed
elif [ "$NOTIFY" = "2" ] ; then
irc_message debian-reproducible "$DEBIAN_URL/$SUITE/$ARCH/$SRCPACKAGE done: $STATUS debug: $NOTIFY"
elif [ "$NOTIFY" = "1" ] && [ "$SAVE_ARTIFACTS" = "0" ] ; then
elif [ "$NOTIFY" = "1" ] ; then
irc_message debian-reproducible "$DEBIAN_URL/$SUITE/$ARCH/$SRCPACKAGE done: $STATUS"
elif [ ! -z "$NOTIFY" ] && [ "$SAVE_ARTIFACTS" = "0" ] ; then
if [ "$NOTIFY" = "diffoscope" ] ; then
elif [ "$NOTIFY" = "diffoscope" ] ; then
irc_message debian-reproducible-changes "$DEBIAN_URL/$SUITE/$ARCH/$SRCPACKAGE $STATUS and $DIFFOSCOPE failed"
else
irc_message debian-reproducible-changes "$DEBIAN_URL/$SUITE/$ARCH/$SRCPACKAGE done: $STATUS debug: $NOTIFY"
fi
elif [ ! -z "$NOTIFY" ] ; then
# a weird value of $NOTIFY that we don't know about
irc_message debian-reproducible-changes "$DEBIAN_URL/$SUITE/$ARCH/$SRCPACKAGE done: $STATUS debug: $NOTIFY"
fi
[ ! -f $RBUILDLOG ] || gzip -9fvn $RBUILDLOG
if [ "$MODE" = "master" ] ; then
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment