Opened 13 years ago

Closed 13 years ago

#2990 closed enhancement (fixed)

kbd 1.15.3.

Reported by: Ivan Wagner Owned by: ken@…
Priority: lowest Milestone: 7.2
Component: Book Version: SVN
Severity: minor Keywords:
Cc:

Description

Change History (15)

comment:1 by Ivan Wagner, 13 years ago

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.

comment:2 by rafe, 13 years ago

Compiles/works fine for me since 2011-11-13 already.

in reply to:  2 comment:3 by Ivan Wagner, 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 rafe, 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 ken@…, 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 rafe, 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 ken@…, 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 bdubbs@…, 13 years ago

Summary: kbd 1.15.3kbd 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 Ivan Wagner, 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 Matthew Burgess, 13 years ago

Milestone: 7.17.2

comment:11 by bdubbs@…, 13 years ago

Priority: normallowest
Severity: normalminor

comment:12 by ken@…, 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 ken@…, 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 ken@…, 13 years ago

Owner: changed from lfs-book@… to ken@…
Status: newassigned

comment:15 by ken@…, 13 years ago

Resolution: fixed
Status: assignedclosed

Fixed in r9841,

Note: See TracTickets for help on using tickets.