diff options
| author | Richard Purdie <richard.purdie@linuxfoundation.org> | 2019-02-11 17:45:25 +0000 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2019-03-26 15:38:28 +0000 |
| commit | 14e86a29689b06384e7a6d91d04d558c7d231f6e (patch) | |
| tree | 930658607e9c4ef5b290e8898de3b209a1c6d097 /scripts | |
| parent | b25e586458d94bfaaa6c7baffa8480153dfdd70d (diff) | |
| download | poky-14e86a29689b06384e7a6d91d04d558c7d231f6e.tar.gz | |
oe-build-perf-report: Fix missing buildstats comparisions
Integers were being compared to strings leading to missing buildstats comparision
data. Fix this.
(From OE-Core rev: 2dbbf598192ae2b3aa488df042f56aa6c6634a00)
(From OE-Core rev: e8651b2e01db0777b23a0359f9413c3b27bae77f)
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'scripts')
| -rwxr-xr-x | scripts/oe-build-perf-report | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/scripts/oe-build-perf-report b/scripts/oe-build-perf-report index f17d932716..1e0b55100a 100755 --- a/scripts/oe-build-perf-report +++ b/scripts/oe-build-perf-report | |||
| @@ -425,8 +425,8 @@ def print_html_report(data, id_comp, buildstats): | |||
| 425 | 425 | ||
| 426 | # Compare buildstats | 426 | # Compare buildstats |
| 427 | bs_key = test + '.' + meas | 427 | bs_key = test + '.' + meas |
| 428 | rev = metadata['commit_num']['value'] | 428 | rev = str(metadata['commit_num']['value']) |
| 429 | comp_rev = metadata['commit_num']['value_old'] | 429 | comp_rev = str(metadata['commit_num']['value_old']) |
| 430 | if (rev in buildstats and bs_key in buildstats[rev] and | 430 | if (rev in buildstats and bs_key in buildstats[rev] and |
| 431 | comp_rev in buildstats and bs_key in buildstats[comp_rev]): | 431 | comp_rev in buildstats and bs_key in buildstats[comp_rev]): |
| 432 | new_meas['buildstats'] = BSSummary(buildstats[comp_rev][bs_key], | 432 | new_meas['buildstats'] = BSSummary(buildstats[comp_rev][bs_key], |