From abe914467aa1ca88e99abd4b96b814b9e4908298 Mon Sep 17 00:00:00 2001 From: Brad Hubbard Date: Wed, 14 Sep 2016 17:13:47 +1000 Subject: [PATCH] run-*make-check.sh: Make DRY_RUN actually mean a dry run DRY_RUN does not currently do an actual dry run as it actually runs an install command. Signed-off-by: Brad Hubbard --- run-cmake-check.sh | 2 +- run-make-check.sh | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/run-cmake-check.sh b/run-cmake-check.sh index 8f95eacd68cf1..1446434abb0e5 100755 --- a/run-cmake-check.sh +++ b/run-cmake-check.sh @@ -38,7 +38,7 @@ function run() { type apt-get > /dev/null 2>&1 && install_cmd="apt-get install -y" type zypper > /dev/null 2>&1 && install_cmd="zypper --gpg-auto-import-keys --non-interactive install" if [ -n "$install_cmd" ]; then - sudo $install_cmd ccache jq + $DRY_RUN sudo $install_cmd ccache jq else echo "WARNING: Don't know how to install packages" >&2 fi diff --git a/run-make-check.sh b/run-make-check.sh index 05230adfd5e35..ce860c3e5eea2 100755 --- a/run-make-check.sh +++ b/run-make-check.sh @@ -56,7 +56,7 @@ function run() { type apt-get > /dev/null 2>&1 && install_cmd="apt-get install -y" type zypper > /dev/null 2>&1 && install_cmd="zypper --gpg-auto-import-keys --non-interactive install" if [ -n "$install_cmd" ]; then - sudo $install_cmd ccache jq + $DRY_RUN sudo $install_cmd ccache jq else echo "WARNING: Don't know how to install packages" >&2 fi