[Merge] ~codyshepherd/livecd-rootfs/+git/livecd-rootfs:cpc-ubuntu-server into livecd-rootfs:ubuntu/xenial
Steve Langasek
steve.langasek at canonical.com
Wed Sep 16 18:18:06 UTC 2020
Review: Needs Fixing
Diff comments:
> diff --git a/live-build/ubuntu-cpc/hooks.d/chroot/999-cpc-fixes.chroot b/live-build/ubuntu-cpc/hooks.d/chroot/999-cpc-fixes.chroot
> index 56b6950..b5a8ecd 100755
> --- a/live-build/ubuntu-cpc/hooks.d/chroot/999-cpc-fixes.chroot
> +++ b/live-build/ubuntu-cpc/hooks.d/chroot/999-cpc-fixes.chroot
> @@ -166,6 +166,10 @@ if [ "${SUBPROJECT:-}" = minimized ]; then
> _xchroot "${rootd}" apt clean
> fi
>
> +# install grub-legacy-ec2 and ubuntu-server for all ubuntu-cpc for LP: #1888575
> +_xchroot "${rootd}" env DEBIAN_FRONTEND=noninteractive \
> + apt-get -y install --no-install-recommends grub-legacy-ec2 ubuntu-server
> +
Shouldn't ubuntu-server already be installed prior to this, so no need to reinstall?
Also I don't think use of --no-install-recommends is correct here. CPC images use recommends in general; structurally, it's the right thing to do for these packages also. In practice, grub-legacy-ec2 has no recommends, and ubuntu-server being already installed means its recommends are also already installed, so presently --no-install-recommends should make 0 difference to what's installed; but it's architecturally more correct not to use it.
> #### END COMMON ARCH FUNCTIONS
>
>
--
https://code.launchpad.net/~codyshepherd/livecd-rootfs/+git/livecd-rootfs/+merge/390849
Your team Ubuntu Core Development Team is subscribed to branch livecd-rootfs:ubuntu/xenial.
More information about the Ubuntu-reviews
mailing list