Bug 6725 - Update request: numlock
Summary: Update request: numlock
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: 1
Hardware: All Linux
Priority: Normal normal
Target Milestone: ---
Assignee: QA Team
QA Contact:
URL:
Whiteboard: has_procedure MGA1-32-OK MGA1-64-OK
Keywords: validated_update
Depends on:
Blocks:
 
Reported: 2012-07-08 23:04 CEST by Luc Menut
Modified: 2012-07-30 18:05 CEST (History)
3 users (show)

See Also:
Source RPM: numlock-2.1.2-5.1.mga1
CVE:
Status comment:


Attachments

Description Luc Menut 2012-07-08 23:04:45 CEST
Hello,

I submitted an update of numlock for mga 1; it should fix numlock under X (bug 1136).

Packages list:
---------------
x86_64 : numlock-2.1.2-5.1.mga1.x86_64.rpm
i586 :   numlock-2.1.2-5.1.mga1.i586.rpm

How to test:
------------
on a system with numpad !!
BEFORE
with numlock-2.1.2-5.mga1, enable_X11_numlock doesn't work correctly; if it is
called many times, enable_X11_numlock switches the numlock (on, then off, then
on, ...) instead of turns it on. So that numlock doesn't work with KDE out of
the box.

AFTER
with numlock-2.1.2-5.1.mga1, numlockx replaces enable_X11_numlock. If you launch many times numlockx, numlock should stay on. Numlock should work correctly with KDE.


Suggested advisory:
-------------------
Numlock doesn't work correctly under X on Mageia 1. This update fix this issue.

regards,
Luc
Comment 1 Samuel Verschelde 2012-07-21 21:03:09 CEST
I see no update for Mageia 2. Does it mean that numlock is fixed already in Mageia 2?

I can't reproduce the problem with numlock on my system, but I can test the update candidate to see there is no regression.

I have one problem with that update candidate: what if users or programs call enable_X11_numlock in their scripts? Won't the update break those scripts because of the script rename? I think a symlink should be provided for backward compatibility, do you agree?

Apart from that, the update candidate seems to work on Mageia 1 32 bits.

CC: (none) => stormi

Comment 2 Luc Menut 2012-07-21 23:18:48 CEST
(In reply to comment #1)
> I see no update for Mageia 2. Does it mean that numlock is fixed already in
> Mageia 2?

yes, mga 2 already uses numlockx (since 2012-01-03).
https://bugs.mageia.org/show_bug.cgi?id=1136#c15
http://svnweb.mageia.org/packages?view=revision&revision=189843

> 
> I can't reproduce the problem with numlock on my system, but I can test the
> update candidate to see there is no regression.
> 
> I have one problem with that update candidate: what if users or programs call
> enable_X11_numlock in their scripts? Won't the update break those scripts
> because of the script rename? I think a symlink should be provided for backward
> compatibility, do you agree?

not necessary, mga 2 uses numlockx, like most others distros.
Comment 3 Samuel Verschelde 2012-07-22 00:04:37 CEST
(In reply to comment #2)
> (In reply to comment #1)
> > I have one problem with that update candidate: what if users or programs call
> > enable_X11_numlock in their scripts? Won't the update break those scripts
> > because of the script rename? I think a symlink should be provided for backward
> > compatibility, do you agree?
> 
> not necessary, mga 2 uses numlockx, like most others distros.

Ok for mga2, but this update candidate is for mga1, and an update must usually not change the way a tool is used (ie, IINM, no binary name change without providing a symlink for compatibility). As it is, this update could potentially break something for users. A symlink would prevent that.
Comment 4 Luc Menut 2012-07-22 01:03:07 CEST
numlockx uses the same default (on, to turn numlock on in X) but doesn't have exactly the same syntax than enable_X11_numlock to toggle the numlock on and off: numlockx uses toggle, enable_X11_numlock uses switch, so I won't add a symlink.

numlockx --help
Usage : numlockx [on|off]
on     - turns NumLock on in X ( default )
off    - turns NumLock off in X
toggle - toggles the NumLock on and off in X

enable_X11_numlock --help
Usage : enable_X11_numlock [on|off|switch]
on     - sets NumLock on in X (default)
off    - sets NumLock off in X
switch - changes NumLock state in X

if this update is not valid, sorry, but personally I won't spend and lose more time on this bug only for mga 1.
Comment 5 Samuel Verschelde 2012-07-22 09:58:07 CEST
(In reply to comment #4)
> if this update is not valid, sorry, but personally I won't spend and lose more
> time on this bug only for mga 1.

I understand "spend" but not "lose" here. Since when is fixing bugs for supported Mageia releases a loss of time?

However, I haven't said that the update is not valid. I tried to see if we could do it better. What could help decide if the update is valid or not is some insight about the chance there is to break something for users with the replacement of enable_X11_numlock with numlockx. I personnally don't know.
Comment 6 Luc Menut 2012-07-24 21:49:28 CEST
(In reply to comment #5)
> (In reply to comment #4)
> > if this update is not valid, sorry, but personally I won't spend and lose more
> > time on this bug only for mga 1.
> 
> I understand "spend" but not "lose" here. Since when is fixing bugs for
> supported Mageia releases a loss of time?

ie I won't try to patch numlockx so that it has exactly the same syntax as enable_X11_numloack, it would be a loss of time.

> 
> However, I haven't said that the update is not valid. I tried to see if we
> could do it better. What could help decide if the update is valid or not is
> some insight about the chance there is to break something for users with the
> replacement of enable_X11_numlock with numlockx. I personnally don't know.

IMHO the risks are limited, but it's difficult to evaluate the benefit/risk ratio.
Perhaps most of affected users already use a workaround for bug 1136, and an update is useless.
Personally I had this bug on all my systems with numpad, but I'm not sure that this bug concerns all KDE users, or users of others DE.
Comment 7 Samuel Verschelde 2012-07-28 17:48:04 CEST
Let's push it, I think the risks to break something for a user are very limited indeed, and many users experienced the numlock bug (I can't reproduce it on this computer but I had it on my work computer before it got upgraded to Mageia 2)

Testing complete on Mageia 1 32 and 64 bits. Update validated. No linking required.

see comment #0 for packages and advisory.

Keywords: (none) => validated_update
CC: (none) => sysadmin-bugs
Whiteboard: (none) => has_procedure MGA1-32-OK MGA1-64-OK

Comment 8 Thomas Backlund 2012-07-30 18:05:38 CEST
Update pushed:
https://wiki.mageia.org/en/Support/Advisories/MGAA-2012-0141

Status: NEW => RESOLVED
CC: (none) => tmb
Resolution: (none) => FIXED


Note You need to log in before you can comment on or make changes to this bug.