Bugzilla – Bug 148683
YaST2 modules are not run as separate processes from ncurses control centre
Last modified: 2007-03-23 14:56:54 UTC
Timo reports: Network Devices -> Network Card -> Network Setup Method alt-u / alt-n to switch to user controlled with NM enter same again: Network Devices -> Network Card (already there) -> Network Setup Method alt-t / alt-n to switch to ifup alt-n to accept at this point we have a divergence between GUI and config file yast shows: Traditional Mathod with ifup if you enter -> Network Card again but: sysconfig/network/config has NETWORKMANAGER="yes" it works with qt The ncurses cc differs in keeping the yast modules from the previous run. (It uses WFM::CallFunction instead of spawning a new y2base like qt does.) Perhaps I should change that because the behavior is then subtly different like in your case and it gets almost no testing.
Created attachment 66861 [details] 148683-nonfix.patch So I have tried this patch but it does not work in an unexpected way: The module is run but the display is missing accented characters and it does not respond to keypresses (tried in Konsole).
Martin, can you update this bug?
The problem is that we could not run a curses program while using yast2-ncurses. That is a different bug (to be looked up). But we may have made some progress on in because of w3m for the registration.
Reassigning to the new maintainer of yast2-ncurses.
*** Bug 221254 has been marked as a duplicate of this bug. ***
Does anyone want to test the fix? You will need patched yast2-ncurses, yast2-core and yast2 itself. It is pretty complex fix and it may break something else. That's why I'm asking for some test effort :-)
Broken yast, yay, gimme gimme!
Are there pre-built RPMs for x86_64? Then yes :)
Go ahead: yast2-core: /work/built/mbuild/bragg-kmachalkova-3642 yast2: /work/built/mbuild/bragg-kmachalkova-3644 yast2-ncurses: /work/built/mbuild/bragg-kmachalkova-3648 (no i386, I'm sorry) Hope that it won't crash ;-) What bothers me a bit is that I don't know how ncurses menu would react to patch restarting yast
Created attachment 108513 [details] Patch for libyui
Created attachment 108514 [details] Patch for ncurses UI
Created attachment 108515 [details] Patch for menu.ycp and /sbin/yast script Besides adjusting menu.ycp, I've somehow changed locale handling in YaST start script. The change is commented in the patch
The fixed packages are yast2-2.15.0, yast2-ncurses-2.15.1 and yast2-core-2.15.1. The only thing that needs to be settled is that UI::RunInTerminal builtin now returns integer (return code of system() call) at all times, while this condition from /sbin/yast: if ((function == "online_update" && ret == -42) || ret == `restart_menu) expect it to return symbol under certain circumstances. The place where it is used now is yast2 inst_source only. Reassigning to Lada for evaluating possible risk. If it is not a problem, feel free to close as fixed then :-)
Symbol `restart_menu is not needed anymore, the reason was to restart ncurses menu to have a fresh package manager. Removed in SVN (will be included in yast2-2.15.1, yast2-packager-2.15.2)
*** Bug 252302 has been marked as a duplicate of this bug. ***