diff options
| author | Markus Lehtonen <markus.lehtonen@linux.intel.com> | 2017-09-15 15:54:51 +0300 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2017-09-18 11:07:30 +0100 |
| commit | a7461ad2a2f06e9cdeead9621ab3c7048ce14227 (patch) | |
| tree | 1e881a844625229b2b5264d355d48d296de3f25a /scripts | |
| parent | 062bdb044cc2517256e02daf012415c51fb4d430 (diff) | |
| download | poky-a7461ad2a2f06e9cdeead9621ab3c7048ce14227.tar.gz | |
scripts/oe-build-perf-report: fix handling of --history-length
Don't crash if 'left' revision is older than the range of commits
specified with '--history-length'. In this case the 'left' revision
takes precedence.
(From OE-Core rev: cbeb1fb27329f8eba4d779d22fcad56f0fb03947)
Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>
Signed-off-by: Ross Burton <ross.burton@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'scripts')
| -rwxr-xr-x | scripts/oe-build-perf-report | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/scripts/oe-build-perf-report b/scripts/oe-build-perf-report index 3a76ab621d..defd930b04 100755 --- a/scripts/oe-build-perf-report +++ b/scripts/oe-build-perf-report | |||
| @@ -563,7 +563,7 @@ def main(argv=None): | |||
| 563 | xml = is_xml_format(repo, revs[index_r].tags[-1]) | 563 | xml = is_xml_format(repo, revs[index_r].tags[-1]) |
| 564 | 564 | ||
| 565 | if args.html: | 565 | if args.html: |
| 566 | index_0 = max(0, index_r - args.history_length) | 566 | index_0 = max(0, min(index_l, index_r - args.history_length)) |
| 567 | rev_range = range(index_0, index_r + 1) | 567 | rev_range = range(index_0, index_r + 1) |
| 568 | else: | 568 | else: |
| 569 | # We do not need range of commits for text report (no graphs) | 569 | # We do not need range of commits for text report (no graphs) |