|
From: Paul F. <pj...@wa...> - 2026-03-14 11:32:12
|
On 13/03/2026 21:36, Paul Floyd via Valgrind-developers wrote: > > I'll do some tests with GCC to see if we really need this. The > configure test fails with on my system with clang so it's doing > nothing there. > I tried this change as a quick test diff --git a/configure.ac b/configure.ac index ca395be82..2f274dfb1 100644 --- a/configure.ac +++ b/configure.ac @@ -3255,7 +3255,7 @@ CFLAGS="-no-pie -Werror" AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[ ]], [[ return 0; ]])], [ -AC_SUBST([FLAG_NO_PIE], ["-no-pie"]) +AC_SUBST([FLAG_NO_PIE], [""]) AC_MSG_RESULT([yes]) ], [ AC_SUBST([FLAG_NO_PIE], [""]) and on Fedora 42 I got no new failures. The original commit message for this was commit 84ad3a82c247c9b15591b542307772f4c6767129 Author: Ivo Raisr <iv...@iv...> Date: Wed May 10 05:43:03 2017 +0000 Fix compilation problems with some unit tests on Ubuntu 16.10+. Compiler produces position independent executables (PIE) by default which gets in the way of some unit tests. Fixes BZ#377066. In https://bugs.kde.org/show_bug.cgi?id=377066 the compiler errors look similar to what you saw with clang. I'll try to clean up all that. A+ Paul |