[Merge] livecd-rootfs:canary-locale into livecd-rootfs:ubuntu/master

Dan Bungert mp+447201 at code.launchpad.net
Wed Jul 19 21:46:53 UTC 2023


Review: Approve

Looks correct to me, I have one further suggestion.

About the layers - desktop and desktop-canary should both drop the minimal layer to simplify image construction, but I don't see that as a requirement for this MP given that we're making desktop-canary more similar to desktop, and desktop still has the minimal layer.

Diff comments:

> diff --git a/live-build/auto/config b/live-build/auto/config
> index f881cc8..b77a824 100755
> --- a/live-build/auto/config
> +++ b/live-build/auto/config
> @@ -744,6 +744,13 @@ case $PROJECT in
>  				remove_package minimal.standard.live ubiquity-frontend-gtk
>  				# the enhanced-secureboot layer, contains all packages for the enhanced secureboot install
>  				add_package minimal.standard.enhanced-secureboot cryptsetup boot-managed-by-snapd
> +
> +                # language support

looks like the whitespace is a bit off, understandable given that that file is a mix

> +				seeded_langs="$(get_seeded_languages desktop-minimal desktop)"
> +				echo "$seeded_langs" | tr ' ' ',' > config/seeded-languages
> +				derive_language_layers minimal desktop-minimal desktop-minimal-default-languages "$seeded_langs"
> +				derive_language_layers minimal.standard desktop desktop-default-languages "$seeded_langs"
> +
>  				# now let's create the neccessary catalog files
>  				# TODO: make sure that we handle locale_support: langpack correctly
>  				cat <<-EOF > config/minimal.catalog-in.yaml
> @@ -753,7 +760,7 @@ case $PROJECT in
>  					id: ubuntu-desktop-minimal

Let's also drop the minimal layer catalog entry, to match non-canary.

>  					type: fsimage-layered
>  					variant: desktop
> -					locale_support: none
> +					locale_support: langpack
>  				EOF
>  				cat <<-EOF > config/minimal.standard.catalog-in.yaml
>  					name: "Ubuntu Desktop"


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




More information about the Ubuntu-reviews mailing list