Hi,

Because RB wants to handle outputs like they are currently handled under windows (and he is right to want todo things that way), this patch isn't getting merged.

However it might still be interesting for people who want the keyboard led stuff under Linux and want it NOW smile

Besides being Linux/X11 specific there is another problem with this patch, which is a bit of a downside, in order for this to work users must make a small change to their xkb configuration.

For older versions of X users must edit /usr/X11R6/lib/X11/xkb/compat/basic and change "indicator.allowExplicit" from False to True and then restart their X server.

For newer versions they must edit both:
/usr/share/X11/xkb/compat/lednum
/usr/share/X11/xkb/compat/ledcaps
And remove the '!' in front of allowExplicit and then restart their X server.

Once this is done you can test of application control of the leds is allowed by running the following cmds from a terminal:
xset led 1
xset -led 1
xset led 2
xset -led 2
Etc.

I know this having to edit these files sucks, I'll try filing a bug about this with the xkb database maintainers.

If you're giving this a spin try the game "galaxian" for example, num lock will start blinking after you've inserted your first coin and caps lock will be added to that with the second coin.

Now that you've read all this, and if you are still interested go grab the patch here:
http://people.atrpms.net/~hdegoede/sdlmame-02-keyb-leds.patch

Regards,

Hans