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

nix-util / meson: Add -latomic on arm #11540

Merged
merged 1 commit into from
Sep 19, 2024
Merged

nix-util / meson: Add -latomic on arm #11540

merged 1 commit into from
Sep 19, 2024

Conversation

roberth
Copy link
Member

@roberth roberth commented Sep 18, 2024

Motivation

The meson build did not get this flag right yet, for the armv6/7 builds.

I couldn't get the test program to work correctly after many attempts, so let's just unblock this without making it perfect.

Context

https://hydra.nixos.org/build/272876163/nixlog/1

Priorities and Process

Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.

I couldn't get the test program to work correctly after many attempts,
so let's just unblock this without making it perfect.
@roberth roberth requested a review from edolstra as a code owner September 18, 2024 22:02
@roberth roberth added the build-problem Nix fails to compile or test; also improvements to build process label Sep 18, 2024
@NickCao
Copy link
Member

NickCao commented Sep 18, 2024

This also applies to riscv.

@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/2024-09-18-nix-team-meeting-minutes-179/52361/1

@roberth
Copy link
Member Author

roberth commented Sep 19, 2024

@NickCao are you sure? The cross build to riscv just works

@edolstra
Copy link
Member

Doesn't seem to be needed for armv7 (since all the Hydra builds succeed)? OTOH it probably doesn't hurt.

@NickCao
Copy link
Member

NickCao commented Sep 19, 2024

@NickCao are you sure? The cross build to riscv just works

Strange, I had to add -latomic to a bunch of package in nixpkgs.

@edolstra edolstra merged commit 9ea29ea into master Sep 19, 2024
20 checks passed
@edolstra edolstra deleted the meson-arm-atomic branch September 19, 2024 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build-problem Nix fails to compile or test; also improvements to build process
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants