bug#53426: mingw-w64-{x86_64, i686}-winpthreads broken after absorption of binutils-next

bug#53426: mingw-w64-{x86_64, i686}-winpthreads broken after absorption of binutils-next

[Carl Dong]

[-- Attachment #1: Type: text/plain, Size: 609 bytes --]

Hi all,

The mingw-w64-{x86_64,i686}-winpthreads have been broken since the absorption of binutils-next in ce362bc1f35102e7d79ad7e0d638d20cef304f01. My very crude debugging (adding and removing configure flags from the binutils in gnu/packages/base.scm until the packages compiled) led me to the culprit: "--enable-compressed-debug-sections=all”.

I’m wondering what the best course of action is to fix the mingw-w64-{x86_64,i686}-winpthreads packages (e.g. just remove the configure flag for mingw-w64? Add that flag only for linux? Or something else?) and would love some help.

Cheers,
Carl Dong

[-- Attachment #2: Type: text/html, Size: 1154 bytes --]

bug#53426: mingw-w64-{x86_64, i686}-winpthreads broken after absorption of binutils-next

[Maxime Devos]

[-- Attachment #1: Type: text/plain, Size: 562 bytes --]

Carl Dong schreef op vr 21-01-2022 om 16:30 [-0500]:
> I’m wondering what the best course of action is to fix the mingw-w64-
> {x86_64,i686}-winpthreads packages (e.g. just remove the configure
> flag for mingw-w64? Add that flag only for linux? Or something else?)
> and would love some help.

I would assume that it works for Linux, it also works for the Hurd,
so I would prefer something like

#$@(if (target-mingw?) '() '("--some-configuration..."))

above

#$@(if (target-linux?) '("--some-configuration...") '())

Greetings,
Maxime.


[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 260 bytes --]

bug#53426: mingw-w64-{x86_64, i686}-winpthreads broken after absorption of binutils-next

[Carl Dong]

[-- Attachment #1: Type: text/plain, Size: 2004 bytes --]

I did some more exploration, and found that not only are the mingw-w64-{x86_64, i686}-winpthreads packages broken, but any --target=x86_64-w64-mingw32 package:

$ guix build --target=x86_64-w64-mingw32 hello

I have this patch which resolves the problem by simply disabling compressed debug sections, but perhaps it’d also be worthwhile to investigate the root cause...

--8<---------------cut here---------------start------------->8---
diff --git a/gnu/packages/cross-base.scm b/gnu/packages/cross-base.scm
index 78cbf871ac..397e4d4c1c 100644
--- a/gnu/packages/cross-base.scm
+++ b/gnu/packages/cross-base.scm
@@ -101,7 +101,8 @@ (define* (cross-binutils target #:optional (binutils binutils))
                                          "ath9k-htc-firmware-binutils.patch")))
                  ((target-mingw? target)
                   (package-with-extra-patches
-                   binutils
+                   (package-with-extra-configure-variable
+                    binutils "--enable-compressed-debug-sections" "no")
                    (search-patches "binutils-mingw-w64-timestamp.patch"
                                    "binutils-mingw-w64-deterministic.patch")))
                  (else binutils))
--8<---------------cut here---------------end--------------->8---

Cheers,
Carl Dong

> On Jan 21, 2022, at 5:10 PM, Maxime Devos <maximedevos@telenet.be> wrote:
> 
> Carl Dong schreef op vr 21-01-2022 om 16:30 [-0500]:
>> I’m wondering what the best course of action is to fix the mingw-w64-
>> {x86_64,i686}-winpthreads packages (e.g. just remove the configure
>> flag for mingw-w64? Add that flag only for linux? Or something else?)
>> and would love some help.
> 
> I would assume that it works for Linux, it also works for the Hurd,
> so I would prefer something like
> 
> #$@(if (target-mingw?) '() '("--some-configuration..."))
> 
> above
> 
> #$@(if (target-linux?) '("--some-configuration...") '())
> 
> Greetings,
> Maxime.
> 


[-- Attachment #2: Message signed with OpenPGP --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

bug#53426: mingw-w64-{x86_64, i686}-winpthreads broken after absorption of binutils-next

[Michael Ford]

[-- Attachment #1: Type: text/plain, Size: 96 bytes --]

Hi,

I think this bug can be closed now that https://issues.guix.gnu.org/53706
has been merged?

[-- Attachment #2: Type: text/html, Size: 204 bytes --]

bug#53426: mingw-w64-{x86_64, i686}-winpthreads broken after absorption of binutils-next

[Maxim Cournoyer]

Hi Michael,

Michael Ford <fanquake@gmail.com> writes:

> Hi,
>
> I think this bug can be closed now that https://issues.guix.gnu.org/53706
> has been merged?

I think so too!  Thanks for the heads-up.

Maxim