[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/1] configure.ac: Fix SDL2 typo by referencing value
From: |
Daniel Kiper |
Subject: |
Re: [PATCH 1/1] configure.ac: Fix SDL2 typo by referencing value |
Date: |
Fri, 11 Aug 2023 18:26:58 +0200 |
User-agent: |
NeoMutt/20170113 (1.7.2) |
On Wed, Jul 19, 2023 at 07:53:39PM +0200, Julian Andres Klode wrote:
> On Wed, Jul 19, 2023 at 12:48:21AM +0200, Michał Grzelak wrote:
> > During configuration of SDL2, variable enable_grub_emu_sdl2 is checked
> > whether to throw an error message. Howerever, error could not happen
> > because two unequal strings were compared. Fix this by referencing
> > value of enable_grub_emu_sdl2, not name.
> >
> > Fixes: 17d6ac1a7 (emu: Add SDL2 support)
> >
> > Signed-off-by: Michał Grzelak <mchl.grzlk@gmail.com>
> > ---
> > configure.ac | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/configure.ac b/configure.ac
> > index 01500ffdf..27c5918fa 100644
> > --- a/configure.ac
> > +++ b/configure.ac
> > @@ -1609,7 +1609,7 @@ if test "$platform" = emu; then
> > AC_SUBST(HAVE_SDL2)],
> > [grub_emu_sdl2_excuse="libSDL2 libraries are required to build
> > \`grub-emu' with SDL2 support"])
> > [fi]
> > - if test x"enable_grub_emu_sdl2" = xyes && test x"$grub_emu_sdl2_excuse"
> > != x ; then
> > + if test x"$enable_grub_emu_sdl2" = xyes && test x"$grub_emu_sdl2_excuse"
> > != x ; then
> > AC_MSG_ERROR([SDL2 support for grub-emu was explicitly requested but
> > can't be compiled ($grub_emu_sdl2_excuse)])
> > fi
> > if test x"$grub_emu_sdl2_excuse" = x ; then
> > --
> > 2.37.3
> >
>
> Reviewed-by: Julian Andres Klode <julian.klode@canonical.com>
Reviewed-by: Daniel Kiper <daniel.kiper@oracle.com>
Daniel
- Re: [PATCH 1/1] configure.ac: Fix SDL2 typo by referencing value,
Daniel Kiper <=