-
Notifications
You must be signed in to change notification settings - Fork 197
build: adjust the autolink library for Windows #1312
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
Conversation
When building statically, ensure that we prefix the Swift libraries with the correct prefix to permit static linking of the libraries.
@swift-ci please test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems fine to me, but before merging could we test the CMake build via a cross-repo test with the swift repo (sadly we don't have the ability to test CMake in this repo) just to make sure we don't miss anything there?
Running the build, but unlikely to catch anything as we are not doing enough of the static library builds. |
Looks like maybe this did break the Windows CMake build? |
Yeah, because we weren't building Synchronization. I've added that and am going to re-try. |
This reverts commit 5d54b93.
When building statically, ensure that we prefix the Swift libraries with the correct prefix to permit static linking of the libraries.