Re: Bug#977223: guile-3.0: FTBFS on hppa - segmentation faults

Hi!

On 2/28/22 11:16, John Paul Adrian Glaubitz wrote:
> On 2/28/22 09:53, John Paul Adrian Glaubitz wrote:
>> I tested your proposed patch on the porterbox perotto.debian.net in a powerpc
>> chroot, but unfortunately it didn't work:
> 
> I think this check is wrong:
> 
> diff --git a/configure.ac b/configure.ac
> index 827e1c09d..dec060414 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -369,6 +369,9 @@ else
>  fi
>  AC_SUBST([SCM_I_GSC_T_PTRDIFF])
>  
> +AM_CONDITIONAL([DEB_GUILE_32_BIT_BIG_ENDIAN],
> +  [test "$ac_cv_words_bigendian" && test "$ac_cv_sizeof_void_p" -lt 8])
> +
>  AC_CHECK_HEADERS([stdatomic.h])
>  
>  AC_MSG_CHECKING([for which prebuilt binary set to use during bootstrap])
> 
> It looks like the correct variable name for the big-endian check is
> 
> 	$ac_cv_c_bigendian

Why not just use the result of this check?

AC_MSG_CHECKING([for which prebuilt binary set to use during bootstrap])
SCM_PREBUILT_BINARIES=
case "$ac_cv_c_bigendian-$ac_cv_sizeof_void_p" in
  yes-8) SCM_PREBUILT_BINARIES=64-bit-big-endian;;
  yes-4) SCM_PREBUILT_BINARIES=32-bit-big-endian;;
  no-8) SCM_PREBUILT_BINARIES=64-bit-little-endian;;
  no-4) SCM_PREBUILT_BINARIES=32-bit-little-endian;;
  *) AC_MSG_ERROR([Unexpected endianness+pointer size combination.])
esac
AC_MSG_RESULT($SCM_PREBUILT_BINARIES)
AC_SUBST([SCM_PREBUILT_BINARIES])

Adrian