[PATCH COMMITTED] configure: Fix spelling of -Wl,--no-error-execstack option

Sam James sam@gentoo.org
Fri Mar 7 07:21:05 GMT 2025


Florian Weimer <fweimer@redhat.com> writes:

> BFD ld recognizes all -no-* options (with a single leading dash)
> unconditionally.
>
> Fixes commit a2bd5008a99032830add3e4005c25b61e3207112
> ("Pass -Wl,--no-error-execstack for tests where -Wl,-z,execstack
> is used [PR32717]").

Thanks. The "helpful" disambiguation it does is why I didn't notice.

>
> ---
>  configure    | 4 ++--
>  configure.ac | 2 +-
>  2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/configure b/configure
> index 2a996cb0d9..674d1d7e4a 100755
> --- a/configure
> +++ b/configure
> @@ -7122,7 +7122,7 @@ cat > conftest.c <<EOF
>  int _start (void) { return 42; }
>  EOF
>  if { ac_try='${CC-cc} $CFLAGS $CPPFLAGS $LDFLAGS $no_ssp
> -		  -Wl,-no-error-execstack -nostdlib -nostartfiles
> +		  -Wl,--no-error-execstack -nostdlib -nostartfiles
>  		  -fPIC -shared -o conftest.so conftest.c
>  		  1>&5'
>    { { eval echo "\"\$as_me\":${as_lineno-$LINENO}: \"$ac_try\""; } >&5
> @@ -7131,7 +7131,7 @@ if { ac_try='${CC-cc} $CFLAGS $CPPFLAGS $LDFLAGS $no_ssp
>    printf "%s\n" "$as_me:${as_lineno-$LINENO}: \$? = $ac_status" >&5
>    test $ac_status = 0; }; }
>  then
> -  if ${CC-cc} $CFLAGS $CPPFLAGS $LDFLAGS $no_ssp -Wl,-no-error-execstack -nostdlib \
> +  if ${CC-cc} $CFLAGS $CPPFLAGS $LDFLAGS $no_ssp -Wl,--no-error-execstack -nostdlib \
>        -nostartfiles -fPIC -shared -o conftest.so conftest.c 2>&1 \
>        | grep "warning: --no-error-execstack ignored" > /dev/null 2>&1; then
>      true
> diff --git a/configure.ac b/configure.ac
> index 5f6f0b72bc..57cd24c87d 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -1318,7 +1318,7 @@ if test $libc_cv_as_noexecstack = yes; then
>  fi
>  AC_SUBST(ASFLAGS_config)
>  
> -LIBC_LINKER_FEATURE([--no-error-execstack], [-Wl,-no-error-execstack],
> +LIBC_LINKER_FEATURE([--no-error-execstack], [-Wl,--no-error-execstack],
>  		    [libc_cv_no_error_execstack=yes], [libc_cv_no_error_execstack=no])
>  AC_SUBST(libc_cv_no_error_execstack)
>  
>
> base-commit: 5b309075456e07b6231db331092a1f307cb7664b


More information about the Libc-alpha mailing list