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

Fix SYSROOT detection on latest nightly #299

Merged
merged 1 commit into from
Jan 26, 2023
Merged

Fix SYSROOT detection on latest nightly #299

merged 1 commit into from
Jan 26, 2023

Conversation

bkchr
Copy link
Contributor

@bkchr bkchr commented Nov 30, 2022

Rust changed the way the SYSROOT is determined in: rust-lang/rust#103660

Before this change the SYSROOT was determined based on the rustc executable. Now it is determined based on the librustc_driver-*.so file. This pr fixes the detection by copying the library to the derivation as we have done it for the rustc executable.

Rust changed the way the SYSROOT is determined in: rust-lang/rust#103660

Before this change the SYSROOT was determined based on the rustc
executable. Now it is determined based on the librustc_driver-*.so file.
This pr fixes the detection by copying the library to the derivation as
we have done it for the rustc executable.
@bkchr
Copy link
Contributor Author

bkchr commented Nov 30, 2022

CC @nbp

@andersk
Copy link
Contributor

andersk commented Dec 2, 2022

Something different may be needed for Darwin?

For reference, corresponding changes from other Rust overlays:

@bkchr
Copy link
Contributor Author

bkchr commented Dec 2, 2022

Something different may be needed for Darwin?

Good question. I don't have any nix installation on Mac OS. Can you test this?

@nbp nbp merged commit f6fe850 into mozilla:master Jan 26, 2023
@nbp
Copy link
Collaborator

nbp commented Jan 26, 2023

Thanks for making these changes!
You are the best.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants