-
Notifications
You must be signed in to change notification settings - Fork 10.5k
Add new _foundation_unicode and _FoundationCShims c modules to windows build script #74796
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
Add new _foundation_unicode and _FoundationCShims c modules to windows build script #74796
Conversation
swiftlang/swift-corelibs-foundation#4991 @swift-ci please build toolchain Windows platform |
This corrected the issue in the windows toolchain build (the build above made it much farther than before) |
@swift-ci please smoke test |
This also needs a change to the packaging manifest in https://github.com/apple/swift-installer-scripts. |
Yep I'll post a change to that repo that also includes updates for all of the new libraries being added |
-1073741515 = 0xc0000135, which I believe is DLL not found. I think that there are some path tweaks needed. |
Thanks! That's super helpful, there's likely still an issue to resolve here but this PR at least gets us a lot closer. Previously the toolchain build failed while building TSC since it couldn't find these modules but now it makes it all the way to the swift-inspect build before failing. I'll investigate what might be missing, perhaps we're not copying one of the new Foundation dll's over properly |
@compnerd it looks like this change currently fails with the existing SCL-F because these |
Hmm, you could check if the directory exists via the |
@swift-ci please smoke test |
@swift-ci please smoke test |
…s build script (swiftlang#74796) * Add new _foundation_unicode and _FoundationCShims c modules to windows build script * Only copy directories that exist * Address feedback
Now that Foundation is producing new C modules (
_foundation_unicode
and_FoundationCShims
) that clients depend upon because we do not enable library evolution for our modules, we need to ensure these modules are correctly installed into the just built SDK like the existing dispatch C modules.