[Merge] ~davidkrauser/livecd-rootfs/+git/livecd-rootfs:ubuntu/master into livecd-rootfs:ubuntu/master

Robert C Jennings robert.jennings at canonical.com
Wed Jul 22 16:37:12 UTC 2020


Review: Needs Fixing

Hey David, the implementation looks good.  I'd like to see comments added to the file we're dropping in the images.

I would like to see the commit message tweaked to include the package version in addition to the commit id.  The development is done in lp:livecd-rootfs but it also get's mirrored in https://git.launchpad.net/ubuntu/+source/livecd-rootfs by the Server Dev import team based on archive package contents.  In that context the commit ID won't be informative.  Also, the version will be helpful in writing a changelog entry where the commit id won't.

Thanks David!

Diff comments:

> diff --git a/live-build/functions b/live-build/functions
> index 483fc80..34c17c0 100644
> --- a/live-build/functions
> +++ b/live-build/functions
> @@ -962,6 +962,25 @@ replace_kernel () {
>      force_boot_without_initramfs ${mountpoint}
>  }
>  
> +track_initramfs_boot_fallback() {
> +    mountpoint=$1
> +    cat <<"END" > "${mountpoint}/etc/grub.d/01_track_initrdless_boot_fallback"
> +#! /bin/sh
> +set -e

I would like to see comments added in the file for end-users to explain the purpose of this hook and where it is coming from (for the later part this is the IMG_STR "This file was created by the Image build process")

> +cat <<"EOF"
> +if [ -n "${have_grubenv}" ]; then
> +  if [ -n "${initrdfail}" ]; then
> +    set initrdless_boot_fallback_triggered="${initrdfail}"
> +  else
> +    set initrdless_boot_fallback_triggered=0
> +  fi
> +  save_env initrdless_boot_fallback_triggered
> +fi
> +EOF
> +END
> +    chmod +x "${mountpoint}/etc/grub.d/01_track_initrdless_boot_fallback"
> +}
> +
>  force_boot_without_initramfs() {
>      mountpoint=$1
>  


-- 
https://code.launchpad.net/~davidkrauser/livecd-rootfs/+git/livecd-rootfs/+merge/387862
Your team Ubuntu Core Development Team is requested to review the proposed merge of ~davidkrauser/livecd-rootfs/+git/livecd-rootfs:ubuntu/master into livecd-rootfs:ubuntu/master.



More information about the Ubuntu-reviews mailing list