http://bugzilla.suse.com/show_bug.cgi?id=1125814
Bug ID: 1125814 Summary: [Build 20190125] YaST2 keyboard module applies settings even when Cancel button is pressed Classification: openSUSE Product: openSUSE Tumbleweed Version: Current Hardware: Other OS: Other Status: NEW Severity: Normal Priority: P5 - None Component: YaST2 Assignee: yast2-maintainers@suse.de Reporter: riafarov@suse.com QA Contact: jsrain@suse.com Found By: --- Blocker: ---
Created attachment 797088 --> http://bugzilla.suse.com/attachment.cgi?id=797088&action=edit YaST logs
## Steps to reproduce: 1. Change X11 keyboard setting (e.g. run setxkbmap -layout us,ru) 2. Run yast2 keyboard module 3. English US is preselected 4. Press Cancel button 5. English US is set as a keyboard layout, so settings from step 1) are overwritten.
Please, find logs in the attachment.
http://bugzilla.suse.com/show_bug.cgi?id=1125814 http://bugzilla.suse.com/show_bug.cgi?id=1125814#c1
David Diaz dgonzalez@suse.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |dgonzalez@suse.com, | |riafarov@suse.com Flags| |needinfo?(riafarov@suse.com | |)
--- Comment #1 from David Diaz dgonzalez@suse.com --- I couldn't reproduce it in a openSUSE TW 20190125 using `yast-country 4.1.7`.
Given steps,
* Changed the keyboard layout to English US through Gnome settings * Run yast2 keyboard module * (Spanish was preselected) * Pressed Cancel button * Spanish layout **was not** set
In the YaST logs it could be found
2019-02-19 09:56:19 <1> linux-9x6v(7484) [Ruby] modules/Keyboard.rb:515 Setting keyboard to: <spanish>
but the keyboard layout still being English US. In fact, when the "OK" button is pressed, is expected to find something like follow in the logs
2019-02-19 10:00:53 <1> linux-9x6v(11065) [Ruby] modules/Keyboard.rb:879 Making keyboard settings persistent: command /usr/bin/localectl set-keymap es end with {"exit"=>0, "stderr"=>"", "stdout"=>""}
Rodion, could you give us more information about the system?
Thank you!
http://bugzilla.suse.com/show_bug.cgi?id=1125814 http://bugzilla.suse.com/show_bug.cgi?id=1125814#c2
--- Comment #2 from Rodion Iafarov riafarov@suse.com --- I have same version as you've mentioned. I have this problem on my laptop, where I have i3wm installed, so will try to reproduce it in some simpler env.
What I've assumed that in case of pressing cancel there should be no log lines like: 2019-02-18 17:08:41 <1> ri-laptop(31225) [Ruby] modules/Keyboard.rb:1401 Setting X11 keyboard to: <english-us> 2019-02-18 17:08:41 <1> ri-laptop(31225) [Ruby] modules/Keyboard.rb:1402 Setting X11 keyboard: /usr/bin/setxkbmap -layout us -model microsoftpro -option terminate:ctrl_alt_bksp
And as from the code: https://github.com/yast/yast-country/blob/master/keyboard/src/modules/Keyboa... it actually executes those commands, which should not be the case. But let me update issue with steps for clean installation.
http://bugzilla.suse.com/show_bug.cgi?id=1125814 http://bugzilla.suse.com/show_bug.cgi?id=1125814#c3
Rodion Iafarov riafarov@suse.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags|needinfo?(riafarov@suse.com | |) |
--- Comment #3 from Rodion Iafarov riafarov@suse.com --- Ok, so I can easily reproduce it with ice-wm. Gnome handles keyboard layout differently. So here are the steps: 1. Install TW with icewm as WM 2. run xterm 3. Execute following commands: setxkbmap -layout us,ru setxkbmap -option 'grp:alt_shift_toggle' 4. One can switch between ru and us with alt-shift 5. Run `setxkbmap -query | grep layout` 6. It returns: "layout: us,ru" 7. Run `xdg-su -c 'yast2 keyboard'` 8. Click cancel in opened dialogue 9. Run `setxkbmap -query | grep layout` 10. It returns: "layout: us"
I hope this helps. Let me know if you are still not able to reproduce the issue. And thanks for the prompt reply, I appreciate this style of work ;)
http://bugzilla.suse.com/show_bug.cgi?id=1125814 http://bugzilla.suse.com/show_bug.cgi?id=1125814#c4
David Diaz dgonzalez@suse.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |CONFIRMED
--- Comment #4 from David Diaz dgonzalez@suse.com --- Thank you so much, Rodion.
Regarding to
(In reply to Rodion Iafarov from comment #2) I have same version as you've mentioned. I have this problem on my laptop, where I have i3wm installed, so will try to reproduce it in some simpler env.
What I've assumed that in case of pressing cancel there should be no log lines like: 2019-02-18 17:08:41 <1> ri-laptop(31225) [Ruby] modules/Keyboard.rb:1401 Setting X11 keyboard to: <english-us> 2019-02-18 17:08:41 <1> ri-laptop(31225) [Ruby] modules/Keyboard.rb:1402 Setting X11 keyboard: /usr/bin/setxkbmap -layout us -model microsoftpro -option terminate:ctrl_alt_bksp
And as from the code: https://github.com/yast/yast-country/blob/master/keyboard/src/modules/ Keyboard.rb#L1400 it actually executes those commands, which should not be the case. But let me update issue with steps for clean installation.
I am not pretty sure right now, but I guess that this is being executed in order to allow to use the "Test" input field.
Anyway, following your detailed steps I was able to reproduce it in a fresh openSUSE TW installation using ice-wm :(
So, I am going to track it in our Trello board.
Thanks!
http://bugzilla.suse.com/show_bug.cgi?id=1125814
David Diaz dgonzalez@suse.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Priority|P5 - None |P4 - Low URL| |https://trello.com/c/wwCpgc | |wU Assignee|yast2-maintainers@suse.de |yast-internal@suse.de
http://bugzilla.suse.com/show_bug.cgi?id=1125814
Rodion Iafarov riafarov@suse.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Summary|[Build 20190125] YaST2 |[Build 20190125] YaST2 |keyboard module applies |keyboard module applies |settings even when Cancel |settings even when Cancel |button is pressed |button is pressed on icewm Severity|Normal |Minor
http://bugzilla.suse.com/show_bug.cgi?id=1125814 http://bugzilla.suse.com/show_bug.cgi?id=1125814#c5
--- Comment #5 from Rodion Iafarov riafarov@suse.com --- (In reply to David Diaz from comment #4)
Thank you so much, Rodion.
Regarding to
(In reply to Rodion Iafarov from comment #2) I have same version as you've mentioned. I have this problem on my laptop, where I have i3wm installed, so will try to reproduce it in some simpler env.
What I've assumed that in case of pressing cancel there should be no log lines like: 2019-02-18 17:08:41 <1> ri-laptop(31225) [Ruby] modules/Keyboard.rb:1401 Setting X11 keyboard to: <english-us> 2019-02-18 17:08:41 <1> ri-laptop(31225) [Ruby] modules/Keyboard.rb:1402 Setting X11 keyboard: /usr/bin/setxkbmap -layout us -model microsoftpro -option terminate:ctrl_alt_bksp
And as from the code: https://github.com/yast/yast-country/blob/master/keyboard/src/modules/ Keyboard.rb#L1400 it actually executes those commands, which should not be the case. But let me update issue with steps for clean installation.
I am not pretty sure right now, but I guess that this is being executed in order to allow to use the "Test" input field.
Anyway, following your detailed steps I was able to reproduce it in a fresh openSUSE TW installation using ice-wm :(
So, I am going to track it in our Trello board.
Thanks!
I've adjusted severity and bug title as per our findings. Thanks a lot!
https://bugzilla.suse.com/show_bug.cgi?id=1125814 https://bugzilla.suse.com/show_bug.cgi?id=1125814#c6
Stefan Hundhammer shundhammer@suse.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|CONFIRMED |RESOLVED Resolution|--- |WORKSFORME
--- Comment #6 from Stefan Hundhammer shundhammer@suse.com --- This is one of the very few places in YaST where we use "instant apply" so you can immediately work with the new settings; including testing the keyboard layout in the "Test" field.
So this works as intended.