xubuntu-dev team mailing list archive

[Sean Davis <mp+435314@xxxxxxxxxxxxxxxxxx>]

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.