]> code.ossystems Code Review - openembedded-core.git/commitdiff
build-perf-test-wrapper.sh: fix handling of -C argument
authorMarkus Lehtonen <markus.lehtonen@linux.intel.com>
Fri, 2 Sep 2016 13:37:25 +0000 (16:37 +0300)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Mon, 5 Sep 2016 10:55:51 +0000 (11:55 +0100)
Not specifying -C caused oe-build-perf-test to try to commit results to
the build directory.

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
scripts/contrib/build-perf-test-wrapper.sh

index efc2b79d6ce005e016f5f26158ac11fe225bcd7a..d61e4389331a442b4b5479f39bee8bb13221fc47 100755 (executable)
@@ -37,7 +37,6 @@ EOF
 # Parse command line arguments
 archive_dir=~/perf-results/archives
 commitish=""
-results_repo=""
 while getopts "ha:c:C:w:" opt; do
     case $opt in
         h)  usage
@@ -48,6 +47,7 @@ while getopts "ha:c:C:w:" opt; do
         c)  commitish=$OPTARG
             ;;
         C)  results_repo=`realpath "$OPTARG"`
+            commit_results=("--commit-results" "$results_repo")
             ;;
         w)  base_dir=`realpath "$OPTARG"`
             ;;
@@ -110,7 +110,7 @@ fi
 if ! oe-build-perf-test --out-dir "$results_dir" \
                         --globalres-file "$globalres_log" \
                         --lock-file "$base_dir/oe-build-perf.lock" \
-                        --commit-results "$results_repo" \
+                        "${commit_results[@]}" \
                         --commit-results-branch "{tester_host}/{git_branch}/$machine" \
                         --commit-results-tag "{tester_host}/{git_branch}/$machine/{git_commit_count}-g{git_commit}/{tag_num}"; then
     echo "oe-build-perf-test script failed!"