Skip to content

Commit 1f5da25

Browse files
committed
configure.ac: rename 'adapterTuple' as 'adapter_driver'
Commit ce3bf66 ("configure.ac: rename M4 macro 'adapter' to prevent accidental conflicts") renames the macro as 'adapterTuple' but since the macro name is printed in error messages, this creates cryptic errors like: configure: error: header sys/mman.h is required for adapterTuple "Bitbanging on EP93xx-based SBCs". Rename it as 'adapter_driver'. It keeps valid the purpose of the former renaming, while keeping readable the error message. Change-Id: Idd68270fbdf879153cd59f4cacf5036aa599b251 Signed-off-by: Antonio Borneo <[email protected]> Fixes: ce3bf66 ("configure.ac: rename M4 macro 'adapter' to prevent accidental conflicts") Reviewed-on: https://review.openocd.org/c/openocd/+/9160 Tested-by: jenkins
1 parent 7c16c38 commit 1f5da25

File tree

1 file changed

+23
-22
lines changed

1 file changed

+23
-22
lines changed

configure.ac

Lines changed: 23 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -301,15 +301,15 @@ AS_IF([test "x$debug_malloc" = "xyes" -a "x$have_glibc" = "xyes"], [
301301
])
302302

303303
m4_define([AC_ARG_ADAPTERS], [
304-
m4_foreach([adapterTuple], [$1],
305-
[AC_ARG_ENABLE(ADAPTER_OPT([adapterTuple]),
306-
AS_HELP_STRING([--enable-ADAPTER_OPT([adapterTuple])[[[=yes/no/auto]]]],
307-
[Enable building support for the ]ADAPTER_DESC([adapterTuple])[ (default is $2)]),
304+
m4_foreach([adapter_driver], [$1],
305+
[AC_ARG_ENABLE(ADAPTER_OPT([adapter_driver]),
306+
AS_HELP_STRING([--enable-ADAPTER_OPT([adapter_driver])[[[=yes/no/auto]]]],
307+
[Enable building support for the ]ADAPTER_DESC([adapter_driver])[ (default is $2)]),
308308
[case "${enableval}" in
309309
yes|no|auto) ;;
310-
*) AC_MSG_ERROR([Option --enable-ADAPTER_OPT([adapterTuple]) has invalid value "${enableval}".]) ;;
310+
*) AC_MSG_ERROR([Option --enable-ADAPTER_OPT([adapter_driver]) has invalid value "${enableval}".]) ;;
311311
esac],
312-
[ADAPTER_VAR([adapterTuple])=$2])
312+
[ADAPTER_VAR([adapter_driver])=$2])
313313
])
314314
])
315315

@@ -607,23 +607,24 @@ PKG_CHECK_MODULES([LIBJAYLINK], [libjaylink >= 0.2],
607607
# Arg $3: What prerequisites are missing, to be shown in an error message
608608
# if an adapter was requested but cannot be enabled.
609609
m4_define([PROCESS_ADAPTERS], [
610-
m4_foreach([adapterTuple], [$1], [
610+
m4_foreach([adapter_driver], [$1], [
611611
AS_IF([test $2], [
612-
AS_IF([test "x$ADAPTER_VAR([adapterTuple])" != "xno"], [
613-
AC_DEFINE([BUILD_]ADAPTER_SYM([adapterTuple]), [1],
614-
[1 if you want the ]ADAPTER_DESC([adapterTuple]).)
612+
AS_IF([test "x$ADAPTER_VAR([adapter_driver])" != "xno"], [
613+
AC_DEFINE([BUILD_]ADAPTER_SYM([adapter_driver]), [1],
614+
[1 if you want the ]ADAPTER_DESC([adapter_driver]).)
615615
], [
616-
AC_DEFINE([BUILD_]ADAPTER_SYM([adapterTuple]), [0],
617-
[0 if you do not want the ]ADAPTER_DESC([adapterTuple]).)
616+
AC_DEFINE([BUILD_]ADAPTER_SYM([adapter_driver]), [0],
617+
[0 if you do not want the ]ADAPTER_DESC([adapter_driver]).)
618618
])
619619
], [
620-
AS_IF([test "x$ADAPTER_VAR([adapterTuple])" = "xyes"], [
621-
AC_MSG_ERROR([$3 is required for [adapterTuple] "ADAPTER_DESC([adapterTuple])".])
620+
AS_IF([test "x$ADAPTER_VAR([adapter_driver])" = "xyes"], [
621+
AC_MSG_ERROR([$3 is required for [adapter_driver] "ADAPTER_DESC([adapter_driver])".])
622622
])
623-
ADAPTER_VAR([adapterTuple])=no
624-
AC_DEFINE([BUILD_]ADAPTER_SYM([adapterTuple]), [0], [0 if you do not want the ]ADAPTER_DESC([adapterTuple]).)
623+
ADAPTER_VAR([adapter_driver])=no
624+
AC_DEFINE([BUILD_]ADAPTER_SYM([adapter_driver]), [0],
625+
[0 if you do not want the ]ADAPTER_DESC([adapter_driver]).)
625626
])
626-
AM_CONDITIONAL(ADAPTER_SYM([adapterTuple]), [test "x$ADAPTER_VAR([adapterTuple])" != "xno"])
627+
AM_CONDITIONAL(ADAPTER_SYM([adapter_driver]), [test "x$ADAPTER_VAR([adapter_driver])" != "xno"])
627628
])
628629
])
629630

@@ -838,7 +839,7 @@ echo
838839
echo
839840
echo OpenOCD configuration summary
840841
echo ---------------------------------------------------
841-
m4_foreach([adapterTuple], [USB1_ADAPTERS,
842+
m4_foreach([adapter_driver], [USB1_ADAPTERS,
842843
HIDAPI_ADAPTERS, HIDAPI_USB1_ADAPTERS, LIBFTDI_ADAPTERS,
843844
LIBFTDI_USB1_ADAPTERS,
844845
LIBGPIOD_ADAPTERS,
@@ -860,8 +861,8 @@ m4_foreach([adapterTuple], [USB1_ADAPTERS,
860861
DUMMY_ADAPTER,
861862
OPTIONAL_LIBRARIES,
862863
COVERAGE],
863-
[s=m4_format(["%-49s"], ADAPTER_DESC([adapterTuple]))
864-
AS_CASE([$ADAPTER_VAR([adapterTuple])],
864+
[s=m4_format(["%-49s"], ADAPTER_DESC([adapter_driver]))
865+
AS_CASE([$ADAPTER_VAR([adapter_driver])],
865866
[auto], [
866867
echo "$s"yes '(auto)'
867868
],
@@ -873,8 +874,8 @@ m4_foreach([adapterTuple], [USB1_ADAPTERS,
873874
],
874875
[
875876
AC_MSG_ERROR(m4_normalize([
876-
Error in [adapterTuple] "ADAPTER_ARG([adapterTuple])": Variable "ADAPTER_VAR([adapterTuple])"
877-
has invalid value "$ADAPTER_VAR([adapterTuple])".]))
877+
Error in [adapter_driver] "ADAPTER_ARG([adapter_driver])": Variable "ADAPTER_VAR([adapter_driver])"
878+
has invalid value "$ADAPTER_VAR([adapter_driver])".]))
878879
])
879880
])
880881
echo

0 commit comments

Comments
 (0)