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

Legacy CMake: Respect CMAKE_<XYZ>_OUTPUT_DIRECTORY #522

Conversation

jschwe
Copy link
Collaborator

@jschwe jschwe commented May 14, 2024

Make a best effort with old CMake versions to respect output directories. We can't know which properties the user might set or change after the crates are imported, but we can respect the CMAKE_<XYZ> variables which are set before the crates are imported.

@jschwe jschwe force-pushed the jschwender/legacy_partially_respect_output_dir branch 6 times, most recently from a3407b7 to 77a03a4 Compare May 14, 2024 18:09
Make a best effort with old CMake version to respect
output directories. We can't know which properties
the user might set or change **after** the crates are
imported, but we can respect the `CMAKE_<XYZ>` variables
which are set **before** the crates are imported.
Create the main CMake targets earlier, so we can
set target properties on them early in the generator,
so we can use them later on during configuration.
@jschwe jschwe force-pushed the jschwender/legacy_partially_respect_output_dir branch from 1339945 to 743f8d3 Compare May 14, 2024 18:30
@jschwe jschwe marked this pull request as ready for review May 14, 2024 19:14
@jschwe jschwe merged commit 221de5e into corrosion-rs:stable/v0.5 May 15, 2024
38 checks passed
@jschwe jschwe deleted the jschwender/legacy_partially_respect_output_dir branch May 15, 2024 06:33
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.

1 participant