From 8fdb9e22a8eb3cca928b3f44d60a9b710b291064 Mon Sep 17 00:00:00 2001 From: Martin Steigerwald Date: Mon, 11 Mar 2013 16:22:56 +0100 Subject: [PATCH] Make test for gnuplot work with empty strings. MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit When I launch fio_generate_plots on a system where gnuplot is not installed I get this error : $ fio_generate_plots test Making bw logs /usr/bin/fio_generate_plots: 42: /usr/bin/fio_generate_plots: -: not found That's because the test checking whether gnuplot is installed is failing because of an empty variable : GNUPLOT=$(which gnuplot) if [ ! -x $GNUPLOT ]; then echo You need gnuplot installed to generate graphs exit 1 fi Indeed the command "test -x" is exiting with code 0. To correct this we should enclose the variable with quotes : if [ ! -x "$GNUPLOT" ]; then Then the script is going into the test properly : $ fio_generate_plots test You need gnuplot installed to generate graphs I copied problem description by Hervé from the Debian bug report below. Fixes: Bug#700580: /usr/bin/fio_generate_plots: -: not found http://bugs.debian.org/700580 Reported-By: Hervé Werner Tested-By: Hervé Werner Tested-By: Martin Steigerwald Signed-off-by: Jens Axboe --- fio_generate_plots | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fio_generate_plots b/fio_generate_plots index 4285415e..5e2febda 100755 --- a/fio_generate_plots +++ b/fio_generate_plots @@ -8,7 +8,7 @@ if [ "$1"x = "x" ]; then fi GNUPLOT=$(which gnuplot) -if [ ! -x $GNUPLOT ]; then +if [ ! -x "$GNUPLOT" ]; then echo You need gnuplot installed to generate graphs exit 1 fi -- 2.25.1