Skip to content

[cmake] Only check HostCompatibilityLibs for bootstrapping builds #73383

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
merged 1 commit into from
May 3, 2024

Conversation

hamishknight
Copy link
Contributor

Locally I build debug builds without the standard library, using a copy of the stdlib in my release build. This hit a CMake error here since the HostCompatibilityLibs target isn't defined. Update to only access it when doing a bootstrapping build.

Locally I build debug builds without the standard library,
using a copy of the stdlib in my release build. This hit
a CMake error here since the `HostCompatibilityLibs` target
isn't defined. Update to only access it when doing a bootstrapping
build.
@hamishknight hamishknight requested a review from edymtt May 1, 2024 21:47
@hamishknight
Copy link
Contributor Author

@edymtt Not sure if this is the best fix, let me know if you have any better ideas

@hamishknight
Copy link
Contributor Author

@swift-ci please test

Copy link
Contributor

@edymtt edymtt left a comment

Choose a reason for hiding this comment

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

Sorry for disrupting your workflow, this looks reasonable to me -- in theory we could check if the HostCompatibilityLibs target exists, but this condition is better since it aligns with the code that references it.

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