Merge lp:~seb128/ubuntu-archive-scripts/include-build-depends into lp:ubuntu-archive-scripts

Proposed by Sebastien Bacher
Status: Merged
Merged at revision: 289
Proposed branch: lp:~seb128/ubuntu-archive-scripts/include-build-depends
Merge into: lp:ubuntu-archive-scripts
Diff against target: 46 lines (+18/-0)
2 files modified
generate-team-p-m (+8/-0)
templates/team-report.html (+10/-0)
To merge this branch: bzr merge lp:~seb128/ubuntu-archive-scripts/include-build-depends
Reviewer Review Type Date Requested Status
Iain Lane Approve
Review via email: mp+389229@code.launchpad.net

Commit message

Include unsatisfiable Build-Depends in the team report

Description of the change

Example of the report with the change

https://people.canonical.com/~seb128/report-build-depends.html

see the firefox entry at the top (which is empty in the current report)

To post a comment you must log in.
Revision history for this message
Iain Lane (laney) wrote :

thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'generate-team-p-m'
2--- generate-team-p-m 2020-07-14 14:12:10 +0000
3+++ generate-team-p-m 2020-08-13 09:39:20 +0000
4@@ -191,6 +191,7 @@
5 waiting = attr.ib(default=None) # [(source_package_name, arches)]
6 data = attr.ib(default=None) # data for package_in_proposed
7 unsatdepends = attr.ib(default=None) # [string]
8+ unsatbuilddep = attr.ib(default=None) # [string]
9
10 _age = attr.ib(default=None)
11
12@@ -294,6 +295,13 @@
13 for p in packages:
14 unsatd[p].append(arch)
15 prob.unsatdepends = ['{}: {}'.format(p, ', '.join(sorted(arches))) for p, arches in sorted(unsatd.items())]
16+ if 'build-depends' in item['policy_info'] and 'unsatisfiable-arch-build-depends' in item['policy_info']['build-depends']:
17+ unsatdbd = defaultdict(list)
18+ for arch, packages in item['policy_info']['build-depends']['unsatisfiable-arch-build-depends'].items():
19+ for p in packages:
20+ unsatdbd[p].append(arch)
21+ prob.unsatbuilddep = ['{}: {}'.format(p, ', '.join(sorted(arches))) for p, arches in sorted(unsatdbd.items())]
22+
23
24 package_to_problems = defaultdict(list)
25
26
27=== modified file 'templates/team-report.html'
28--- templates/team-report.html 2020-06-30 15:18:07 +0000
29+++ templates/team-report.html 2020-08-13 09:39:20 +0000
30@@ -103,6 +103,16 @@
31 </ul>
32 </li>
33 {% endif %}
34+ {% if prob.unsatbuilddep %}
35+ <li>
36+ Unsatisfiable build depends:
37+ <ul>
38+ {% for unsatbd in prob.unsatbuilddep %}
39+ <li>{{ unsatbd }}</li>
40+ {% endfor %}
41+ </ul>
42+ </li>
43+ {% endif %}
44 {% if d["policy_info"]["block-bugs"] %}
45 <li>Blocked by bug:
46 {% for bug in d["policy_info"]["block-bugs"]|map("int")|sort|map("string") %}

Subscribers

People subscribed via source and target branches