Your message dated Wed, 6 Aug 2014 12:42:33 +0200 with message-id <[email protected]> and subject line Re: Bug#721087: udev: force-release keymaps files does not accept empty lines has caused the Debian Bug report #721087, regarding udev: force-release keymaps files does not accept empty lines to be marked as done.
This means that you claim that the problem has been dealt with. If this is not the case it is now your responsibility to reopen the Bug report if necessary, and/or fix the problem forthwith. (NB: If you are a system administrator and have no idea what this message is talking about, this may indicate a serious mail system misconfiguration somewhere. Please contact [email protected] immediately.) -- 721087: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=721087 Debian Bug Tracking System Contact [email protected] with problems
--- Begin Message ---Package: udev Version: 175-7.2 Severity: normal Tags: patch Hi, I've noticed that /lib/udev/keymaps/force-release/* files does not accept empty lines but the last one. This is due to the manner /lib/udev/keyboard-force-release.sh is written: with an empty line read, scancode == "" and $(($scancode)) provokes an error. The issue is still present in experimental package udev_204-2. Bash accepts $(()) but sh do not. Moreover, sh is called with -e option so the script keyboard-force-release.sh stops and the real job is never done, do even the first part of the keymap file is not used for the final step. Fix proposition: handle the empty scancode case: --- keyboard-force-release.sh.debian 2013-08-27 22:49:16.947684034 +0200 +++ keyboard-force-release.sh 2013-08-27 22:52:52.499674424 +0200 @@ -11,6 +11,7 @@ read attr <"/sys/$1/force_release" while read scancode dummy; do + [ -z $scancode ] && continue case "$scancode" in \#*) ;; *) Cheers, -- Sylvain -- System Information: Debian Release: jessie/sid APT prefers testing APT policy: (500, 'testing') Architecture: amd64 (x86_64) Kernel: Linux 3.11.0-rc6+ (SMP w/4 CPU cores) Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8) Shell: /bin/sh linked to /bin/dash Versions of packages udev depends on: ii debconf [debconf-2.0] 1.5.50 ii libc6 2.17-92 ii libselinux1 2.1.13-2 ii libudev0 175-7.2 ii lsb-base 4.1+Debian12 ii procps 1:3.3.4-2 ii util-linux 2.20.1-5.5 Versions of packages udev recommends: ii pciutils 1:3.2.0-3 ii usbutils 1:007-2 udev suggests no packages. -- debconf information: udev/new_kernel_needed: false udev/title/upgrade: udev/reboot_needed: udev/sysfs_deprecated_incompatibility:--- keyboard-force-release.sh.debian 2013-08-27 22:49:16.947684034 +0200 +++ keyboard-force-release.sh 2013-08-27 22:52:52.499674424 +0200 @@ -11,6 +11,7 @@ read attr <"/sys/$1/force_release" while read scancode dummy; do + [ -z $scancode ] && continue case "$scancode" in \#*) ;; *)
--- End Message ---
--- Begin Message ---Version: 208-1 Martin Pitt [2014-04-29 14:53 +0200]: > These are gone in current systemd versions, replaced with a single > 60-keyboard.hwdb. Marking as fixed in experimental. 208 is in unstable/testing now, closing. Martin -- Martin Pitt | http://www.piware.de Ubuntu Developer (www.ubuntu.com) | Debian Developer (www.debian.org)
--- End Message ---
