aboutsummaryrefslogtreecommitdiffstats
path: root/scripts
diff options
context:
space:
mode:
authorMarkus Lehtonen <markus.lehtonen@linux.intel.com>2018-03-15 14:40:45 +0200
committerRichard Purdie <richard.purdie@linuxfoundation.org>2018-03-16 03:42:03 -0700
commitf3afd2c47f4c740df52dfd80e208ce721d5ebf6e (patch)
tree840c571de546dff1ed6f93ef583fa75964090d47 /scripts
parent3c14c6612c48d46677837f5f4c31cfd1a6ff6174 (diff)
downloadopenembedded-core-f3afd2c47f4c740df52dfd80e208ce721d5ebf6e.tar.gz
openembedded-core-f3afd2c47f4c740df52dfd80e208ce721d5ebf6e.tar.bz2
openembedded-core-f3afd2c47f4c740df52dfd80e208ce721d5ebf6e.zip
scripts/oe-build-perf-report: fix comparing arbitrary commits
Fix a crash when generating a txt report and the two commits to be compared were not consecutive (but there were some tested commits between them). Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'scripts')
-rwxr-xr-xscripts/oe-build-perf-report11
1 files changed, 4 insertions, 7 deletions
diff --git a/scripts/oe-build-perf-report b/scripts/oe-build-perf-report
index ac88f0fce5..dc999c45c1 100755
--- a/scripts/oe-build-perf-report
+++ b/scripts/oe-build-perf-report
@@ -639,10 +639,6 @@ def main(argv=None):
data.append(AggregateTestData(aggregate_metadata(raw_m),
aggregate_data(raw_d)))
- # Re-map list indexes to the new table starting from index 0
- index_r = index_r - index_0
- index_l = index_l - index_0
-
# Read buildstats only when needed
buildstats = None
if args.dump_buildstats or args.html:
@@ -653,10 +649,11 @@ def main(argv=None):
# Print report
if not args.html:
- print_diff_report(data[index_l].metadata, data[index_l].results,
- data[index_r].metadata, data[index_r].results)
+ print_diff_report(data[0].metadata, data[0].results,
+ data[1].metadata, data[1].results)
else:
- print_html_report(data, index_l, buildstats)
+ # Re-map 'left' list index to the data table where index_0 maps to 0
+ print_html_report(data, index_l - index_0, buildstats)
return 0