Skip to content
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

Xcode 12.5 Beta 3: Umbrella header for module 'Segment' does not include header 'SEGReachability.h' #988

Closed
sebastian-foreflight opened this issue Mar 4, 2021 · 3 comments · Fixed by #994

Comments

@sebastian-foreflight
Copy link

I'm trying to build with Xcode 12.5 Beta 3 and I'm encountering this error

Umbrella header for module 'Segment' does not include header 'SEGReachability.h'

I'm not sure if this is new in Beta 3 or not as I haven't tried to build with the previous betas.

I'm using SPM and am currently including commit ce4280d514ef55f7b46f507714c5924ce0e83af0. The SDK is included as a module with @import Segment;

@Camji55
Copy link

Camji55 commented Mar 8, 2021

Try pointing SPM to the master branch until the next release is published.

@sebastian-foreflight
Copy link
Author

The commit I'm using, ce4280d514ef55f7b46f507714c5924ce0e83af0, is the current tip of the master branch. Looking at https://github.com/segmentio/analytics-ios/blob/master/Segment/Classes/Segment.h I also see that SEGReachability.h is not currently included there. To be sure, I've tested with SPM set to master and hit the same build error.

From the commit history, it looks like some other Xcode 12.5 issues have been recently fixed, but this one is still an issue as far as I can tell. I've fixed similar issues in SPM packages under my control by adding the missing headers.

@sebastian-foreflight
Copy link
Author

As an update, I've replicated this in a sample project that includes master via SPM I can upload it if that's helpful.

One finding, the issue is actually a warning in Xcode, but it's converted to an error because we have Treat Warnings as Errors set to Yes like I suspect many others do.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants