zig: fail to link SDL2

  • Open
  • quality assurance status badge
Details
2 participants
  • Nguy?n Gia Phong
  • Ekaitz Zarraga
Owner
unassigned
Submitted by
Nguy?n Gia Phong
Severity
normal
N
N
Nguy?n Gia Phong wrote on 23 Oct 2023 09:34
CWFN02XCWX0T.2T2PRVXKN89Y@guix
I am trying to package Mepo, an OpenStreetMap frontend written in Zig.
The currently packaged Zig fails to find transitive dependencies of SDL2:

Here's a minimal reproducible Zig project:

// main.zig
const c = @cImport(@cInclude("SDL2/SDL.h"));
pub fn main() !void {
if (c.SDL_Init(c.SDL_INIT_VIDEO) != 0)
return error.SDLInitializationFailed;
c.SDL_Quit();
}

// build.zig
pub fn build(b: *@import("std").build.Builder) void {
const exe = b.addExecutable("sdl2-smoke-test", "main.zig");
exe.linkSystemLibrary("SDL2");
exe.setTarget(b.standardTargetOptions(.{}));
exe.setBuildMode(b.standardReleaseOptions());
exe.install();
}

For convenience, here goes a manifest for use with guix build -f:

(use-modules (guix build-system zig)
(guix gexp)
(guix licenses)
(guix packages)
(gnu packages pkg-config)
(gnu packages sdl))
(package
(name "sdl2-smoke-test")
(version "0.0.0")
(source (local-file "." "sdl2-smoke-test" #:recursive? #t))
(build-system zig-build-system)
(arguments (list #:tests? #f
#:zig-release-type "safe"))
(native-inputs (list pkg-config))
(inputs (list sdl2))
(synopsis "Zig build system SDL2 linkage failure reproducer")
(description
(string-append "This is a reproducer of zig-build-system"
" failing to link SDL2's transitive dependencies,"
" as seen when trying to package mepo."))
(license zlib))
-----BEGIN PGP SIGNATURE-----

iIQEABYIACwWIQSDiv4NVdwHTjYPlDqEtpzm8/a3ZwUCZTYiAQ4cY254QGxvYW5n
Lm5ldAAKCRCEtpzm8/a3Z+agAP9boZoEOJcAnHgH6DVVEMXnqIGmL1n2lWAMRjP0
OsQ+TwD/R0ymTEfk1LkadSDLEOepRuVTKCuwT8EcvmHwmHFKTgI=
=jf6+
-----END PGP SIGNATURE-----


E
E
Ekaitz Zarraga wrote on 1 Nov 2023 21:52
(name . Nguy?n Gia Phong)(address . cnx@loang.net)(address . bug-guix@gnu.org)
MFUQhf68MW3SdqqcyO6PFuQ-Ih3I6BkV8N0wDS-xf52TunStisoYtvdxG6KUSOBBkLn-E_4tEJRlD8jvTSnbFUhR88uYuSk8b4l2GmcbFmk=@elenq.tech
------- Original Message -------
On Monday, October 23rd, 2023 at 09:34, Nguy?n Gia Phong <cnx@loang.net> wrote:


Toggle quote (49 lines)
> I am trying to package Mepo, an OpenStreetMap frontend written in Zig.
> The currently packaged Zig fails to find transitive dependencies of SDL2:
> https://paste.sr.ht/~cnx/db21e1ef9a6828f3ec898db43b790d71b5ef213a
>
> Here's a minimal reproducible Zig project:
>
> // main.zig
> const c = @cImport(@cInclude("SDL2/SDL.h"));
> pub fn main() !void {
> if (c.SDL_Init(c.SDL_INIT_VIDEO) != 0)
> return error.SDLInitializationFailed;
> c.SDL_Quit();
> }
>
> // build.zig
> pub fn build(b: *@import("std").build.Builder) void {
> const exe = b.addExecutable("sdl2-smoke-test", "main.zig");
> exe.linkSystemLibrary("SDL2");
> exe.setTarget(b.standardTargetOptions(.{}));
> exe.setBuildMode(b.standardReleaseOptions());
> exe.install();
> }
>
> For convenience, here goes a manifest for use with guix build -f:
>
> (use-modules (guix build-system zig)
> (guix gexp)
> (guix licenses)
> (guix packages)
> (gnu packages pkg-config)
> (gnu packages sdl))
> (package
> (name "sdl2-smoke-test")
> (version "0.0.0")
> (source (local-file "." "sdl2-smoke-test" #:recursive? #t))
> (build-system zig-build-system)
> (arguments (list #:tests? #f
> #:zig-release-type "safe"))
> (native-inputs (list pkg-config))
> (inputs (list sdl2))
> (synopsis "Zig build system SDL2 linkage failure reproducer")
> (description
> (string-append "This is a reproducer of zig-build-system"
> " failing to link SDL2's transitive dependencies,"
> " as seen when trying to package mepo."))
> (home-page "https://larkspur.one/notice/Ab24w1IMGVoZ72J9xg")
> (license zlib))


Hi,

I have good and bad news for you.

I tried to replicate the problem, and I certainly did. I also managed to mitigate the problem adding `exe.linkSystemLibrary()` lines to the build.zig with the transitive dependencies.
And later I wondered how did this all work in GCC, just in case.

(NOTE: take in account you also need to link with libc! for that you can add it as a linkSystemLibrary("c"))

The problem is not Zig specific, look:

I tried a random example from a repo in Github using SDL2 and SDL2-Image:

Made a shell for it:

guix shell sdl2 sdl2-image gcc-toolchain glibc:static

And tried to build:

$ gcc a.c -static -lSDL2 -lSDL2_image -lm -lc 2>&1 | head
ld: /gnu/store/0mhh6ycg8dq6n43qwpvpvr2smbxlixpi-profile/lib/libSDL2.a(SDL_dynapi.o):(.text+0x27d8): warning: Using 'dlopen' in statically linked applications requires at runtime the shared libraries from the glibc version used for linking
ld: /gnu/store/0mhh6ycg8dq6n43qwpvpvr2smbxlixpi-profile/lib/libSDL2.a(SDL_waylandvideo.o):(.text+0x48f): undefined reference to `wl_proxy_get_version'
ld: /gnu/store/0mhh6ycg8dq6n43qwpvpvr2smbxlixpi-profile/lib/libSDL2.a(SDL_waylandvideo.o):(.text+0xd3e): undefined reference to `wl_proxy_get_version'
ld: /gnu/store/0mhh6ycg8dq6n43qwpvpvr2smbxlixpi-profile/lib/libSDL2.a(SDL_waylandvideo.o):(.text+0xd5a): undefined reference to `wl_proxy_marshal_flags'
ld: /gnu/store/0mhh6ycg8dq6n43qwpvpvr2smbxlixpi-profile/lib/libSDL2.a(SDL_waylandvideo.o):(.text+0xd73): undefined reference to `wl_proxy_add_listener'
ld: /gnu/store/0mhh6ycg8dq6n43qwpvpvr2smbxlixpi-profile/lib/libSDL2.a(SDL_waylandvideo.o):(.text+0xeb9): undefined reference to `wl_proxy_get_version'
ld: /gnu/store/0mhh6ycg8dq6n43qwpvpvr2smbxlixpi-profile/lib/libSDL2.a(SDL_waylandvideo.o):(.text+0xfa8): undefined reference to `wl_proxy_marshal_flags'
ld: /gnu/store/0mhh6ycg8dq6n43qwpvpvr2smbxlixpi-profile/lib/libSDL2.a(SDL_waylandvideo.o):(.text+0x1002): undefined reference to `wl_proxy_marshal_flags'
ld: /gnu/store/0mhh6ycg8dq6n43qwpvpvr2smbxlixpi-profile/lib/libSDL2.a(SDL_waylandvideo.o):(.text+0x1071): undefined reference to `wl_proxy_add_listener'
ld: /gnu/store/0mhh6ycg8dq6n43qwpvpvr2smbxlixpi-profile/lib/libSDL2.a(SDL_waylandvideo.o):(.text+0x107e): undefined reference to `wl_proxy_set_tag'

The list of errors are basically the same you have.
Zig builds statically if I'm not mistaken so it's triggering the same problem GCC would for that case.

We need to research this further...

I also checked Mepo, how do they manage to build directly?

I don't have the Zig knowledge to know how does all this work, but I know it's not just the build-system's fault because it happens in GCC too, and also with a normal shell with `zig` available, and running the command by hand. The build-system just replicates that behavior.

We need to dig on this further. Are you, Nguy?n, a Zig programmer? Maybe you can help me figure out this?

Thanks for the report, the patience and the help!

All the best,
Ekaitz
E
E
Ekaitz Zarraga wrote on 10 Nov 2023 10:57
(name . Nguy?n Gia Phong)(address . cnx@loang.net)(address . 66699@debbugs.gnu.org)
WpV4q86pbeIZROkO_ywSnbkSq4t2_A4QIyV0iLfIC2B0BsM738sOk3HC6BJspkrHljtZjU-oe8jsB_Qq4pKdNGNzn8gcdyoh3n4iS1Y6X_U=@elenq.tech
Hi,


On Friday, November 10th, 2023 at 06:01, Nguy?n Gia Phong <cnx@loang.net> wrote:

Toggle quote (29 lines)
> On 2023-11-01 at 20:52+00:00, Ekaitz Zarraga wrote:
>
> > On Monday, October 23rd, 2023 at 09:34, Nguy?n Gia Phong wrote:
> >
> > > I am trying to package Mepo, an OpenStreetMap frontend written in Zig.
> > > The currently packaged Zig fails to find transitive dependencies of SDL2:
> > > https://paste.sr.ht/~cnx/db21e1ef9a6828f3ec898db43b790d71b5ef213a
> >
> > The problem is not Zig specific, look: [...]
> >
> > guix shell sdl2 sdl2-image gcc-toolchain glibc:static
> >
> > And tried to build: [...] The list of errors are basically the same
> > you have. Zig builds statically if I'm not mistaken
> > so it's triggering the same problem GCC would for that case.
> >
> > We need to research this further...
> >
> > I don't have the Zig knowledge to know how does all this work,
> > but I know it's not just the build-system's fault
> > because it happens in GCC too, and also with a normal shell
> > with `zig` available, and running the command by hand.
> > The build-system just replicates that behavior.
>
>
> Thanks, this clears it up for me. So far Zig does not offer a way
> to prefer dynamic or static and just search alphabetically:
> https://github.com/ziglang/zig/issues/14111

Maybe there are more flags for this, but I'm not sure.
Toggle quote (16 lines)
> On 2023-05-23 at 17:11Z, Michael Dusan wrote:
>
> > A workaround is if you are only using the zig cc driver,
> > the syntax -l:<FULLNAME> will find the first fully-named library file
> > that matches against search dirs. For example:
> >
> > * -lz will search dirs, prefer lib<BASENAME>.a over lib<BASENAME>.so
> > * -l:libz.so will search directly for libz.so
> > and not care about any other name permutations
>
>
> I do wonder though, if it's a generally good idea have separate
> shared and static outputs for e.g. SDL2. Obviously Zig programs
> would benefit from this, and specifying linkage preference
> under GCC is via the same -l:lib*.* flag.

I'm not very versed on this so I can't give you a proper answer.

Toggle quote (7 lines)
> On 2023-11-01 at 20:52+00:00, Ekaitz Zarraga wrote:
>
> > I also checked Mepo, how do they manage to build directly?
>
> Sorry, could you rephrase this question?
> I don't get what you meant by directly here.

Yes, it was pretty badly phrased.
The question here was about how do the Mepo devs build mepo in other
linux distributions... They should have a similar problem to what we
have. Or other systems like Nix, how do they build?
In Nix the problem should appear too.

With *directly* I meant the Mepo repository doesn't contain the
transitive dependencies so... It would never work for us, but it
should be working for them.

Reading your message before seems like they manage to link against
the SDL .so so nothing of this happens...

I'll try to check all this with the Zig people to see if we can
fix.

Thanks!
E
E
Ekaitz Zarraga wrote on 11 Nov 2023 14:23
(name . Nguy?n Gia Phong)(address . cnx@loang.net)(address . 66699@debbugs.gnu.org)
VogPLQH6oApxEQbHyfSSxYEip9tSK6qWCBObn_SEM4sLgh_BTC8l3MUACnrynd8-zc4QBmZl-31mLSXtfr_goHS9eG5bi7nUtEzCrO4bXf4=@elenq.tech
Hi,

I tracked this pretty far:

This is a related issue that mentions the `mepo` package:

And I realized the problem comes from `pkg-config`. It's telling zig to use the static library first.
Removing `pkg-config` from the `native-inputs` in your proposed package make it build but the `runpath` validation phase fails.

I'm not sure about what to do now.

In the issue the people working on freebsd-ports mention they did this:
But I don't know if we can replicate that.

I'll keep researching on this a little bit further.

Best,
Ekaitz
?
Your comment

Commenting via the web interface is currently disabled.

To comment on this conversation send an email to 66699@debbugs.gnu.org

To respond to this issue using the mumi CLI, first switch to it
mumi current 66699
Then, you may apply the latest patchset in this issue (with sign off)
mumi am -- -s
Or, compose a reply to this issue
mumi compose
Or, send patches to this issue
mumi send-email *.patch