Applied some updates and responded to feedback.

Diff comments:

> diff --git a/data/lunar/preseed/xubuntu-minimal/xubuntu-minimal.seed 
> b/data/lunar/preseed/xubuntu-minimal/xubuntu-minimal.seed
> new file mode 100644
> index 0000000..cf77909
> --- /dev/null
> +++ b/data/lunar/preseed/xubuntu-minimal/xubuntu-minimal.seed
> @@ -0,0 +1,7 @@
> +# Enable extras.ubuntu.com.
> +d-i  apt-setup/extras        boolean true

Removed from both xubuntu and xubuntu-minimal. Thanks!

> +# Install the Xubuntu desktop.
> +tasksel      tasksel/first   multiselect xubuntu-core

Yes, I think this would make sense. I'll hold on package surgery until I know 
Xubuntu Minimal is approved (or is it already?)

> +d-i  preseed/early_command   string . /usr/share/debconf/confmodule; db_get 
> debconf/priority; case $RET in low|medium) db_fset tasksel/first seen false; 
> echo 'tasksel tasksel/first seen false' >>/var/lib/preseed/log ;; esac

I don't *think* we do. This line exists in (nearly?) every preseed. Not sure if 
it's still needed these days.

> +# No XFCE translation packages yet.
> +d-i  pkgsel/language-pack-patterns   string
> diff --git a/tools/add_live_filesystem b/tools/add_live_filesystem
> index 952ca36..23e362d 100755
> --- a/tools/add_live_filesystem
> +++ b/tools/add_live_filesystem
> @@ -122,6 +122,8 @@ case $PROJECT in
>                               ;;
>               esac
>               ;;
> +     xubuntu-minimal)
> +             
> RELEASE_NOTES="http://www.ubuntu.com/getubuntu/releasenotes?os=xubuntu&ver=${DEBVERSION%%
>  *}&lang=\${LANG}"

Clever solution! Applied.

>  esac
>  if [ "$RELEASE_NOTES" ]; then
>       mkdir -p "$BDIR/CD1/.disk"
> diff --git a/tools/boot/lunar/common.sh b/tools/boot/lunar/common.sh
> index 349f024..9746f3a 100644
> --- a/tools/boot/lunar/common.sh
> +++ b/tools/boot/lunar/common.sh
> @@ -23,6 +23,9 @@ default_kernel_params() {
>          xubuntu)
>              KERNEL_PARAMS="${KERNEL_PARAMS:+$KERNEL_PARAMS 
> }file=/cdrom/preseed/xubuntu.seed maybe-ubiquity quiet splash --- "
>              ;;
> +        xubuntu-minimal)
> +            KERNEL_PARAMS="${KERNEL_PARAMS:+$KERNEL_PARAMS 
> }file=/cdrom/preseed/xubuntu-minimal.seed quiet splash --- "
> +            ;;

This was a suggestion from another in my team. I'll confirm with them.

>          ubuntustudio)
>              KERNEL_PARAMS="${KERNEL_PARAMS:+$KERNEL_PARAMS 
> }file=/cdrom/preseed/ubuntustudio.seed maybe-ubiquity quiet splash --- "
>              ;;


-- 
https://code.launchpad.net/~xubuntu-dev/debian-cd/+git/debian-cd/+merge/435314
Your team Xubuntu Developers is subscribed to branch 
~xubuntu-dev/debian-cd:xubuntu-core.


_______________________________________________
Mailing list: https://launchpad.net/~xubuntu-dev
Post to     : xubuntu-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~xubuntu-dev
More help   : https://help.launchpad.net/ListHelp

Reply via email to