Author: juhliarik Date: Fri Nov 7 10:33:33 2008 New Revision: 53034 URL: http://svn.opensuse.org/viewcvs/yast?rev=53034&view=rev Log: added several fixes: - powerlilo (writing kdump settings) - using fix_chs - double boot entry for UEFI Modified: trunk/bootloader/package/yast2-bootloader.changes trunk/bootloader/src/grub/misc.ycp trunk/bootloader/src/modules/BootELILO.ycp trunk/bootloader/src/modules/BootPOWERLILO.ycp trunk/bootloader/src/modules/Bootloader.ycp trunk/bootloader/src/routines/misc.ycp Modified: trunk/bootloader/package/yast2-bootloader.changes URL: http://svn.opensuse.org/viewcvs/yast/trunk/bootloader/package/yast2-bootloader.changes?rev=53034&r1=53033&r2=53034&view=diff ============================================================================== --- trunk/bootloader/package/yast2-bootloader.changes (original) +++ trunk/bootloader/package/yast2-bootloader.changes Fri Nov 7 10:33:33 2008 @@ -1,4 +1,13 @@ ------------------------------------------------------------------- +Fri Nov 7 10:29:26 CET 2008 - juhliarik@suse.cz + +- added fix for writing crashkernel to bootloader from kdump on ppc + (bnc #441547) +- added fix for double boot entry twice by efibootmgr (bnc #438215) +- added fix for using fix_chs (bnc #367304) +- 2.17.36 + +------------------------------------------------------------------- Fri Oct 31 12:49:14 CET 2008 - juhliarik@suse.cz - added better proposal checking elilo ,lilo Modified: trunk/bootloader/src/grub/misc.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/bootloader/src/grub/misc.ycp?rev=53034&r1=53033&r2=53034&view=diff ============================================================================== --- trunk/bootloader/src/grub/misc.ycp (original) +++ trunk/bootloader/src/grub/misc.ycp Fri Nov 7 10:33:33 2008 @@ -401,14 +401,6 @@ integer exit = out["exit"]:0; y2milestone ("Command output: %1", out); ret = ret && (0 == exit); - command = sformat ("/usr/sbin/fix_chs %1 %2", - mbr_dev, - num); - y2milestone ("Running command %1", command); - out = (map)SCR::Execute (.target.bash_output, command); - exit = out["exit"]:0; - y2milestone ("Command output: %1", out); - ret = ret && (0 == exit); } } else Modified: trunk/bootloader/src/modules/BootELILO.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/bootloader/src/modules/BootELILO.ycp?rev=53034&r1=53033&r2=53034&view=diff ============================================================================== --- trunk/bootloader/src/modules/BootELILO.ycp (original) +++ trunk/bootloader/src/modules/BootELILO.ycp Fri Nov 7 10:33:33 2008 @@ -490,7 +490,22 @@ "--loader '\\efi\\SuSE\\elilo.efi' --write-signature >> %4 2>&1", BootCommon::globals["boot_efilabel"]:"", boot_disk, boot_part, bl_logfile ); - ret = ret && installBootLoader (bl_command, bl_logfile); + + // command for checking same boot entry in efi bnc #438215 (YaST creates efibootloader entry twice) + string cmd = sformat("/usr/sbin/efibootmgr -v | grep -c \"%1.*HD(%2.*File(.\\efi.\\SuSE.\\elilo.efi)\"", + BootCommon::globals["boot_efilabel"]:"", boot_part); + y2milestone("Command for checking same boot entry: %1", cmd); + + // check how many entries with same label and partition is actually in efi + map out = (map)SCR::Execute (.target.bash_output, cmd); + + // check number of same boot entries in efi + // if boot entry is added -> don't add it again + if (deletechars(out["stdout"]:"","\n") == "0") + ret = ret && installBootLoader (bl_command, bl_logfile); + else + y2milestone("Skip adding boot entry: %1 to EFI. There already exist: %2 with same label and partition.", + BootCommon::globals["boot_efilabel"]:"", deletechars(out["stdout"]:"","\n")); } Modified: trunk/bootloader/src/modules/BootPOWERLILO.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/bootloader/src/modules/BootPOWERLILO.ycp?rev=53034&r1=53033&r2=53034&view=diff ============================================================================== --- trunk/bootloader/src/modules/BootPOWERLILO.ycp (original) +++ trunk/bootloader/src/modules/BootPOWERLILO.ycp Fri Nov 7 10:33:33 2008 @@ -875,22 +875,25 @@ BootCommon::UpdateAppend (); } + // check if there is selected "none" bootloader + string bl = BootCommon::getLoaderType (false); + + if (bl == "none") + { + BootCommon::InitializeLibrary (init, bl); + return true; + } + if (! BootCommon::InitializeLibrary (init, "ppc")) // send current disk/partition information to perl-Bootloader BootCommon::SetDiskInfo (); - // Sanity check the sections list: we can only pass strings - // through the perl interface - list