-
Notifications
You must be signed in to change notification settings - Fork 10.5k
[shims] Remove some obsolete shims modules #37600
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
@swift-ci test |
Build failed |
Build failed |
Oh yeah, this depends on #37587. |
@swift-ci test |
Build failed |
Well well well, what do we have here. |
Let's pick this up again. The bogus XCTest overlay was removed (#37587) by the time the failing build above started. It is possible we picked up an older version of it from the SDK somehow, in which case we'll probably need to keep the XCTest shims in the toolchain for a year or two yet. But first, let's try again with the current state of this branch. |
@swift-ci test |
Ah, the console logs were in fact pretty clear about the issue -- the version of the real XCTest overlay in the Xcode we currently use in CI (Xcode 12.3 (12C33)) apparently still imports the old toolchain shims:
This is unexpected, as I thought these imports were removed as far back as Xcode 11.4 (or so). 🤔 |
Oh, a stray import remained until Xcode 12.5. That's okay -- we just need to keep the XCTest shims around a bit longer. |
Build failed |
The corresponding overlays are maintained outside of this repository, and they haven’t been importing these shims for years. SDKs that shipped between Xcode 10.2 and Xcode 11 (or so) contained overlay modules that imported these shims modules, so future toolchain builds won’t work correctly with those SDKs. (This is fine! We do not expect toolchain builds to work with years old Xcode installations anyway.)
@swift-ci test |
The corresponding overlays are maintained outside of this repository, and they haven’t been importing these shims for years.
SDKs that shipped between Xcode 10.2 and Xcode 11 (or so) contained overlay modules that imported these shims modules, so future toolchain builds won’t work correctly with those SDKs. (This is fine! We do not expect toolchain builds to work with years old Xcode installations anyway.)
The shims that remain are used either by stdlib components, or by one of the overlays that still exist in the repository (ObjectiveC, Darwin, CoreGraphics, CoreFoundation, Foundation).