From 7cf80be345190813bcd972b7688bcb928e58224d Mon Sep 17 00:00:00 2001 From: Michael Jeanson Date: Fri, 12 Nov 2021 14:51:57 -0500 Subject: [PATCH] fix: remove autoconf features default value in help message The default values of yes|no can be confusing combined with the --enable / --disable switches of autoconf, remove them from the help message. Before : --disable-man-pages Do not build and install man pages (already built in a distributed tarball [default=yes] After : --disable-man-pages Do not build and install man pages (already built in a distributed tarball Change-Id: I7a290aaacbcd0d04e0efd161357755451e5d4d4d Signed-off-by: Michael Jeanson Reviewed-on: https://review.lttng.org/c/babeltrace/+/6742 Reviewed-by: Simon Marchi Tested-by: jenkins --- m4/ae_config_feature.m4 | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/m4/ae_config_feature.m4 b/m4/ae_config_feature.m4 index 7582c00f..1d7cd583 100644 --- a/m4/ae_config_feature.m4 +++ b/m4/ae_config_feature.m4 @@ -95,7 +95,7 @@ # modified version of the Autoconf Macro, you may extend this special # exception to the GPL to apply to your modified version as well. -#serial 1 +#serial 2 # AE_FEATURE_DEFAULT_ENABLE: The next feature defined with AE_FEATURE will @@ -211,7 +211,7 @@ m4_pushdef([FEATURE], patsubst([$1], -, _))dnl ]) -dnl Disabled by default, unless already overriden +dnl Disabled by default, unless already overridden m4_ifndef([ae_feature_default_arg],[AE_FEATURE_DEFAULT_DISABLE]) @@ -230,7 +230,7 @@ AS_IF([test "x$enable_[]FEATURE[]" = x && test "ae_feature_default_arg" = yes],[ AC_ARG_ENABLE([$1], AS_HELP_STRING([--ae_feature_default_switch-$1],dnl - [$2 [default=ae_feature_default_arg]]),[ + [$2]),[ case "${enableval}" in yes) enable_[]FEATURE[]=yes -- 2.34.1