Ubuntu
console-setup package

ckbcomp cannot handle multiple consequents in xkb rules

Bug #139710 reported by Adam Simpkins
6
Affects Status Importance Assigned to Milestone
console-setup (Ubuntu)
Fix Released
Undecided
Colin Watson

Bug Description

Binary package hint: console-setup

My XKB rules file has a set of rules with multiple consequents:

  ! model layout = keycodes symbols
    dbied qwerty = maltron(qwerty) maltron(qwerty)
    dbied maltron = maltron(maltron) maltron(maltron)
    dbied unix = maltron(qwerty) maltron(unix-qwerty)

ckbcomp gets confused by the fact that there are two separate items on the
right hand side of the "=". Because of this, it cannot determine the keycodes,
so it exits with the error "No keycodes, nor layout specified".

The rules file works just fine with X.

I'll attach a patch to ckbcomp that fixes the problem.

Revision history for this message
Adam Simpkins (simpkins) wrote :
Revision history for this message
Miroslav Sabljić (civija) wrote :

Thank you for taking the time to report this bug and helping to make Ubuntu better. You reported this bug a while ago and there hasn't been any activity in it recently. We were wondering is this still an issue for you? Can you try with latest Ubuntu release? Thanks in advance.

Revision history for this message
Miroslav Sabljić (civija) wrote : Re: ckbcomp cannot handle multiple consequents in xkbrules

To get your fix included in Ubuntu, it would help if you tried
transforming it into a debdiff
(http://wiki.ubuntu.com/PackagingGuide/Recipes/Debdiff) and submit it
for review (http://wiki.ubuntu.com/SponsorshipProcess). If you prefer
somebody else to do that, that's fine - please just indicate if you're
available to do that.

Revision history for this message
Miroslav Sabljić (civija) wrote :

Setting status: In Progress

Changed in console-setup:
status: New → In Progress
Revision history for this message
Colin Watson (cjwatson) wrote :

There is no need to worry about supplying a debdiff.

Changed in console-setup:
status: In Progress → Confirmed
Revision history for this message
Colin Watson (cjwatson) wrote :

(FWIW, I just haven't had time to digest this patch yet!)

Revision history for this message
Colin Watson (cjwatson) wrote :

Committed, thanks! The only change I made was to return to using split ' ' rather than split '[ \t]', as split ' ' is magic in Perl and splits on any whitespace.

Changed in console-setup:
assignee: nobody → kamion
status: Confirmed → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package console-setup - 1.21ubuntu5

---------------
console-setup (1.21ubuntu5) hardy; urgency=low

  * Silence error messages for keycodes 214, 215, 216, and 217, which are
    various special keys in the default X.org PC keyboard map that we don't
    need to handle.
  * Map dead_stroke to U+002D HYPHEN-MINUS.
  * Add a "Do not configure keyboard; keep kernel keymap" model option,
    preseedable by console-setup/modelcode=SKIP (LP: #59889).
  * Drop the layout/variant priority to high on fresh installs
    (LP: #205959).
  * Support rules with multiple consequents (thanks, Adam Simpkins;
    LP: #139710).
  * Allow specifying alternate rules using XKBRULES (thanks, Adam Simpkins;
    LP: #139712).

 -- Colin Watson <email address hidden> Mon, 31 Mar 2008 14:33:25 +0100

Changed in console-setup:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.