search

bazel-xcode / PodToBUILD

An easy way to integrate CocoaPods into Bazel
Apache License 2.0
323 stars 69 forks source link

Rework module maps #149

Closed jerrymarino closed 4 years ago

jerrymarino commented 4 years ago

This PR fixes issues with module maps. First, it implements public headers per the podspec. Module maps are normalized into a single instance per spec. This is achieved through the objc_library.module_map attribute, there is virtually no way around using this with swift.

I've added a couple snapshot tests of the output and we'll have more build tests for this with swift soon.

jerrymarino commented 4 years ago

@jparise It looks like Set.filter and Set.sorted return Arrays, so I just removed the redundant calls to Array