https://bugzilla.novell.com/show_bug.cgi?id=784980
https://bugzilla.novell.com/show_bug.cgi?id=784980#c4
--- Comment #4 from Michael Chang
But this does not eliminate error message which triggered this bug report. grub2 post still runs. I think, it has to check for LOADER_TYPE too and skip if LOADER_TYPE is grub2-efi.
Hi Andrey, Ok. Now I'm getting clear about the issue. Those errors are triggered in the %post which tries to add testing grub2 sections to legacy grub menu if current loader is set to grub (It's a relic from 11.4, and for certain reason it's still there). It's obvious to fail in what you mentioned way because there's no additional checks thus grub2-install run blindly. The new improved fix in factory seems to fix the problem, as it now checks current loader and performing actions accordingly .. == # If the grub is the current loader, we'll handle the grub2 testing entry if [ "x${LOADER_TYPE}" = "xgrub" ]; then exec >/dev/null 2>&1 # check if entry for grub2's core.img exists in the config # if yes, we will correct obsoleted path and update grub2 stuff and config to make it work # if no, do nothing if [ -f /boot/grub/menu.lst ]; then # If grub config contains obsolete core.img path, remove and use the new one if /usr/bin/grep -l "^\s*kernel\s*.*/boot/%{name}/core.img" /boot/grub/menu.lst; then /sbin/update-bootloader --remove --image /boot/%{name}/core.img || true /sbin/update-bootloader --add --image /boot/%{name}/i386-pc/core.img --name "GNU GRUB 2" || true fi # Install grub2 stuff and config to make the grub2 testing entry to work with updated version if /usr/bin/grep -l "^\s*kernel\s*.*/boot/%{name}/i386-pc/core.img" /boot/grub/menu.lst; then # Determine the partition with /boot BOOT_PARTITION=$(df -h /boot |(read; awk '{print $1; exit}')) # Generate core.img, but don't let it be installed in boot sector %{name}-install --grub-setup=/bin/true $BOOT_PARTITION || true # Create a working grub2 config, otherwise that entry is un-bootable /usr/sbin/grub2-mkconfig -o /boot/%{name}/grub.cfg fi fi elif [ "x${LOADER_TYPE}" = "xgrub2" ]; then # It's enought to call update-bootloader --refesh to install grub2 and update it's config /sbin/update-bootloader --refresh || true fi == It's not likely to be grub set as current bootloader in use on UEFI platform but to be more conservative your patch would still be valid to fix the problem completely.
Is it planned for 12.2 update?
No. not yet .. I'll do it. Base on where we are at, the right fix would be to finish bnc#782891 and completely drop supporting the nasty "testing grub2 sections" in grub1 to get out all of the mess. 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.