[OE-core] [PATCH] build-perf-test-wrapper.sh: fix handling of -C argument

Markus Lehtonen markus.lehtonen at linux.intel.com
Fri Sep 2 13:37:25 UTC 2016


Not specifying -C caused oe-build-perf-test to try to commit results to
the build directory.

Signed-off-by: Markus Lehtonen <markus.lehtonen at linux.intel.com>
---
 scripts/contrib/build-perf-test-wrapper.sh | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/scripts/contrib/build-perf-test-wrapper.sh b/scripts/contrib/build-perf-test-wrapper.sh
index efc2b79..d61e438 100755
--- a/scripts/contrib/build-perf-test-wrapper.sh
+++ b/scripts/contrib/build-perf-test-wrapper.sh
@@ -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!"
-- 
2.6.6




More information about the Openembedded-core mailing list