Re: [PATCH] configure.ac: avoid bashisms

On 30 Oct 2025, at 12:01 PM, Lars Wendler via Grub-devel <[email protected]> wrote:
> 
> or else configure check doesn't succeed with non-bash shell (e.g. dash):
> 
>  checking whether to enable AMD64 as(1) feature detection... /var/tmp/portage/sys-boot/grub-2.14_rc1/work/grub-2.14~rc1/configure: 39176: test: xx86_64: unexpected operator
>  no
> 
> and later build fails with
> 
>  /var/tmp/portage/sys-boot/grub-2.14_rc1/work/grub-2.14~rc1/grub-core/lib/libgcrypt-grub/src/hwf-x86.c: In function ‘detect_x86_gnuc’:
>  /var/tmp/portage/sys-boot/grub-2.14_rc1/work/grub-2.14~rc1/grub-core/lib/libgcrypt-grub/src/hwf-x86.c:252:17: error: ‘HWF_INTEL_CPU’ undeclared (first use in this function)
>    252 |       result |= HWF_INTEL_CPU;
>        |                 ^~~~~~~~~~~~~
> 
> and other corresponding HWF_INTEL_* definitions because HAVE_CPU_ARCH_X86 was
> erroneously not defined by configure script.
> 
> Signed-off-by: Lars Wendler <[email protected]>

Reviewed-by: Sudhakar Kuppusamy <[email protected]>

> ---
> configure.ac | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/configure.ac b/configure.ac
> index 8a72c078e..17937baf4 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -1150,7 +1150,7 @@ fi
> 
> # Implementation of the --disable-amd64-as-feature-detection switch.
> AC_MSG_CHECKING([whether to enable AMD64 as(1) feature detection])
> -if test x$target_cpu == xx86_64 -a x$platform == xefi; then
> +if test x$target_cpu = xx86_64 -a x$platform = xefi; then
>   CPPFLAGS_GCRY_ASM="-D__x86_64 -DHAVE_CPU_ARCH_X86"
>   AC_ARG_ENABLE(amd64-as-feature-detection,
>     AS_HELP_STRING([--disable-amd64-as-feature-detection],
> -- 
> 2.51.2
> 
> 
> _______________________________________________
> Grub-devel mailing list
> [email protected]
> https://lists.gnu.org/mailman/listinfo/grub-devel


_______________________________________________
Grub-devel mailing list
[email protected]
https://lists.gnu.org/mailman/listinfo/grub-devel