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 <bhubbard@redhat.com>
ceph · Sep 14, 2016 · abe9144 · abe9144
1 parent 6a3e5c4
commit abe9144
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git 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
@@ -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