-
Notifications
You must be signed in to change notification settings - Fork 1.4k
[Build] Generate ObjC compatibility header for library Swift targets #2341
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
afd2a96
to
7fe28d6
Compare
@swift-ci smoke test |
7fe28d6
to
599c23a
Compare
@swift-ci smoke test |
This will generate the ObjC header for library Swift targets so they can be used in other ObjC targets. This is only done for macOS. <rdar://problem/51780725>
599c23a
to
0a04609
Compare
@swift-ci smoke test |
@aciidb0mb3r: Is this released in Xcode 11.2? Because I'm still seeing the issue. I have a sample if it helps? |
No, this change is not present in Xcode 11.2 |
Is this going to be released with Xcode 11.3 or we have to wait for Swift 5.2? |
How about Xcode 11.3? |
Maybe with 11.4? |
Is there an update on when this is released / available in Xcode? |
@aciidb0mb3r Any reason not to do this for iOS, as well? |
I wish this could be supported in iOS as well. Currently, I am facing cases like delivering Swift packages to ObjC based App target. Without able to directly import the autogenerated swift header in ObjC, it will be very painful to add a Swift adaptor layer in the app. |
This is working in iOS project on Xcode 11.4.1 for me. I've tested it on an old Objective-C project and successfully can use Swift Package manager with it. Here is what I did:
|
I'm able to add a SwiftPackage into MainTarget Objective-C.m file |
I'm unable to use @objc protocols (declared in Swift files in a Swift Package) in Obj. C files in an Xcode project framework that explicitly links to that package. |
This will generate the ObjC header for library Swift targets so they can be
used in other ObjC targets. This is only done for macOS.
rdar://problem/51780725