Merge ~sylvain-pineau/checkbox/+git/support:fixes into ~checkbox-dev/checkbox/+git/support:master

Proposed by Sylvain Pineau
Status: Merged
Approved by: Sylvain Pineau
Approved revision: e61cb7ba189bd0bad7b7125116a92216b743173f
Merged at revision: b7d64381879b380b9243bd50da4ccb8f356e099f
Proposed branch: ~sylvain-pineau/checkbox/+git/support:fixes
Merge into: ~checkbox-dev/checkbox/+git/support:master
Diff against target: 61 lines (+20/-11)
2 files modified
release/git/lp-roadmap-query (+12/-10)
release/git/release_me.py (+8/-1)
Reviewer Review Type Date Requested Status
Sylvain Pineau (community) Approve
Review via email: mp+316347@code.launchpad.net

Description of the change

Fixes for the release tools

To post a comment you must log in.
Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

self-approved

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/release/git/lp-roadmap-query b/release/git/lp-roadmap-query
2index e76c3b8..7b59730 100755
3--- a/release/git/lp-roadmap-query
4+++ b/release/git/lp-roadmap-query
5@@ -50,14 +50,16 @@ def main():
6 packages = {
7 'checkbox-support': 'Checkbox Support Library',
8 'plainbox': 'PlainBox (Toolkit)',
9- 'plainbox-provider-certification-client': 'Canonical Certification '
10- 'Client Provider '
11- 'for PlainBox',
12- 'plainbox-provider-certification-server': 'Canonical Certification '
13- 'Server Provider '
14- 'for PlainBox',
15- 'plainbox-provider-checkbox': 'Checkbox Provider for PlainBox',
16- 'plainbox-provider-resource': 'Resource Provider for PlainBox',
17+ 'plainbox-provider-certification-client': 'Provider for Plainbox - '
18+ 'Canonical Certification '
19+ 'Client',
20+ 'plainbox-provider-certification-server': 'Provider for Plainbox - '
21+ 'Canonical Certification '
22+ 'Server',
23+ 'plainbox-provider-checkbox': 'Provider for Plainbox - Checkbox',
24+ 'plainbox-provider-resource': 'Provider for Plainbox - Resource',
25+ 'plainbox-provider-sru': 'Provider for Plainbox - SRU',
26+ 'plainbox-provider-tpm2': 'Provider for Plainbox - TPM2',
27 'checkbox-ng': 'Next Generation Checkbox (CLI)',
28 'checkbox-converged': 'Checkbox Converged',
29 }
30@@ -81,8 +83,8 @@ def main():
31 for m in sorted(matching_milestone, key=lambda m: m.date_targeted):
32 if m.date_targeted.isoformat() not in milestone_codes:
33 milestone_codes[m.date_targeted.isoformat()] = ""
34- mi_codes = [mi["value"] for mi in milestones if "{} {}".format(
35- packages[p], m.name) in mi["id"]]
36+ mi_codes = [mi["value"] for mi in milestones if re.search(
37+ "{}\s+{}".format(packages[p], m.name), mi["id"])]
38 if mi_codes:
39 milestone_codes[m.date_targeted.isoformat()] += \
40 "&field.milestone%3Alist={}".format(mi_codes[-1])
41diff --git a/release/git/release_me.py b/release/git/release_me.py
42index 0da3075..4e0149e 100755
43--- a/release/git/release_me.py
44+++ b/release/git/release_me.py
45@@ -238,8 +238,15 @@ class Release(Command):
46 version_pattern = '*[^c][0-9]' # Up to 9 RC :)
47 code_change = run(
48 'git diff $(git describe --abbrev=0 --tags --match '
49- '"v{}") --name-only'.format(version_pattern),
50+ '"v{}") -- . ":(exclude).*ignore"'.format(version_pattern),
51 shell=True, cwd=project, check=True).stdout
52+ version_change = run(
53+ 'git diff $(git describe --abbrev=0 --tags --match "v{}")'
54+ ' -G "__version__|current_version|version="'.format(
55+ version_pattern),
56+ shell=True, cwd=project, check=True).stdout
57+ if code_change == version_change:
58+ code_change = False
59 packaging_change = run(
60 'git diff $(git describe --abbrev=0 --tags --match '
61 '"debian-{}") --name-only'.format(version_pattern),

Subscribers

People subscribed via source and target branches