public inbox for openembedded-core@lists.openembedded.org
 help / color / mirror / Atom feed
* [PATCH] build-perf-test-wrapper.sh: fix handling of -C argument
@ 2016-09-02 13:37 Markus Lehtonen
  0 siblings, 0 replies; only message in thread
From: Markus Lehtonen @ 2016-09-02 13:37 UTC (permalink / raw)
  To: openembedded-core

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>
---
 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



^ permalink raw reply related	[flat|nested] only message in thread

only message in thread, other threads:[~2016-09-02 13:37 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-09-02 13:37 [PATCH] build-perf-test-wrapper.sh: fix handling of -C argument Markus Lehtonen

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox