#1569 PR merged: Honor $KERNEL_CMDLINE in bootloaders

Labels: enhancement, bug, cleanup, fixed / solved / done

OliverO2 opened issue at 2017-11-08 13:30:

Currently, some bootloader configurations do not honor the $KERNEL_CMDLINE configuration variable. This PR aims to fix the situation.

Background: I am trying to integrate support for TCG Opal-encrypted disks in ReaR. To do so, ReaR has to honor a setting like KERNEL_CMDLINE="$KERNEL_CMDLINE libata.allow_tpm=1" on rescue systems.

I've tried to fix all affected spots in ReaR, but did not dare to touch the section beginning at line 175 in usr/share/rear/output/USB/Linux-i386/300_create_extlinux.sh:

# We generate a ReaR syslinux.cfg based on existing ReaR syslinux.cfg files.
Log "Creating /rear/syslinux.cfg"
{
    syslinux_write <<EOF
label rear
    say Relax-and-Recover - Recover $HOSTNAME from $time
    menu hide
    kernel $HOSTNAME-$time

EOF

I just did not do sufficient research do discover the motivation behind all the different syslinux configurations in that file.

Also, I am aware that duplicate code for bootloader configuration exists and probably more stuff from usr/share/rear/lib/bootloader-functions.sh could be used across ReaR. It is not my intention to re-structure all that code as I do not have the resources to test such extensive changes.

jsmeix commented at 2017-11-08 15:30:

@gozora
I dare to assign it to you because I see "bootloader"
and the code change contains "EFI" and even "ebiso" ;-)

jsmeix commented at 2017-11-08 15:32:

It seems there is also a cleanup in
output/USB/Linux-i386/100_create_efiboot.sh

gozora commented at 2017-11-09 15:39:

@OliverO2 Thanks for this contribution!

V.

OliverO2 commented at 2017-11-09 15:41:

@gozora, @gdha Thanks for reviewing!


[Export of Github issue for rear/rear.]