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

Ensure build directory contains a copy of rust-toolchain.toml #616

Merged
merged 1 commit into from
Aug 23, 2024

Conversation

gmartin82
Copy link
Contributor

@gmartin82 gmartin82 commented Aug 22, 2024

When building using cargo's --manifest-path option the toolchain configuration file within the manifest directory is ignored. Ensuring a copy of the rust-toolchain.toml file is in the build directory ensures the toolchain settings are picked up when building via cmake.

When building using cargo's --manifest-path option the toolchain configuration file within the manifest directory is ignored. Ensuring a copy of the rust-toolchain.toml file is in the build directory ensures the toolchain settings are pciked up when building via cmake.
@gmartin82
Copy link
Contributor Author

To clarify the toolchain configuration file is only ignored if the directory containing the manifest referenced by --manifest-path is not the same as the build directory.

@Mallets
Copy link
Member

Mallets commented Aug 23, 2024

@yellowhatter can you test this PR?

@yellowhatter
Copy link
Contributor

yellowhatter commented Aug 23, 2024

@Mallets well, this PR seems to be working and rust-toolchain.toml is copied and respected by Cargo further.

@Mallets
Copy link
Member

Mallets commented Aug 23, 2024

So, is it an OK to merge it?

Copy link
Contributor

@yellowhatter yellowhatter left a comment

Choose a reason for hiding this comment

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

Yes, sure

@Mallets Mallets merged commit c996fd3 into eclipse-zenoh:dev/1.0.0 Aug 23, 2024
7 checks passed
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