Repo not structured properly for Swift Package
PatrickDanino opened this issue · 16 comments
Xcode is generating dozens of warnings about the Branch SDK when included via the SPM. These warnings cannot be turned off by default and are generating noise that hides other legitimate warnings.
Example of a warning when archiving:
Warning: Umbrella header for module 'Branch' does not include header 'BNCTuneUtility.h'`
SPM requires that the headers be split between two folders, one for public headers and the other for private headers. You can then specify the private folder as a headerSearchPath and add it to the exclude list in Package.swift. By default, public headers put in an include folder will be automatically picked up.
@PatrickDanino I see they have a separate repo for SPM. Perhaps that one has fewer warnings (haven't tried it yet). https://github.com/BranchMetrics/ios-branch-sdk-spm
Per the description, that repo is a mirror of this one (with some history removed). As such, both have the same issues described above.
We have the same problem. Those warnings become errors when GCC_TREAT_WARNINGS_AS_ERRORS is enabled. We had to disable it, at least now.
We are using the SPM specific repo (https://github.com/BranchMetrics/ios-branch-sdk-spm) with version 1.40.2
We have this earmarked for end of quarter.
Hi @echo-branch is there an update on this issue?
So it was in progress, but got shelved for other work. I'm picking it back up this next sprint.
+1
@echo-branch Hi, any updates?
It slipped yet again. This time due to dev illness and vacations. It is still the top item though.
@echo-branch Hi, any chance for updates?
Thanks
In progress. The layout refactor has cascading impact on our build, testing and documentation. So it's taking some time to get everything lined up.
In progress. The layout refactor has cascading impact on our build, testing and documentation. So it's taking some time to get everything lined up.
Thanks for the update
@echo-branch any update here? This was fine for Xcode 13.4 but we';re getting tons of warnings in Xcode 14
@jhurray
I suspect you had the type of warning ignored before, cause it's been an issue for almost a year now.
We're still chipping away at the problem internally, the fix will required renaming our SDK which means updating a lot of docs, build and testing. We also have dependencies such as RN, Unity etc to manage along with this. Our initial 1 or 2 sprint estimate was way off. I don't have a firm date, but it'll be a major release.
This was shipped with 2.0.0