cargo-quickinstall icon indicating copy to clipboard operation
cargo-quickinstall copied to clipboard

Broken build for jj-cli

Open senekor opened this issue 1 year ago • 2 comments

related issue in the jj repo

jj provides prebuilt binaries itself, but only statically linked against musl on linux. Which is fine, because that works on any linux system, whatever the system libc is. However, cargo binstall jj-cli fell back to the (borked) binary provided by quickinstall. I did not notice this at first, even though it does say so in the output of cargo-binstall.

I'm not sure what the right thing to do here is, but having cargo-binstall install from a different source than the official one, even though the official one does in fact provide binaries, is incredibly confusing.

senekor avatar Jun 08 '24 17:06 senekor

for reference, this seems to be the CI job that built jj

senekor avatar Jun 08 '24 17:06 senekor

We use glibc 2.17, probably that's the reason of the failure?

NobodyXu avatar Jun 09 '24 01:06 NobodyXu

We use glibc 2.17, probably that's the reason of the failure?

As per linked discussion comment, yes glibc 2.33 would avoid the issue, or updating CI from Zig 0.10.0 to a version from Zig 0.12.0 (you have many CI runs apparently failing when I was looking over workflow history, not sure if this subtle bug affects other bins you distribute).

polarathene avatar Sep 10 '24 05:09 polarathene

Previously we have some build failure with newer zig-cc (I think it might be related to cargo-auditable), I think it's time to try it again

NobodyXu avatar Sep 12 '24 14:09 NobodyXu

If you have more context on that failure it would be good to know. Perhaps it was also fixed from Zig 0.12.0? I like to use zig for builds but I am a little wary at times due to these non-obvious caveats/surprises occurring 😓

polarathene avatar Sep 12 '24 21:09 polarathene

If you have more context on that failure it would be good to know.

rust-cross/cargo-zigbuild#162

IIRC the upstream issue in zig-cc is still open

Anyway we don't use cargo-audiable anymore (because it can be overwritten by the crate), I have bumped zig-cc to latest version

NobodyXu avatar Sep 13 '24 09:09 NobodyXu