Merge pull request #11074 from badone/wip-dry-run

run-*make-check.sh: Make DRY_RUN actually mean a dry run Reviewed-by: Kefu Chai <kchai@redhat.com>
ceph · Sep 14, 2016 · 0492c23 · 0492c23
2 parents 52b0689 + abe9144
commit 0492c23
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