[Merge] ~samirakarioh/livecd-rootfs:add-metadata-rocks into livecd-rootfs:ubuntu/master

[Cristovao Cordeiro]

Review: Needs Fixing

Thanks. A few comments below

Diff comments:

> diff --git a/live-build/functions b/live-build/functions
> index 3de9857..5523024 100644
> --- a/live-build/functions
> +++ b/live-build/functions
> @@ -931,6 +932,11 @@ configure_oci() {
>      echo 'exit 101' >> ${chroot}/usr/sbin/policy-rc.d
>      Chroot ${chroot} "chmod +x /usr/sbin/policy-rc.d"
>  
> +    # add file

pls make the comment a bit more meaningful

> +    mkdir -p chroot/etc/cloud
> +    cat > chroot/etc/cloud/build.info << EOF

should be ${chroot}, otherwise you're not writing the file into the provided chroot path

> +serial: $serial
> +EOF
>      # https://github.com/docker/docker/blob/9a9fc01af8fb5d98b8eec0740716226fadb3735c/contrib/mkimage/debootstrap#L54-L56
>      Chroot ${chroot} "dpkg-divert --local --rename --add /sbin/initctl"
>      cp -a ${chroot}/usr/sbin/policy-rc.d ${chroot}/sbin/initctl


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