-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Remove previous hack for SR-5206 #1259
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
Remove remaining references to SR-5206 hack by bringing the implementation inline with the Foundation overlay.
@swift-ci Please test |
@swift-ci please test |
Looks like these failures aren't related - @spevans it seems to be to do with the NSLock tests?
|
@alblue, yes I think there might be a bug in the lock test unrelated to this change. I think I can see the issue so I'll create a PR which should hopefully fix it. |
@swift-ci please smoke test |
1 similar comment
@swift-ci please smoke test |
@swift-ci Please clean test |
@swift-ci please test |
@itaiferber according to @shahmishal the foundation build doesn't understand smoke tests or clean tests ... so I've kicked it off again with the please test :) |
@alblue Ah, right — I never remember which project supports what. Thanks! |
@itaiferber that's ok, I only found out about half an hour ago too :) |
I just looked in the swift-4.0-branch but it looks like these changes are already in there. ¯\(ツ)/¯ Less work for me, I guess... |
What's in this pull request?
Remove remaining references to SR-5206 hack by bringing the implementation inline with the Foundation overlay.
Applies swiftlang/swift#11315 to swift-corelibs-foundation. This will go into the swift-4.0-branch as well.