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

Thomas Bechtold mp+430736 at code.launchpad.net
Thu Sep 29 10:49:00 UTC 2022


Review: Needs Fixing



Diff comments:

> diff --git a/live-build/functions b/live-build/functions
> index 3de9857..25383d4 100644
> --- a/live-build/functions
> +++ b/live-build/functions
> @@ -923,6 +923,7 @@ configure_oci() {
>      # at https://github.com/tianon/docker-brew-ubuntu-core/blob/master/update.sh
>  
>      local chroot=$1
> +    local serial=$2

can we check here that $serial is not empty? Otherwise we might write a file that contains nothing.

>  
>      echo "==== Configuring OCI ===="
>  


-- 
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.




More information about the Ubuntu-reviews mailing list