-
Notifications
You must be signed in to change notification settings - Fork 229
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
Repo not structured properly for Swift Package #1153
Comments
@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 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? |
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 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 |
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.
The text was updated successfully, but these errors were encountered: