]> git.ipfire.org Git - thirdparty/automake.git/commit
tests: fix yacc C++ tests with some C++ compilers
authorMike Frysinger <vapier@gentoo.org>
Sun, 20 Feb 2022 19:28:11 +0000 (14:28 -0500)
committerMike Frysinger <vapier@gentoo.org>
Thu, 24 Feb 2022 04:04:41 +0000 (23:04 -0500)
commit13ae585a0184a4c0113fd4f3a19a0cedf1cfe5c2
treea4760e4f2f13e6709f951364a1fc991e74073eb2
parent2cd42b7cd55b2713c7d6d976dc0791875ea7c3a5
tests: fix yacc C++ tests with some C++ compilers

Fixes automake bug https://bugs.gnu.org/20031.

The C++ standard does not require symbols be placed into the global
namespace, just in the std namespace.  The GNU implementation will
place symbols in both.  For our specific code, we don't care either.

Unfortunately, it looks like generated flex code assumes that some
stdlib.h symbols (free, malloc, exit) are in the global namespace,
even when compiling for C++.  So when we include <cstdlib> but not
<stdlib.h>, we might not get the symbols in the global namespace.

We can workaround this by including stdlib.h in these tests without
invalidating the point of the tests in general.

* t/yacc-cxx.sh: Include stdlib.h.
* t/yacc-d-cxx.sh: Likewise.
* t/yacc-mix-c-cxx.sh: Likewise.
t/yacc-cxx.sh
t/yacc-d-cxx.sh
t/yacc-mix-c-cxx.sh