diff options
author | Richard Purdie <richard.purdie@linuxfoundation.org> | 2020-10-09 15:08:53 +0100 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2020-10-22 12:16:42 +0100 |
commit | 7a2bbca3daf83e6982dcf569ef7fd1a068392a1e (patch) | |
tree | d2483ecffe69798fcd5896edfa66cd2e83c358ec /scripts | |
parent | 97ac0a2f7c43b3df1aca641d84af6194588d3bc6 (diff) | |
download | poky-7a2bbca3daf83e6982dcf569ef7fd1a068392a1e.tar.gz |
oe-build-perf-report: Ensure correct data is shown for multiple branch options
If --branch and --branch2 are the same, there is extra confusing data plotted
onto the graph. Only do that if the branches are actually different.
(From OE-Core rev: e2240d0a93a9a2932506a356315ad6702ecee9b6)
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit c263d810b7eb47ee90f2adeb5ab6decf7332bec2)
Signed-off-by: Steve Sakoman <steve@sakoman.com>
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 b46d61869c..7812ea4540 100755 --- a/scripts/oe-build-perf-report +++ b/scripts/oe-build-perf-report | |||
@@ -509,7 +509,7 @@ def main(argv=None): | |||
509 | 509 | ||
510 | revs = gitarchive.get_test_revs(log, repo, args.tag_name, hostname=args.hostname, | 510 | revs = gitarchive.get_test_revs(log, repo, args.tag_name, hostname=args.hostname, |
511 | branch=args.branch, machine=args.machine) | 511 | branch=args.branch, machine=args.machine) |
512 | if args.branch2: | 512 | if args.branch2 and args.branch2 != args.branch: |
513 | revs2 = gitarchive.get_test_revs(log, repo, args.tag_name, hostname=args.hostname, | 513 | revs2 = gitarchive.get_test_revs(log, repo, args.tag_name, hostname=args.hostname, |
514 | branch=args.branch2, machine=args.machine) | 514 | branch=args.branch2, machine=args.machine) |
515 | if not len(revs2): | 515 | if not len(revs2): |
@@ -533,7 +533,7 @@ def main(argv=None): | |||
533 | else: | 533 | else: |
534 | index1 = len(revs) - 1 | 534 | index1 = len(revs) - 1 |
535 | 535 | ||
536 | if args.branch2: | 536 | if args.branch2 and args.branch2 != args.branch: |
537 | revs2.append(revs[index1]) | 537 | revs2.append(revs[index1]) |
538 | index1 = len(revs2) - 1 | 538 | index1 = len(revs2) - 1 |
539 | revs = revs2 | 539 | revs = revs2 |