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

Draft: CI check #451

Closed
wants to merge 5 commits into from
Closed

Draft: CI check #451

wants to merge 5 commits into from

Conversation

jschwe
Copy link
Collaborator

@jschwe jschwe commented Nov 3, 2023

No description provided.

tronical and others added 5 commits November 3, 2023 11:36
…dows on windows hosts

- Defer the determination of whether to apply a .exe suffix for the byproduct copying
  and imported location setting if possible
- Determine the .exe suffix first based on the existence of the hostbuild property and then
  fall back Rust_CARGO_TARGET_OS
The variable `suffix` could by chance already be
set by user CMake code.
Always overwriting the parent scope value ensures
we don't rely on the variable being unset or empty
before the function is called.
@jschwe jschwe closed this Nov 3, 2023
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