Opened 13 years ago
Closed 13 years ago
#2990 closed enhancement (fixed)
kbd 1.15.3.
Reported by: | Ivan Wagner | Owned by: | |
---|---|---|---|
Priority: | lowest | Milestone: | 7.2 |
Component: | Book | Version: | SVN |
Severity: | minor | Keywords: | |
Cc: |
Description
New version, see http://ftp.altlinux.org/pub/people/legion/kbd/.
Change History (15)
comment:1 by , 13 years ago
comment:3 by , 13 years ago
Replying to rafe:
Compiles/works fine for me since 2011-11-13 already.
I got the same error mentioned http://lists.altlinux.org/pipermail/kbd/2011-May/000301.html using the 12/28/2011 SVN book version. The error is fixed in the git repo.
comment:4 by , 13 years ago
Oh, I think I remember, there is a tiny mistake (a missing $ sign or two) in po/es.po. Read the output from the compile, it leads one to fixing the error like I did or get the fixed source herehttp://www.roy.lv/kbd-1.15.3.tar.xz
comment:5 by , 13 years ago
Please see #2873 and also #2925 - in particular the reference to http://lists.altlinux.org/pipermail/kbd/2011-May/000308.html
I'd love to see this in the book (my own compile fix - just rm the Spanish translation, which was what upstream did) is detailed in the first of those tickets, but the regression detailed on the altlinux list is the dealbreaker. I haven't looked at the git tree - is there a simple patch to fix the regression ?
If so, I'd like to see this go in - I'm getting feedback on LatGrkCyr from altlinux users, would be nice to make it available to current LFS builders. If the fix isn't simple, I guess we can wait for the next release.
comment:6 by , 13 years ago
diff -r kbd-1.15.3/po/es.po kbd-1.15.3-fixed/po/es.po 1366c1366 < msgstr "Se ha guardado el fichero de tipos %2$dx%3$d de %1$d caracteres en %s\n" --- > msgstr "Se ha guardado el fichero de tipos %2$dx%3$d de %1$d caracteres en %4$s\n"
comment:7 by , 13 years ago
Well, that one-line change to es.po fixes the compilation. But it does nothing to address the issue with loadkeys that Matt identified as a showstopper:
# echo keycode 1 = Caps_Lock | loadkeys # echo keycode 58 = Escape | loadkeys - on 1.15.2, swaps caps lock and escape - on 1.15.3, does nothing and returns 0
I've tested this on 1.15.2 (where it works), 1.15.3 where it doesn't, and to cut a long story short, current git master from http://git.altlinux.org/people/legion/packages/kbd.git i.e. kbd-1371bd5 - still doesn't work.
Unless someone disputes my results (its certainly possible I mistyped what I was doing - the newer versions were in DESTDIRs so I had to pass a path), or provides evidence why the test is now invalid, this is heading for wontfix.
I've looked at loadkeys.1 in git - some parts clearly out of date, no apparent changes since 1994. I thought loadkeys had supposedly been fixed for this to work, but I see no evidence.
comment:8 by , 13 years ago
Summary: | kbd 1.15.3 → kbd 1.15.3 broken - leave at 1.15.2 until a new version is released. |
---|
Don't mark this ticket as wontfix, but leave it until a new version of kbd is released.
comment:9 by , 13 years ago
Don't forget to list the new kbd home page to the the packages.ent file: <!ENTITY kbd-home "http://www.kbd-project.org/">
comment:10 by , 13 years ago
Milestone: | 7.1 → 7.2 |
---|
comment:11 by , 13 years ago
Priority: | normal → lowest |
---|---|
Severity: | normal → minor |
comment:12 by , 13 years ago
I didn't realise that debian offer kbd. Their version of 1.15.3 includes a patch which is headed
From: Alexey Gladkov <gladkov.alexey@gmail.com> Subject: Fix important regression in loadkeys Origin: commit:83f0743c1fed872d80a0dd31924575d9bf9c2770 loadkeys from 1.15.3 introduced a regression where it doesn’t default to reading from stdin when started without a filename argument. This behavior is widely used, even in /etc/init.d/kbd.
I'll add this to my current build, with the es.po patch (looks as if debian are doing something different in gettext itself).
comment:13 by , 13 years ago
Summary: | kbd 1.15.3 broken - leave at 1.15.2 until a new version is released. → kbd 1.15.3. |
---|
Works. Instead of silence, it reports
Loading <stdin>
but I doubt that will break anything. Reverting ticket to the original title, I'll pick this up in a day or two unless osmeone complains about the extra output.
comment:14 by , 13 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
Ack, jumped the gun. I see why LFS hasn't moved to this version. It doesn't compile. Oh well at least you can update add a Homepage for kbd in the Chapter 3 listing of all packages. See http://lists.altlinux.org/pipermail/kbd/2011-December/000326.html.