On Sat, Feb 25, 2023 at 8:41 PM Javier Martinez Canillas <javi...@redhat.com>
wrote:

> Oliver Steffen <ostef...@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 <ostef...@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.
>

Yes! Thanks! I'll fix that in the next round :-)

-Oliver



> --
> Best regards,
>
> Javier Martinez Canillas
> Core Platforms
> Red Hat
>
>
_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/grub-devel

Reply via email to