Re: [PATCH v2 4/7] util/grub.d: activate bli module on EFI

[Javier Martinez Canillas]

Oliver Steffen <osteffen@redhat.com> writes:

Hello Oliver,

> Add a new configuration drop-in file that loads the
> boot-loader-interface (bli) module and runs the command in case we are
> booting on  the EFI platform.
>
> Signed-off-by: Oliver Steffen <osteffen@redhat.com>
> ---

[...]

> +
> +cat << EOF
> +if [ "\$grub_platform" = "efi" ]; then
> +  insmod boot-loader-interface
> +  boot-loader-interface
> +fi

Just a drop by comment, you renamed in v2 the module and command to
"bli", so this snippet won't work anymore. I also noticed that you
missed to update the module and command name in some commit messages
as well.

-- 
Best regards,

Javier Martinez Canillas
Core Platforms
Red Hat