diff mbox series

[16/16] configure: No longer mark --enable-thread-safety as EXPERIMENTAL

Message ID 20231010134300.53830-16-mark@klomp.org
State Superseded
Delegated to: Aaron Merey
Headers
Series [01/16] lib: Add new once_define and once macros to eu-config.h |

Commit Message

Mark Wielaard Oct. 10, 2023, 1:43 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: Mark Wielaard <mark@klomp.org>
---
 configure.ac | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

Comments

Mark Wielaard Oct. 12, 2023, 10:09 p.m. UTC | #1 
Hi Heather,

On Tue, Oct 10, 2023 at 03:43:00PM +0200, Mark Wielaard wrote:
> 
> 	* configure.ac (--enable-thread-safety): Remove experimental
> 	warning.
> 

This all looks good. We should push this once we have all libelf fixes
in.

BTW. Your original ChangeLog entry also mentioned a NEWS change, but
that wasn't in your original patch.

Cheers,

Mark
diff mbox series

Patch

diff --git a/configure.ac b/configure.ac
index 29ed32fe..be2103e2 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.])
 
@@ -907,10 +905,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}