Merge lp:~smoser/maas-images/trunk.lp1640519 into lp:maas-images

Proposed by Scott Moser
Status: Merged
Merged at revision: 350
Proposed branch: lp:~smoser/maas-images/trunk.lp1640519
Merge into: lp:maas-images
Diff against target: 24 lines (+11/-3)
1 file modified
bin/maas-cloudimg2ephemeral (+11/-3)
To merge this branch: bzr merge lp:~smoser/maas-images/trunk.lp1640519
Reviewer Review Type Date Requested Status
Lee Trager (community) Approve
Review via email: mp+312064@code.launchpad.net

Commit message

Never add u-boot-tools to image on zesty or newer.

u-boot-tools was added to arm64 and armhf images as a crutch in order to
aid curtin in installation of older releases. In newer curtin this is
fixed by dynamic installation similar to how grub is handled.

See LP: #1640519 for more information.

To post a comment you must log in.
Revision history for this message
Lee Trager (ltrager) wrote :

LGTM!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'bin/maas-cloudimg2ephemeral'
--- bin/maas-cloudimg2ephemeral 2016-09-23 19:00:56 +0000
+++ bin/maas-cloudimg2ephemeral 2016-11-29 16:19:53 +0000
@@ -297,9 +297,17 @@
297 rel_pkgs="${rel_pkgs} maas-enlist";;297 rel_pkgs="${rel_pkgs} maas-enlist";;
298 esac298 esac
299299
300 case "$arch" in300 change="add u-boot-tools to arch_pkgs for arch '$arch' (LP: #1640519)"
301 arm64|armhf) arch_pkgs="u-boot-tools";;301 if [ "$arch" = "arm64" -o "$arch" = "armhf" ]; then
302 esac302 case "$release" in
303 precise|quantal|raring|saucy|trusty|utopic|vivid|wily|\
304 xenial|yakkety)
305 arch_pkgs="${arch_pkgs} u-boot-tools"
306 debug 1 "release '$release' <= yakkety: $change"
307 ;;
308 *) debug 1 "release '$release' > yakkety: do not $change";;
309 esac
310 fi
303311
304 local removes="^linux-.*" keeps="^linux-base$" pkglist=""312 local removes="^linux-.*" keeps="^linux-base$" pkglist=""
305 pkglist=$(dpkg-query --show) || { error "failed dpkg-query"; return 1; }313 pkglist=$(dpkg-query --show) || { error "failed dpkg-query"; return 1; }

Subscribers

People subscribed via source and target branches