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

rav1e: fix cross #225358

Closed
wants to merge 677 commits into from
Closed

rav1e: fix cross #225358

wants to merge 677 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Apr 9, 2023

Description of changes

cargo cbuild needs to have the C{C,XX}_FOR_{${platform}} variables set just like cargo since it is basically a wrapper around cargo. Without these variables, it will try to use the hostPlatform C compiler to compile build.rs scripts, and will pass flags to that compiler which only make sense on the buildPlatform. So it's just doomed without the environment variables.

Right now it looks like rav1e is the only package we have that is using cargo-c, but if that changes in the future we should factor this out as its own hook, like maturinBuildHook and the others.

Closes #225356

Things done
  • Built on platform(s)
    • aarch64-linux (cross from x86_64-linux)
  • Fits CONTRIBUTING.md.

jtojnar and others added 30 commits April 6, 2023 01:35
GTK 4 API version is bumped to 6.0.
i.e. we now only have webkitgtk-6.0, and no more webkit2gtk-5.0.
https://gitlab.gnome.org/GNOME/gtk/-/compare/3.24.36...3.24.37

- Removes legacy Wayland text input module.

Changelog-Reviewed-By: Jan Tojnar <[email protected]>
@ghost ghost requested a review from Ericson2314 as a code owner April 9, 2023 00:53
@ofborg ofborg bot added the 6.topic: cross-compilation Building packages on a different platform than they will be used on label Apr 9, 2023
@github-actions github-actions bot added 6.topic: emacs Text editor 6.topic: GNOME GNOME desktop environment and its underlying platform 6.topic: golang 6.topic: lua 6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 6.topic: pantheon The Pantheon desktop environment 6.topic: python 6.topic: qt/kde 6.topic: ruby 6.topic: rust 6.topic: stdenv Standard environment 6.topic: systemd 6.topic: TeX Issues regarding texlive and TeX in general 6.topic: vim 8.has: changelog 8.has: documentation This PR adds or changes documentation 8.has: module (update) This PR changes an existing module in `nixos/` labels Apr 9, 2023
@ghost ghost closed this Apr 9, 2023
@ghost ghost locked as spam and limited conversation to collaborators Apr 9, 2023
@ghost ghost deleted the pr/fixcross/rav1e branch April 9, 2023 00:57
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
6.topic: cross-compilation Building packages on a different platform than they will be used on 6.topic: emacs Text editor 6.topic: GNOME GNOME desktop environment and its underlying platform 6.topic: golang 6.topic: lua 6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 6.topic: pantheon The Pantheon desktop environment 6.topic: python 6.topic: qt/kde 6.topic: ruby 6.topic: rust 6.topic: stdenv Standard environment 6.topic: systemd 6.topic: TeX Issues regarding texlive and TeX in general 6.topic: vim 8.has: changelog 8.has: clean-up 8.has: documentation This PR adds or changes documentation 8.has: module (update) This PR changes an existing module in `nixos/` 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
Status: Done
Development

Successfully merging this pull request may close these issues.

pkgsCross.*.rav1e is broken