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

staging-next 2023-05-25 #233944

Merged
merged 239 commits into from
May 31, 2023
Merged

staging-next 2023-05-25 #233944

merged 239 commits into from
May 31, 2023

Conversation

vcunat
Copy link
Member

@vcunat vcunat commented May 25, 2023

thefloweringash and others added 30 commits March 5, 2022 15:48
This was causing some regions to escape the fixupHook. As a concrete
example, pkgconf's signature on aarch64-darwin was not valid because
the library was not fixed up.
The NIX_LIB64|32_IN_SELF_RPATH environment variables control whether
to add lib64 and lib32 to rpaths. However, they're set depending
on the build paltform, not the target platform and thus their values
are incorrect for for cross-builds.

On the other hand, setting them according to the build platform introduce
pointless differences in build outputs; see #221350 for details.

This change fixes the issues by boldly removes the NIX_LIB*_IN_SELF_RPATH
facility altogether, in the hope that it is no longer necessary. They
were introduced in 2009, long before nixpkgs had good support for
cross-builds.

Fixes #221350
At present, there are two blockers from using webrtc with
the gstreamer included by Nix. The first is that on Darwin,
the explicit WebRTC functionality is disabled, because at
time of adding, libnice did not work on Darwin. libnice does
now work on darwin, so we can re-enable it. The second is that
webrtc also needs sctp, which is provided by usrsctp. This
was disabled at authorship time because there was no nix
derivation. There now is, so it can be safely enabled.
The main change is drop of the upstream patch that fixes parallel build
failures.

Changes: https://lists.nongnu.org/archive/html/m17n-list/2023-05/msg00002.html
@vcunat vcunat mentioned this pull request May 28, 2023
12 tasks
@github-actions github-actions bot added the 6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS label May 28, 2023
@ofborg ofborg bot added the 2.status: merge conflict This PR has merge conflicts with the target branch label May 28, 2023
@ofborg ofborg bot removed the 2.status: merge conflict This PR has merge conflicts with the target branch label May 29, 2023
@vcunat vcunat merged commit 329824b into master May 31, 2023
@vcunat
Copy link
Member Author

vcunat commented May 31, 2023

For reference, a channel-blocking regression got unnoticed until merge:
https://hydra.nixos.org/build/222130292
(currently being bisected; chat)

@vcunat vcunat mentioned this pull request Jun 2, 2023
1 task
@arianvp arianvp mentioned this pull request Jun 2, 2023
12 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1.severity: security Issues which raise a security issue, or PRs that fix one 6.topic: kernel The Linux kernel 6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 6.topic: python 6.topic: stdenv Standard environment 6.topic: vim 8.has: package (new) This PR adds a new package 10.rebuild-darwin: 501+ 10.rebuild-darwin: 5001+ 10.rebuild-darwin-stdenv This PR causes stdenv to rebuild 10.rebuild-linux: 501+ 10.rebuild-linux: 5001+ 10.rebuild-linux-stdenv This PR causes stdenv to rebuild
Projects
None yet
Development

Successfully merging this pull request may close these issues.