[core-updates-frozen] "multiple definition of..." build failures

Sarah Morgensen wrote on 28 Aug 2021 23:23

Recipients:(address . bug-guix@gnu.org)

Message-ID:86k0k5462m.fsf@mgsn.dev

Hello Guix,

GCC 10 changed the default behavior when encountering multiple

definitions of a global (often an enum) [0]. This causes error messages

like [1]:

Toggle snippet (3 lines)

ld: a.o:(.bss+0x0): multiple definition of `a'; main.o:(.data+0x0): first defined here

Gentoo's tracker for this issue [2] also has some more info.  I manually
searched Cuirass build logs and found 28 packages failing on x86_64 with
this error:

arcan-sdl@0.5.5.2-1.b4dd1fb
aris@2.2
blastem@0.6.2
chocolate-doom@3.0.1
crispy-doom@5.8.0
geeqie@1.5
glabels@3.4.1
gmtp@1.3.11
gpredict@2.2.1
gtksheet@4.3.5
jumpnbump@1.61
lablgtk@2.18.10
libappindicator@12.10.0
libopenshot@0.2.5
libsrtp@2.3.0
linsmith@0.99.31
megaglest@3.13.0
mupen64plus-core@2.5
ocaml4.07-lablgtk@1.18.10
ocl-icd@2.2.12
opencpn@5.0.0
pcsxr@1.9.95
pipewire@0.2.7
transcode@1.1.7
transmission-remote-gtk@1.4.1
uget@2.2.1
uim-gtk@1.8.8
uim-qt@1.8.8

I submitted an issue upstream for transmission-remote-gtk [3], but I
haven't taken a look at the others.  Any takers?

[0] https://gcc.gnu.org/gcc-10/porting_to.html
[1] https://wiki.gentoo.org/wiki/Project:Toolchain/Gcc_10_porting_notes/fno_common
[2] https://bugs.gentoo.org/705764
[3] https://github.com/transmission-remote-gtk/transmission-remote-gtk/issues/122

--
Sarah

Ludovic Courtès wrote on 4 Sep 2021 16:41

Recipients:(name . Sarah Morgensen)(address . iskarian@mgsn.dev)(address . 50243@debbugs.gnu.org)

Message-ID:87h7f0z9ml.fsf@gnu.org

Hi Sarah,

Sarah Morgensen <iskarian@mgsn.dev> skribis:

Toggle quote (36 lines)> Gentoo's tracker for this issue [2] also has some more info.  I manually
> searched Cuirass build logs and found 28 packages failing on x86_64 with
> this error:
>
> arcan-sdl@0.5.5.2-1.b4dd1fb
> aris@2.2
> blastem@0.6.2
> chocolate-doom@3.0.1
> crispy-doom@5.8.0
> geeqie@1.5
> glabels@3.4.1
> gmtp@1.3.11
> gpredict@2.2.1
> gtksheet@4.3.5
> jumpnbump@1.61
> lablgtk@2.18.10
> libappindicator@12.10.0
> libopenshot@0.2.5
> libsrtp@2.3.0
> linsmith@0.99.31
> megaglest@3.13.0
> mupen64plus-core@2.5
> ocaml4.07-lablgtk@1.18.10
> ocl-icd@2.2.12
> opencpn@5.0.0
> pcsxr@1.9.95
> pipewire@0.2.7
> transcode@1.1.7
> transmission-remote-gtk@1.4.1
> uget@2.2.1
> uim-gtk@1.8.8
> uim-qt@1.8.8
>
> I submitted an issue upstream for transmission-remote-gtk [3], but I
> haven't taken a look at the others.  Any takers?

Great that you built a list of affected packages.

I guess there are several options:

1. Build with ‘-fcommon’, effectively restoring pre-GCC-10 behavior;

2. Build with GCC 8.x or so, for example by adding it to

‘native-inputs’;

3. Upgrade packages and hope for the best;

4. Add the missing ‘extern’ keyword in declarations, assuming that

really is the problem.

I suppose we’ll have to see on a case-by-cases basis which approach is

the easiest. Since we have a list, let’s parallelize work and

synchronize on IRC and in this issue!

Thanks,

Ludo’.

Sarah Morgensen wrote on 5 Sep 2021 04:41

Recipients:(name . Ludovic Courtès)(address . ludo@gnu.org)(address . 50243@debbugs.gnu.org)

Message-ID:86a6krkam1.fsf@mgsn.dev

Hi Ludo,

Toggle quote (4 lines)

>> lablgtk@2.18.10

>> libsrtp@2.3.0

>> ocaml4.07-lablgtk@1.18.10

I've sent patches [0] [1] to fix these by updating them. Together they

have 79 dependents.

[0] https://issues.guix.gnu.org/50388

[1] https://issues.guix.gnu.org/50389

Sarah

Sarah Morgensen wrote on 5 Sep 2021 08:00

Recipients:(name . Ludovic Courtès)(address . ludo@gnu.org)(address . 50243@debbugs.gnu.org)

Message-ID:867dfvk1d1.fsf@mgsn.dev

Hi again,

Toggle quote (3 lines)

>> libappindicator@12.10.0

>> pipewire@0.2.7

I've sent patches for these [0][1], with a total of 22 dependents.

The remaining packages have a total of 19 dependents, 7 of which belong

to ocl-icd@2.2.12. As soon as ocl-icd's website is back online, updating

it to 2.2.13 should fix the build.

[0] https://issues.guix.gnu.org/50392

[1] https://issues.guix.gnu.org/50396

Sarah

Guillaume Le Vaillant wrote on 5 Sep 2021 11:52

Recipients:(name . Sarah Morgensen)(address . iskarian@mgsn.dev)

Message-ID:877dfv2vpe.fsf@kitej

Sarah Morgensen <iskarian@mgsn.dev> skribis:

Toggle quote (2 lines)

> opencpn@5.0.0

Hi,

I fixed opencpn in 79edda38742b37603ccd48dd8660ff7fcfca293e.

-----BEGIN PGP SIGNATURE-----

iIUEAREKAC0WIQTLxZxm7Ce5cXlAaz5r6CCK3yH+PwUCYTST/Q8cZ2x2QHBvc3Rl

by5uZXQACgkQa+ggit8h/j83agEApQWzYudDBVzR0mTeZd9Ml65cOgggSeCrhp+S

fhoicfcA/0fvXu1B80bxEFUsyBHf5TxWyfAA52dte/Et7jW7pOJ4

=fNeK

-----END PGP SIGNATURE-----

Guillaume Le Vaillant wrote on 5 Sep 2021 13:28

Recipients:(name . Sarah Morgensen)(address . iskarian@mgsn.dev)

Message-ID:8735qj2r5q.fsf@kitej

Sarah Morgensen <iskarian@mgsn.dev> skribis:

Toggle quote (4 lines)

> The remaining packages have a total of 19 dependents, 7 of which belong

> to ocl-icd@2.2.12. As soon as ocl-icd's website is back online, updating

> it to 2.2.13 should fix the build.

I just saw that on the master branch ocl-icd has been deprecated and
replaced by opencl-icd-loader. However I've not tried merging master
into core-updates-frozen to see if it the build works.

-----BEGIN PGP SIGNATURE-----

iIUEAREKAC0WIQTLxZxm7Ce5cXlAaz5r6CCK3yH+PwUCYTSrAQ8cZ2x2QHBvc3Rl

by5uZXQACgkQa+ggit8h/j8n2QD+MywR+Vh8jTH+Fels5M2HBtdFpvxVs14/vY5B

CpTjUMkA/iQhPvF6W6bP2l4UIhY+j1A3g/J/WoG/Kn+w3kX+Piny

=mURb

-----END PGP SIGNATURE-----

Sarah Morgensen wrote on 6 Sep 2021 04:52

Recipients:(name . Guillaume Le Vaillant)(address . glv@posteo.net)

Message-ID:8635qijtz4.fsf@mgsn.dev

Guillaume Le Vaillant <glv@posteo.net> writes:

Toggle quote (10 lines)> Sarah Morgensen <iskarian@mgsn.dev> skribis:
>
>> The remaining packages have a total of 19 dependents, 7 of which belong
>> to ocl-icd@2.2.12.  As soon as ocl-icd's website is back online, updating
>> it to 2.2.13 should fix the build.
>
> I just saw that on the master branch ocl-icd has been deprecated and
> replaced by opencl-icd-loader. However I've not tried merging master
> into core-updates-frozen to see if it the build works.

I've just submitted a patch to fix ocl-icd [0].

Hmmm. It looks like it was removed because it "disappeared", but it

just changed locations...

[0] https://issues.guix.gnu.org/50418

Sarah

Guillaume Le Vaillant wrote on 10 Sep 2021 10:23

Recipients:(name . Sarah Morgensen)(address . iskarian@mgsn.dev)(address . 50243@debbugs.gnu.org)

Message-ID:87r1dwsurp.fsf@kitej

Sarah Morgensen <iskarian@mgsn.dev> skribis:

Toggle quote (3 lines)

> gpredict@2.2.1

> transcode@1.1.7

gpredict and transcode fixed in bd8013ab33159a41a94f6a6cd023d585c91ae2ed

and 92d04bcab34ef7145bd18e5a7d242e38d01d3922.

-----BEGIN PGP SIGNATURE-----

iIUEAREKAC0WIQTLxZxm7Ce5cXlAaz5r6CCK3yH+PwUCYTsWOg8cZ2x2QHBvc3Rl

by5uZXQACgkQa+ggit8h/j+hYwD+LGb1pVhrxugbh8Mro4DZh3P5IsHySl+8jlEv

DGQAdhAA/iCCll8ak9vjN/N+wd9UlbFnFky7oMHKehva5BbZm2Bp

=OtEn

-----END PGP SIGNATURE-----

Guillaume Le Vaillant wrote on 16 Sep 2021 10:48

Recipients:(name . Sarah Morgensen)(address . iskarian@mgsn.dev)(address . 50243@debbugs.gnu.org)

Message-ID:87o88soqd4.fsf@kitej

Sarah Morgensen <iskarian@mgsn.dev> skribis:

Toggle quote (8 lines)

> arcan-sdl@0.5.5.2-1.b4dd1fb

> aris@2.2

> blastem@0.6.2

> crispy-doom@5.8.0

> geeqie@1.5

> glabels@3.4.1

> gmtp@1.3.11

These are fixed in c9f7770eeec3a7b493dcbf45c96a4528f2635604 and

following.

-----BEGIN PGP SIGNATURE-----

iIUEAREKAC0WIQTLxZxm7Ce5cXlAaz5r6CCK3yH+PwUCYUMFhw8cZ2x2QHBvc3Rl

by5uZXQACgkQa+ggit8h/j8cJgD5AfYpVhV5vXLchit3/ClrT0Dk++IINqw65DRd

y8u27DEA/22gCC4Jf68qUutaorCQkWNCA9r4zOG9ituGFxnGqQIs

=FvxV

-----END PGP SIGNATURE-----

Ricardo Wurmus wrote on 22 Nov 2021 18:25

[core-updates-frozen] "multiple definition of..." build failures

Recipients:(address . 50243@debbugs.gnu.org)

Message-ID:87k0h0w19b.fsf@elephly.net

linsmith is now fixed with commit 3d32fd1a6e.

Ricardo

Ricardo Wurmus wrote on 22 Nov 2021 20:32

Recipients:(address . 50243@debbugs.gnu.org)

Message-ID:87fsrovvas.fsf@elephly.net

More fixes:

3d109b0306 gnu: uim-qt: Pass -fcommon to compiler.

108b923d92 gnu: uim-gtk: Pass -fcommon to compiler.

1882282843 gnu: megaglest: Pass -fcommon to compiler.

(Note that uim-qt still fails to install, because it attempts to

install its library into the prefix directory of qtbase:

Toggle snippet (13 lines)

make[3]: Entering directory

'/tmp/guix-build-uim-qt-1.8.8.drv-0/uim-1.8.8/qt5/immodule'

/gnu/store/fildpz4qc0llifg8si9s7ag0vwlq1r2z-qtbase-5.15.2/bin/qmake

-install qinstall -exe libuimplatforminputcontextplugin.so

/gnu/store/fildpz4qc0llifg8si9s7ag0vwlq1r2z-qtbase-5.15.2/lib/qt5/plugins/platforminputcontexts/libuimplatforminputcontextplugin.so

Error copying libuimplatforminputcontextplugin.so to

/gnu/store/fildpz4qc0llifg8si9s7ag0vwlq1r2z-qtbase-5.15.2/lib/qt5/plugins/platforminputcontexts/libuimplatforminputcontextplugin.so:

Cannot create

/gnu/store/fildpz4qc0llifg8si9s7ag0vwlq1r2z-qtbase-5.15.2/lib/qt5/plugins/platforminputcontexts/libuimplatforminputcontextplugin.so

for output

Ricardo

Ricardo Wurmus wrote on 22 Nov 2021 23:17

Recipients:(address . 50243@debbugs.gnu.org)

Message-ID:87pmqrvnn0.fsf@elephly.net

More fixes:

a24b83d7ff gnu: uget: Add -fcommon to CFLAGS.

eceecd8ab4 gnu: transmission-remote-gtk: Update to 1.4.2.

92deae62c1 gnu: jumpnbump: Remove trailing #T.

02296cb0c8 gnu: jumpnbump: Add -fcommon to CFLAGS.

3e678de04c gnu: mupen64plus-core: Add -fcommon to CFLAGS.

This leaves pcsxr and ocaml4.07-lablgtk. I wasn’t able to fix

pcsxr and there’s no more recent tagged version to test. I don’t

really want to touch ocaml4.07-lablgtk, because it’s probably

going to fail in ways I don’t understand.

Any takers for the remaining two packages?

Ricardo

is:open	open issues
is:done	closed issues
submitter:<who>	search issue submitter
author:<who>	search by message author
date:yesterday..now	search by issue date
mdate:3m..2d	search by message date