Merge ~sespiros/ubuntu-security-tools/+git/ubuntu-security-tools:esm-chroot-names-2 into ubuntu-security-tools:master

Proposed by Spyros Seimenis
Status: Merged
Merged at revision: 311820e01773d10d69f9749c25ae3ced8b7539f7
Proposed branch: ~sespiros/ubuntu-security-tools/+git/ubuntu-security-tools:esm-chroot-names-2
Merge into: ubuntu-security-tools:master
Diff against target: 25 lines (+10/-2)
1 file modified
build-tools/umt (+10/-2)
Reviewer Review Type Date Requested Status
Alex Murray Approve
Review via email: mp+424692@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Spyros Seimenis (sespiros) wrote :
Revision history for this message
Alex Murray (alexmurray) wrote :

This looks like a good usability improvement since in general all packages published to ESM have 'esm' in the version and so should be built in ESM chroots.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/build-tools/umt b/build-tools/umt
index 24beb0b..a2b8f9f 100755
--- a/build-tools/umt
+++ b/build-tools/umt
@@ -637,10 +637,18 @@ def perform_binary_build(details, opt):
637 if opt.hardening:637 if opt.hardening:
638 extra_deps.append('hardening-wrapper')638 extra_deps.append('hardening-wrapper')
639639
640 details['chroot'] = "%s-i386" % details['release']640 chroot = details['release']
641
642 if 'esm' in details['version']:
643 chroot += "-esm"
644
641 (rc, arch) = runcmd(['uname', '-m'])645 (rc, arch) = runcmd(['uname', '-m'])
642 if arch.strip() == "x86_64":646 if arch.strip() == "x86_64":
643 details['chroot'] = "%s-amd64" % details['release']647 chroot += "-amd64"
648 else:
649 chroot += "-i386"
650
651 details['chroot'] = chroot
644652
645 if opt.chroot:653 if opt.chroot:
646 details['chroot'] = opt.chroot654 details['chroot'] = opt.chroot

Subscribers

People subscribed via source and target branches