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

(From OE-Core rev: f3afd2c47f4c740df52dfd80e208ce721d5ebf6e)

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
This commit is contained in:
Markus Lehtonen 2018-03-15 14:40:45 +02:00 committed by Richard Purdie
parent 0fd6d0dd00
commit dec7f4f17d

View File

@ -639,10 +639,6 @@ def main(argv=None):
data.append(AggregateTestData(aggregate_metadata(raw_m), data.append(AggregateTestData(aggregate_metadata(raw_m),
aggregate_data(raw_d))) 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 # Read buildstats only when needed
buildstats = None buildstats = None
if args.dump_buildstats or args.html: if args.dump_buildstats or args.html:
@ -653,10 +649,11 @@ def main(argv=None):
# Print report # Print report
if not args.html: if not args.html:
print_diff_report(data[index_l].metadata, data[index_l].results, print_diff_report(data[0].metadata, data[0].results,
data[index_r].metadata, data[index_r].results) data[1].metadata, data[1].results)
else: 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 return 0