diff mbox series

[9/9,v2] configure: No longer mark --enable-thread-safety as EXPERIMENTAL

Message ID 20240717223409.1788719-10-amerey@redhat.com
State Superseded
Headers
Series Fix thread-safety for elfutils |

Commit Message

Aaron Merey July 17, 2024, 10:34 p.m. UTC 
  From: Heather McIntyre <hsm2@rice.edu>

        * configure.ac (--enable-thread-safety): Remove experimental
        warning.

Signed-off-by: Heather S. McIntyre <hsm2@rice.edu>
Signed-off-by: Aaron Merey <amerey@redhat.com>
Signed-off-by: Mark Wielaard <mark@klomp.org>
---

No changes in v2.  However PR31967 (datarace in elf_compress[_gnu])
was recently filed and this patch set does not address it.  Maybe
we should hold off on merging this patch and keep thread safety
"experimental" until PR31967 is fixed.

 configure.ac | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)
diff mbox series

Patch

diff --git a/configure.ac b/configure.ac
index 24e68d94..2c885c8a 100644
--- a/configure.ac
+++ b/configure.ac
@@ -79,12 +79,10 @@  AC_DEFINE_UNQUOTED(DEFAULT_AR_DETERMINISTIC, $default_ar_deterministic,
 
 AC_ARG_ENABLE([thread-safety],
 AS_HELP_STRING([--enable-thread-safety],
-               [enable thread safety of libraries EXPERIMENTAL]),
+               [enable thread safety of libraries]),
                use_locks=$enableval, use_locks=no)
 AM_CONDITIONAL(USE_LOCKS, test "$use_locks" = yes)
 AS_IF([test "$use_locks" = yes], [AC_DEFINE(USE_LOCKS)])
-AS_IF([test "$use_locks" = yes],
-      [AC_MSG_WARN([thread-safety is EXPERIMENTAL tests might fail.])])
 
 AH_TEMPLATE([USE_LOCKS], [Defined if libraries should be thread-safe.])
 
@@ -939,10 +937,10 @@  AC_MSG_NOTICE([
     Symbol versioning                  : ${enable_symbol_versioning}
 
   NOT RECOMMENDED FEATURES (should all be no)
-    Experimental thread safety         : ${use_locks}
     install elf.h                      : ${install_elfh}
 
   OTHER FEATURES
+    Enable thread safety               : ${use_locks}
     Deterministic archives by default  : ${default_ar_deterministic}
     Native language support            : ${USE_NLS}
     Extra Valgrind annotations         : ${use_vg_annotations}