]> git.ipfire.org Git - thirdparty/grub.git/commit
build: Fix option to explicitly disable memory debugging
authorPatrick Steinhardt <ps@pks.im>
Sat, 7 Mar 2020 16:29:08 +0000 (17:29 +0100)
committerDaniel Kiper <daniel.kiper@oracle.com>
Tue, 10 Mar 2020 20:56:39 +0000 (21:56 +0100)
commit2c43ab4ef792ca910cf905677bf3ffc2b3b431ba
treee127ff6dd34cf1c7f6d21af8d56c827a26193fa1
parent20def1a3c3952982395cd7c3ea7e78638527962b
build: Fix option to explicitly disable memory debugging

The memory management system supports a debug mode that can be enabled
at build time by passing "--enable-mm-debug" to the configure script.
Passing the option will cause us define MM_DEBUG as expected, but in
fact the reverse option "--disable-mm-debug" will do the exact same
thing and also set up the define. This currently causes the build of
"lib/gnulib/base64.c" to fail as it tries to use `grub_debug_malloc()`
and `grub_debug_free()` even though both symbols aren't defined.

Seemingly, `AC_ARG_ENABLE()` will always execute the third argument if
either the positive or negative option was passed. Let's thus fix the
issue by moving the call to`AC_DEFINE()` into an explicit `if test
$xenable_mm_debug` block, similar to how other defines work.

Signed-off-by: Patrick Steinhardt <ps@pks.im>
Reviewed-by: Daniel Kiper <daniel.kiper@oracle.com>
Reviewed-by: Paul Menzel <pmenzel@molgen.mpg.de>
configure.ac