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
Fix missing -ObjC for xcframeworks - take 2 #10460
Conversation
# that come shipped with the Pod. | ||
# | ||
def vendored_static_xcframeworks | ||
vendored_xcframeworks.select do |path| |
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.
do these need to be compact
?
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.
I guess not it seems it only selects the ones that are static frameworks?
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.
I think since select
just needs a boolean we don't need to return the path
select do |path|
Xcode::XCFramework.new(path).build_type == BuildType.static_framework
end
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.
Confirmed. Thanks @amorde !
@paulb777 whats left here? is it just landing integration specs, merging this and publishing 1.10.2? |
@dnkoutso Yes. It should be ready to go. |
OK merged integration specs! |
Integration tests updated in CocoaPods/cocoapods-integration-specs#313 |
Fix #10459 - a better file_accessor solution that undoes the build_settings.rb changes from #10234
Integration test companion is CocoaPods/cocoapods-integration-specs#312