-
Andrej Shadura authored
This code is unmaintainable as is, and is buggy. If run with set -e, it would not detect failures properly and fail only because test would fail with an unquoted $s argument, which would be unset because echo would fail to run after the command failed. Instead, let's just drop this horrific wizardry and use plain simple || return $? to pass the return code further on. Also, reduce the function of DEBUG to enabling all output, defaulting to disabling both stdout and stderr. The original code didn't do what it said it was doing anyway. Signed-off-by: Andrew Shadura <andrew.shadura@collabora.co.uk> Reviewed-by: Emanuele Aina <emanuele.aina@collabora.co.uk> Differential Revision: https://phabricator.apertis.org/D7506