From 7e0da185272d4e690183e9946d5a8e02fa495055 Mon Sep 17 00:00:00 2001 From: Jim Meyering Date: Thu, 21 Aug 2008 19:31:55 +0000 Subject: [PATCH] autobuild.sh: Fix minor shell-quoting bugs. * autobuild.sh: Remove unnecessary quotes. Don't choke on a file name argument containing a space. Don't misbehave for $AUTOBUILD_INSTALL_ROOT containing a shell meta-character. --- ChangeLog | 8 ++++++++ autobuild.sh | 8 ++++---- 2 files changed, 12 insertions(+), 4 deletions(-) diff --git a/ChangeLog b/ChangeLog index 7a2e1e7034..765c6d2bfe 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,11 @@ +Thu Aug 21 21:29:28 CEST 2008 Jim Meyering + + autobuild.sh: Fix minor shell-quoting bugs. + * autobuild.sh: Remove unnecessary quotes. + Don't choke on a file name argument containing a space. + Don't misbehave for $AUTOBUILD_INSTALL_ROOT containing + a shell meta-character. + Wed Aug 21 11:14:09 BST 2008 Daniel P. Berrange * src/remote_internal.c: Clarify error messages from various diff --git a/autobuild.sh b/autobuild.sh index ce1269258f..7ae5d1e76a 100755 --- a/autobuild.sh +++ b/autobuild.sh @@ -5,7 +5,7 @@ set -v # Make things clean. -test -n "$1" && RESULTS="$1" || RESULTS="results.log" +test -n "$1" && RESULTS=$1 || RESULTS=results.log test -f Makefile && make -k distclean || : rm -rf coverage @@ -14,7 +14,7 @@ rm -rf coverage #mkdir build #cd build -./autogen.sh --prefix=$AUTOBUILD_INSTALL_ROOT \ +./autogen.sh --prefix="$AUTOBUILD_INSTALL_ROOT" \ --enable-test-coverage \ --enable-compile-warnings=error \ --with-openvz \ @@ -25,8 +25,8 @@ make make install set -o pipefail -make check 2>&1 | tee $RESULTS -make syntax-check 2>&1 | tee -a $RESULTS +make check 2>&1 | tee "$RESULTS" +make syntax-check 2>&1 | tee -a "$RESULTS" test -x /usr/bin/lcov && make cov rm -f *.tar.gz