[Merge] ~philroche/livecd-rootfs/+git/livecd-rootfs:bugfix/unminimize-no-lxd-installer-package-jammy into livecd-rootfs:ubuntu/jammy
Philip Roche
mp+458910 at code.launchpad.net
Mon Jan 22 12:21:05 UTC 2024
Diff comments:
> diff --git a/live-build/ubuntu-server/hooks/01-unminimize.chroot_early b/live-build/ubuntu-server/hooks/01-unminimize.chroot_early
> index f70b59b..69fbbcc 100755
> --- a/live-build/ubuntu-server/hooks/01-unminimize.chroot_early
> +++ b/live-build/ubuntu-server/hooks/01-unminimize.chroot_early
> @@ -10,11 +10,22 @@ case ${PASS} in
> ;;
> esac
>
> -# The unminimize script will try to install the lxd snap. We can't
> -# do that at this stage so just neuter the snap command (the snap
> +# The unminimize script will try to install the lxd snap using the shim script
> +# /usr/sbin/lxd from the lxd-installer package.
> +# We can't do that at this stage so just neuter the lxd command (the snap
> # will get properly seeded by generic machinery).
> -dpkg-divert --add --divert /usr/bin/snap.REAL --rename /usr/bin/snap
> -ln -s /bin/true /usr/bin/snap
> -yes | /usr/local/sbin/unminimize
> -rm /usr/bin/snap
> -dpkg-divert --remove --rename /usr/bin/snap
> +if [[ -f "/usr/sbin/lxd" ]]; then
ack. will remove
> + rm --verbose --force /usr/sbin/lxd
understood.
> + ln -s /bin/true /usr/sbin/lxd
> + yes | /usr/local/sbin/unminimize
> + # unminimize also uninstalls lxd-installer package
> + # and also removed `/usr/sbin/lxd` as a result so we don't need to restore
> +else
> + # if /usr/sbin/lxd doesn't exist when lxd-installer package isn't installed so we can mock the command
my bad.
# if /usr/sbin/lxd doesn't exist then lxd-installer package isn't installed.
# Instead we can mock the command to avoid the unminimize script failing
is the correct comment
> + # to avoid the unminimize script failing
> + ln -s /bin/true /usr/sbin/lxd
> + yes | /usr/local/sbin/unminimize
> + # as the lxd-installer package was not installed and thus not removed the mock
> + # /usr/sbin/lxd will still be present so we need to remove it
> + rm --verbose --force /usr/sbin/lxd
understood.
> +fi
> \ No newline at end of file
--
https://code.launchpad.net/~philroche/livecd-rootfs/+git/livecd-rootfs/+merge/458910
Your team Ubuntu Core Development Team is subscribed to branch livecd-rootfs:ubuntu/jammy.
More information about the Ubuntu-reviews
mailing list