Thank you!

I think the patch below belongs better to gnulib's m4/host-cpu-c-abi.m4,
could you propose a patch for it instead?  Or if someone on the gnulib
list (cc'ed) has ideas how to adapt the patch below into gnulib.

https://git.savannah.gnu.org/cgit/gnulib.git/tree/m4/host-cpu-c-abi.m4

I think upstream 'guile-fibers' should incorporate a newer version of
host-cpu-c-abi.m4 (after that has been fixed), and do a new release, and
then the Debian packaging gets the fix.

We can add a temporary workaround until that is complete, but I don't
think patching the ./configure script directly works since that file
should be discarded and re-generated.  Doesn't that happen when building
the Debian package for you?  How did you locally verify the patch?

Also looking at the build log:

https://buildd.debian.org/status/fetch.php?pkg=guile-fibers&arch=loong64&ver=1.3.1-4&stamp=1740038860&raw=0

checking build system type... loongarch64-unknown-linux-gnu
checking host system type... loongarch64-unknown-linux-gnu
...
checking 32-bit host C ABI... no

Isn't that the right result?  Is that the failing compilation you refer
to, or is there another build log to look at?

/Simon

wuruilong <wuruil...@loongson.cn> writes:

> Package: guile-fibers
> Severity: normal
> Tags: sid patch
> X-Debbugs-Cc: wuruil...@loongson.cn
> User: debian-loonga...@lists.debian.org
> Usertags: loong64
>
> Dear Maintainer,
>
>     guile-fibers compiles incorrectly on loongarch architecture, refer
> to riscv to modify configure file to adapt. It has been tested
> locally, please merge the attached patch.
>
> wuruilong
>
> --- guile-fibers-1.3.1.orig/configure
> +++ guile-fibers-1.3.1/configure
> @@ -7134,6 +7134,26 @@ ia64-*-hpux*)
>    fi
>    rm -rf conftest*
>    ;;
> +         loong64 | loongarch64 )
> +           # There are 6 ABIs: ilp32, ilp32f, ilp32d, lp64, lp64f, lp64d.
> +           # Size of 'long' and 'void *':
> +           cat confdefs.h - <<_ACEOF >conftest.$ac_ext
> +/* end confdefs.h.  */
> +#if defined __LP64__
> +                    int ok;
> +                  #else
> +                    error fail
> +                  #endif
> +
> +_ACEOF
> +if ac_fn_c_try_compile "$LINENO"
> +then :
> +  gl_cv_host_cpu_c_abi_32bit=no
> +else $as_nop
> +  gl_cv_host_cpu_c_abi_32bit=yes
> +fi
> +rm -f core conftest.err conftest.$ac_objext conftest.beam conftest.$ac_ext
> +           ;;
>  
>  mips64*-*linux*)
>    # Find out what ABI is being produced by ac_compile, and set linker
> @@ -13283,7 +13303,7 @@ else $as_nop
>         case "$gl_cv_host_cpu_c_abi" in
>           i386 | x86_64-x32 | arm | armhf | arm64-ilp32 | hppa | ia64-ilp32 | 
> mips | mipsn32 | powerpc | riscv*-ilp32* | s390 | sparc)
>             gl_cv_host_cpu_c_abi_32bit=yes ;;
> -         x86_64 | alpha | arm64 | hppa64 | ia64 | mips64 | powerpc64 | 
> powerpc64-elfv2 | riscv*-lp64* | s390x | sparc64 )
> +         x86_64 | alpha | arm64 | hppa64 | ia64 | loongarch64 | mips64 | 
> powerpc64 | powerpc64-elfv2 | riscv*-lp64* | s390x | sparc64 )
>             gl_cv_host_cpu_c_abi_32bit=no ;;
>           *)
>             gl_cv_host_cpu_c_abi_32bit=unknown ;;
>

Attachment: signature.asc
Description: PGP signature

Reply via email to