From 91de11099dcd8c37e308992ccfe26dd14cee5f64 Mon Sep 17 00:00:00 2001 From: Yoshisato Yanagisawa Date: Thu, 15 Nov 2018 03:50:16 +0000 Subject: [PATCH] autoninja: fix shellcheck warnings Change-Id: Ic7ca117435de34400fdd117741d0b321339f8cfd Reviewed-on: https://chromium-review.googlesource.com/c/1325250 Commit-Queue: Yoshisato Yanagisawa Reviewed-by: Dirk Pranke --- autoninja | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/autoninja b/autoninja index b7b6d59879..2bcee51c60 100755 --- a/autoninja +++ b/autoninja @@ -5,16 +5,16 @@ # found in the LICENSE file. # Set unique build ID. -export AUTONINJA_BUILD_ID="$(python -c "import uuid; print uuid.uuid4()")" +AUTONINJA_BUILD_ID="$(python -c "import uuid; print uuid.uuid4()")" +export AUTONINJA_BUILD_ID # Execute whatever is printed by autoninja.py. # Also print it to reassure that the right settings are being used. -command=$(python $(dirname -- "$0")/autoninja.py "$@") -echo $command -eval $command -if [ $? -eq 0 ]; then +command=$(python "$(dirname -- "$0")/autoninja.py" "$@") +echo "$command" +if eval "$command"; then if [ "$NINJA_SUMMARIZE_BUILD" == "1" ]; then - python $(dirname -- "$0")/post_build_ninja_summary.py $@ + python "$(dirname -- "$0")/post_build_ninja_summary.py" "$@" fi exit fi