summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMarkus Lehtonen <markus.lehtonen@linux.intel.com>2016-09-02 16:37:25 +0300
committerRichard Purdie <richard.purdie@linuxfoundation.org>2016-09-05 11:55:51 +0100
commit6f4786f5522c366a7fd92f630c3f32629a9f9471 (patch)
tree5444fc68df0a337f784b5abfb7c31762aeaaa437
parent1cb0eb9a013ad8a4092f610faeab2ee2720b9e66 (diff)
downloadopenembedded-core-6f4786f5522c366a7fd92f630c3f32629a9f9471.tar.gz
openembedded-core-6f4786f5522c366a7fd92f630c3f32629a9f9471.tar.bz2
openembedded-core-6f4786f5522c366a7fd92f630c3f32629a9f9471.zip
build-perf-test-wrapper.sh: fix handling of -C argument
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>
-rwxr-xr-xscripts/contrib/build-perf-test-wrapper.sh4
1 files changed, 2 insertions, 2 deletions
diff --git a/scripts/contrib/build-perf-test-wrapper.sh b/scripts/contrib/build-perf-test-wrapper.sh
index efc2b79d6c..d61e438933 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!"