Skip to content

[Linux] Make sure we install libexec. #67483

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
Jul 24, 2023

Conversation

al45tair
Copy link
Contributor

We weren't installing the libexec component for Linux.

rdar://112662487

We weren't installing the libexec component for Linux.

rdar://112662487
@al45tair
Copy link
Contributor Author

@swift-ci Please smoke test

@al45tair al45tair requested review from etcwilde and mikeash July 24, 2023 12:05
@al45tair
Copy link
Contributor Author

@swift-ci Please smoke test Windows platform

@al45tair al45tair merged commit 93e209c into swiftlang:main Jul 24, 2023
@al45tair
Copy link
Contributor Author

I'm not confident that this has actually fixed the problem; it should have, but testing locally I still don't see the installed content. Going to keep looking at this.

@al45tair
Copy link
Contributor Author

Looks like there's a typo in stdlib/cmake/modules/AddSwiftStdlib.cmake. Testing now to make sure that's the final piece of the puzzle. Will raise a separate PR to fix that as well.

@al45tair
Copy link
Contributor Author

al45tair commented Jul 25, 2023

That's fixed in #67505

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.

4 participants