Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

buck2: init at unstable-2023-07-11 #243148

Merged
merged 1 commit into from
Jul 13, 2023

Conversation

thoughtpolice
Copy link
Member

@thoughtpolice thoughtpolice commented Jul 12, 2023

The Buck2 team has (rather unexpectedly!) done its first binary prerelease, with an actual version tag; this binary is (still) built with the unstable rust compiler, which blocked #232471 from being merged. But the binary should run fine with a little patchelf magic, and, is at least built and deployed directly from GitHub Actions.

This adds binaries for all 4 major Unix platforms and just makes the patchelf hook succeeds. It adds myself as the maintainer, as I already am doing a bunch of work on/with buck2. New updates will happen every 2 weeks; the included update.sh script can helpfully dump out the hashes for you.

This should help alleviate #226677; though a source build is of course ideal. That's going to probably be a very far ways off, though. It might explode. Handle with care.

I've only tested this on x86_64-linux, so we're going to hand off to borg to try it elsewhere.

Things done
  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin

@thoughtpolice thoughtpolice force-pushed the nixpkgs/buck2-binary branch 2 times, most recently from d5ec838 to 27deff8 Compare July 12, 2023 21:10
@thoughtpolice
Copy link
Member Author

Note that we could drop the dynamic glibc binary in the aarch64-linux case because there's a -musl variant; but apparently the x86_64-linux static musl build is broken. Once that's fixed we can migrate both of them and drop the patchelfHook.

@thoughtpolice thoughtpolice force-pushed the nixpkgs/buck2-binary branch 5 times, most recently from 0370c60 to 7b6aa3c Compare July 12, 2023 22:54
@thoughtpolice
Copy link
Member Author

This has also been updated with an update.sh script that should make it compatible with Ryan's bot, normal maintainer flows, etc.

@thoughtpolice thoughtpolice force-pushed the nixpkgs/buck2-binary branch from 7b6aa3c to 457d5d0 Compare July 13, 2023 00:10
Signed-off-by: Austin Seipp <aseipp@pobox.com>
@thoughtpolice thoughtpolice force-pushed the nixpkgs/buck2-binary branch from 457d5d0 to 995167e Compare July 13, 2023 00:11
@thoughtpolice thoughtpolice merged commit e481f12 into NixOS:master Jul 13, 2023
@thoughtpolice thoughtpolice deleted the nixpkgs/buck2-binary branch July 13, 2023 03:23
nickgerace added a commit to systeminit/si that referenced this pull request Jul 13, 2023
Enabled by the following PR:
NixOS/nixpkgs#243148

Signed-off-by: Nick Gerace <nick@systeminit.com>
si-bors-ng bot added a commit to systeminit/si that referenced this pull request Jul 13, 2023
2393: Use upstream buck2 from nixpkgs (ENG-1696) r=nickgerace a=nickgerace

Enabled by the following PR:
NixOS/nixpkgs#243148

<img src="https://media3.giphy.com/media/xiqeqfNuUMwb1ma26f/giphy.gif"/>

Co-authored-by: Nick Gerace <nick@systeminit.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant