commit:     c8a1d365ff1349cc438352917863ed98ee34a80f
Author:     Sam James <sam <AT> gentoo <DOT> org>
AuthorDate: Wed Jan 11 07:09:53 2023 +0000
Commit:     Sam James <sam <AT> gentoo <DOT> org>
CommitDate: Wed Jan 11 07:16:07 2023 +0000
URL:        https://gitweb.gentoo.org/proj/elfix.git/commit/?id=c8a1d365

fix-gnustack: pass -Wl,-z,execstack for tests with Clang

Clang doesn't create executable stacks by default, so we need to force it
for the purposes of the tests.

Signed-off-by: Sam James <sam <AT> gentoo.org>

 misc/fix-gnustack/tests/Makefile.am | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/misc/fix-gnustack/tests/Makefile.am 
b/misc/fix-gnustack/tests/Makefile.am
index ad573a3..17bec0d 100644
--- a/misc/fix-gnustack/tests/Makefile.am
+++ b/misc/fix-gnustack/tests/Makefile.am
@@ -8,8 +8,10 @@ bad-gnustack.s: bad-gnustack.c
        $(CC) $(CPPFLAGS) $(CFLAGS) -S $<
        $(SED) -i -e 's/GNU-stack,"",/GNU-stack,"x",/' $@
 
+# Clang doesn't create executable stacks by default, so
+# tests for fix-gnustack fail without this.
 bad-gnustack$(EXEEXT): bad-gnustack.s
-       $(CC) $(CFLAGS) $(LDFLAGS) -o $@ $<
+       $(CC) $(CFLAGS) $(LDFLAGS) -Wl,-z,execstack -o $@ $<
 
 check_SCRIPTS = gnustacktest
 TEST = $(check_SCRIPTS)

Reply via email to