https://bugzilla.novell.com/show_bug.cgi?id=773143 https://bugzilla.novell.com/show_bug.cgi?id=773143#c35 --- Comment #35 from Michael Chang <mchang@suse.com> 2013-02-21 10:12:44 UTC --- (In reply to comment #33)
Notice that this patch will have side effects. It makes YaST2 bootloader module to think there is explicit default entry set - while it is not. So the following is possible
- user starts YaST2 bootloader, sees "Some entry title" which happens to be the first one and thinks it is OK. yast2-bootloader does *not* update grub2 (i.e. does not write saved_entry) because it believes nothing was changed (is it true?)
If you haven't touched SetSettings() I think it's the reason why changes not be saved eventually.
- after that grub2-mkconfig is run which now changes boot entries order so "Some entry title" is no more the first one
This seems to me a generic problem if we set an explicit entry. If we set it to, say "Advanced options for .... > openSUSE 12.2 with Linux 3.4.6-2.10-desktop" And received an kernel updates to different version ... would grub2-mkconfig handle that by updating the saved_entry as well as the config ?
- now the effective default is no more "Some entry title" but whatever happens to become the first menu entry after grub2-mkconfig
But I suppose your patch will show the new default .. right ? If yes this is not a big problem to me. :)
Also if user changes GRUB_DEFAULT from "saved" to explicit menu entry this is going to be totally off mark.
I doubt any better people want to do that? It blows out things like grub2-reboot which used in hibernation (and other circumstance). PBL need to improve to support that, but not urgent to me.
But adding notion of "not explicitly set, but bootloader default" menu entry to YaST2 is definitely beyond my abilities.
Ok. This change involves ybl .. But I would treat it as enhancement and this bug can be cured with your patch (ie I dont think the side effects is really that hurt to us. :) thanks. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.