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
diff --git a/release/git/lp-roadmap-query b/release/git/lp-roadmap-query
index e76c3b8..7b59730 100755
--- a/release/git/lp-roadmap-query
+++ b/release/git/lp-roadmap-query
@@ -50,14 +50,16 @@ def main():
50 packages = {50 packages = {
51 'checkbox-support': 'Checkbox Support Library',51 'checkbox-support': 'Checkbox Support Library',
52 'plainbox': 'PlainBox (Toolkit)',52 'plainbox': 'PlainBox (Toolkit)',
53 'plainbox-provider-certification-client': 'Canonical Certification '53 'plainbox-provider-certification-client': 'Provider for Plainbox - '
54 'Client Provider '54 'Canonical Certification '
55 'for PlainBox',55 'Client',
56 'plainbox-provider-certification-server': 'Canonical Certification '56 'plainbox-provider-certification-server': 'Provider for Plainbox - '
57 'Server Provider '57 'Canonical Certification '
58 'for PlainBox',58 'Server',
59 'plainbox-provider-checkbox': 'Checkbox Provider for PlainBox',59 'plainbox-provider-checkbox': 'Provider for Plainbox - Checkbox',
60 'plainbox-provider-resource': 'Resource Provider for PlainBox',60 'plainbox-provider-resource': 'Provider for Plainbox - Resource',
61 'plainbox-provider-sru': 'Provider for Plainbox - SRU',
62 'plainbox-provider-tpm2': 'Provider for Plainbox - TPM2',
61 'checkbox-ng': 'Next Generation Checkbox (CLI)',63 'checkbox-ng': 'Next Generation Checkbox (CLI)',
62 'checkbox-converged': 'Checkbox Converged',64 'checkbox-converged': 'Checkbox Converged',
63 }65 }
@@ -81,8 +83,8 @@ def main():
81 for m in sorted(matching_milestone, key=lambda m: m.date_targeted):83 for m in sorted(matching_milestone, key=lambda m: m.date_targeted):
82 if m.date_targeted.isoformat() not in milestone_codes:84 if m.date_targeted.isoformat() not in milestone_codes:
83 milestone_codes[m.date_targeted.isoformat()] = ""85 milestone_codes[m.date_targeted.isoformat()] = ""
84 mi_codes = [mi["value"] for mi in milestones if "{} {}".format(86 mi_codes = [mi["value"] for mi in milestones if re.search(
85 packages[p], m.name) in mi["id"]]87 "{}\s+{}".format(packages[p], m.name), mi["id"])]
86 if mi_codes:88 if mi_codes:
87 milestone_codes[m.date_targeted.isoformat()] += \89 milestone_codes[m.date_targeted.isoformat()] += \
88 "&field.milestone%3Alist={}".format(mi_codes[-1])90 "&field.milestone%3Alist={}".format(mi_codes[-1])
diff --git a/release/git/release_me.py b/release/git/release_me.py
index 0da3075..4e0149e 100755
--- a/release/git/release_me.py
+++ b/release/git/release_me.py
@@ -238,8 +238,15 @@ class Release(Command):
238 version_pattern = '*[^c][0-9]' # Up to 9 RC :)238 version_pattern = '*[^c][0-9]' # Up to 9 RC :)
239 code_change = run(239 code_change = run(
240 'git diff $(git describe --abbrev=0 --tags --match '240 'git diff $(git describe --abbrev=0 --tags --match '
241 '"v{}") --name-only'.format(version_pattern),241 '"v{}") -- . ":(exclude).*ignore"'.format(version_pattern),
242 shell=True, cwd=project, check=True).stdout242 shell=True, cwd=project, check=True).stdout
243 version_change = run(
244 'git diff $(git describe --abbrev=0 --tags --match "v{}")'
245 ' -G "__version__|current_version|version="'.format(
246 version_pattern),
247 shell=True, cwd=project, check=True).stdout
248 if code_change == version_change:
249 code_change = False
243 packaging_change = run(250 packaging_change = run(
244 'git diff $(git describe --abbrev=0 --tags --match '251 'git diff $(git describe --abbrev=0 --tags --match '
245 '"debian-{}") --name-only'.format(version_pattern),252 '"debian-{}") --name-only'.format(version_pattern),

Subscribers

People subscribed via source and target branches