[Pkg-ltsp-devel] Bug#421762: ltsp-client-builder: Fail to set correct default X keyboard layout

[Petter Reinholdtsen]

Package: ltsp-client-builder
Version: 0.99debian11

When using the ltsp-client-builder udeb to install the LTSP chroot
from within debian-installer, the default X keyboard layout used by
the thin client is not inherited from the keyboard layout setting
selected in debian-installer.  In the Debian Edu subproject, we want
the keyboard layout selected during installation to take effect also
in LTSP, to reduce the amount of configuration required before the
system is ready to use.

I believe this patch solve the problem, by preseeding the
debian-installer/keymap value into the ltsp chroot, and leaving it to
the xserver-xorg postinst script to set the X keyboard layout like it
does on a normal Debian install.  It add a dependency on the
kbd-chooser udeb, to make sure the user selected the keyboard layout
before ltsp-client-builder is executed.

The patch add a new progress bar step as well for the keyboard layout
passing.  Not sure if it make sense or not.

According to Oliver at Ubuntu, they set the keyboard layout using the
locales ltsp-server plugin, by copying the console-setup config from
the server to the client and run locale-gen $LANG.  It might be an
option for Debian too.

--- ltsp-0.99debian11/debian/ltsp-client-builder.postinst       2007-01-11 08:20:46.000000000 +0100
+++ ltsp-0.99debian11-pere/debian/ltsp-client-builder.postinst  2007-05-01 11:47:54.000000000 +0200
@@ -33,12 +33,21 @@

 # if everything is fine, run ltsp-build-client on the target disk

-db_progress START 0 2 ltsp-client-builder/progress
+db_progress START 0 3 ltsp-client-builder/progress

 apt-install ltsp-server

 db_progress STEP 1

+# Make the d-i keyboard layout setting available in the ltsp chroot
+db_get debian-installer/keymap
+if [ "$RET" ] ; then
+    echo "d-i debian-installer/keymap string $RET" > /tmp/ltsp-preseed
+    BUILD_CLIENT_OPTS="$BUILD_CLIENT_OPTS --debconf-seeds /tmp/ltsp-preseed"
+fi
+
+db_progress STEP 1
+
 # workaround for: http://bugs.debian.org/390647
 in-target /bin/touch /etc/resolv.conf

diff -ur ltsp-0.99debian11/debian/control ltsp-0.99debian11-pere/debian/control
--- ltsp-0.99debian11/debian/control    2007-05-01 12:38:35.000000000 +0200
+++ ltsp-0.99debian11-pere/debian/control       2007-01-11 08:20:46.000000000 +0100
@@ -53,7 +53,7 @@
 Section: debian-installer
 XC-Package-Type: udeb
 Architecture: all
-Depends: ${misc:Depends}, ${shlibs:Depends}, kbd-chooser, pkgsel
+Depends: ${misc:Depends}, ${shlibs:Depends}, pkgsel
 XB-Installer-Menu-Item: 70
 Description: Build an LTSP environment in the installer target
  This udeb bootstraps the LTSP client chroot in the installer target