diff options
| author | Markus Lehtonen <markus.lehtonen@linux.intel.com> | 2016-09-02 16:37:25 +0300 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2016-09-05 11:55:51 +0100 |
| commit | 6f4786f5522c366a7fd92f630c3f32629a9f9471 (patch) | |
| tree | 5444fc68df0a337f784b5abfb7c31762aeaaa437 | |
| parent | 1cb0eb9a013ad8a4092f610faeab2ee2720b9e66 (diff) | |
| download | openembedded-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-x | scripts/contrib/build-perf-test-wrapper.sh | 4 |
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!" |