-
Notifications
You must be signed in to change notification settings - Fork 246
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Simplifications for USB_BOOTLOADER="grub" #2659
Conversation
Simplified test when output/USB/Linux-i386/300_create_extlinux.sh is skipped because GRUB2 is specified to be used as USB bootloader and have that test at the very beginning of 300_create_extlinux.sh to completely skip it.
Simpler and more straightforward code in output/USB/Linux-i386/300_create_grub.sh
Simplified test when output/USB/Linux-i386/850_make_USB_bootable.sh is skipped because GRUB2 is specified to be used as USB bootloader
In default.conf tell that USB_BOOTLOADER="grub" uses GRUB2 as bootloader for USB and that GRUB1 is not supported.
Use local variables actually as local variables. Tell that grub2_set_usb_root is a global variable that is used in the create_grub2_cfg() function
Better wording in default.conf that USB_BOOTLOADER="grub" is only for systems "with BIOS" and that "GRUB Legacy" (not "GRUB1") is not supported.
To be on the safe side have the label "REARBOOT" only 8 characters long.
Use GRUB2_SET_USB_ROOT instead of grub2_set_usb_root for that global variable
Use GRUB2_SET_USB_ROOT instead of grub2_set_usb_root for that global variable
Use GRUB2_SET_USB_ROOT instead of grub2_set_usb_root for that global variable
Comment that explains where the $BUILD_DIR/outputfs/$USB_PREFIX directory is needed.
This pull request is a part of #2648 |
Choose the right GRUB2 config depending on what there is on the original system and error out here if there is neither /boot/grub/grub.cfg nor /boot/grub2/grub.cfg
grub[2]-install creates the $BUILD_DIR/outputfs/boot/grub[2] sub-directory that is needed to create the GRUB2 config $BUILD_DIR/outputfs/boot/grub[2].cfg in the next step
With the recent adaptions With etc/rear/local.conf
I get
That USB disk boots with GRUB2 What does not yet work is: Selection in the GRUB2 boot menue what to boot i.e.
Booting the existing system from the built-in local harddisk does not work
The GRUB2 default is to boot the recovery system via |
I think some things will get already fixed by So my plan is to merge this pull request here soon |
@rear/contributors |
you are correct, #2661 fixes some of the issues you face with grub config. |
Type: Enhancement
Impact: Normal
Reference to related issue (URL):
added grub bootloader for USB #2655
How was this pull request tested?
Not at all tested by me
Brief description of the changes in this pull request:
Simplifications for USB_BOOTLOADER="grub"