Skip to content

Set --sysroot to sysroot generated by rust_toolchain #2223

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

Merged

Conversation

daivinhtran
Copy link
Contributor

Addresses #2039

@daivinhtran daivinhtran force-pushed the set-toolchain-generated-sysroot-to-rustc branch from 0fb5fef to 249b821 Compare October 27, 2023 21:18
@daivinhtran daivinhtran reopened this Oct 27, 2023
@daivinhtran daivinhtran force-pushed the set-toolchain-generated-sysroot-to-rustc branch from e6ae298 to d667de6 Compare October 27, 2023 21:22
@daivinhtran daivinhtran marked this pull request as ready for review October 27, 2023 21:22
Copy link
Collaborator

@UebelAndre UebelAndre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you also add an experimental flag for this?

@daivinhtran daivinhtran force-pushed the set-toolchain-generated-sysroot-to-rustc branch 6 times, most recently from 7f854c1 to 00078eb Compare October 30, 2023 19:06
@daivinhtran
Copy link
Contributor Author

Could you also add an experimental flag for this?

@UebelAndre Done.

@daivinhtran daivinhtran force-pushed the set-toolchain-generated-sysroot-to-rustc branch from ced7397 to d34afde Compare October 30, 2023 20:23
@daivinhtran daivinhtran force-pushed the set-toolchain-generated-sysroot-to-rustc branch from 2bc828b to b1a7e1a Compare October 31, 2023 14:51
@daivinhtran daivinhtran force-pushed the set-toolchain-generated-sysroot-to-rustc branch from d985ad8 to 6eb7a7c Compare October 31, 2023 15:34
Copy link
Collaborator

@UebelAndre UebelAndre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good! Just one last question 😄

@daivinhtran daivinhtran force-pushed the set-toolchain-generated-sysroot-to-rustc branch from d1bb016 to cddb545 Compare October 31, 2023 16:53
@daivinhtran daivinhtran force-pushed the set-toolchain-generated-sysroot-to-rustc branch from cddb545 to f8027f2 Compare October 31, 2023 16:56
Copy link
Collaborator

@UebelAndre UebelAndre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me! Thank you so much for all the hard work on this!

@UebelAndre UebelAndre merged commit 4aafa0a into bazelbuild:main Oct 31, 2023
@daivinhtran daivinhtran deleted the set-toolchain-generated-sysroot-to-rustc branch November 21, 2023 22:01
UebelAndre added a commit that referenced this pull request Nov 25, 2023
#2223 only sets
`--@rules_rust//rust/settings:experimental_toolchain_generated_sysroot`
in a few targets on CI. This PR defaults the feature flag to `True`.

This ensures the feature flag works at a larger scope because we remove
it.

---------

Co-authored-by: UebelAndre <[email protected]>
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.

2 participants