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

Umbrella header does not include a variety of header files #114

Open
ghost opened this issue Sep 30, 2022 · 13 comments
Open

Umbrella header does not include a variety of header files #114

ghost opened this issue Sep 30, 2022 · 13 comments

Comments

@ghost
Copy link

ghost commented Sep 30, 2022

Installing through SPM I see about 15 warnings regarding the public headers.

Could these be included? This messes with projects that treat warnings as errors.

/Users/AnyUser/Library/Developer/Xcode/DerivedData/AnyProject/SourcePackages/checkouts/sift-ios/Sift/Sift.h:179:1: warning build: Umbrella header for module 'Sift' does not include header 'SiftCircularBuffer.h'

/Users/AnyUser/Library/Developer/Xcode/DerivedData/AnyProject/SourcePackages/checkouts/sift-ios/Sift/Sift.h:179:1: warning build: Umbrella header for module 'Sift' does not include header 'NSData+GZIP.h'

/Users/AnyUser/Library/Developer/Xcode/DerivedData/AnyProject/SourcePackages/checkouts/sift-ios/Sift/Sift.h:179:1: warning build: Umbrella header for module 'Sift' does not include header 'SiftEvent+Private.h'

/Users/AnyUser/Library/Developer/Xcode/DerivedData/AnyProject/SourcePackages/checkouts/sift-ios/Sift/Sift.h:179:1: warning build: Umbrella header for module 'Sift' does not include header 'SiftIosAppState.h'

/Users/AnyUser/Library/Developer/Xcode/DerivedData/AnyProject/SourcePackages/checkouts/sift-ios/Sift/Sift.h:179:1: warning build: Umbrella header for module 'Sift' does not include header 'SiftIosDeviceProperties.h'

/Users/AnyUser/Library/Developer/Xcode/DerivedData/AnyProject/SourcePackages/checkouts/sift-ios/Sift/Sift.h:179:1: warning build: Umbrella header for module 'Sift' does not include header 'Sift+Private.h'

/Users/AnyUser/Library/Developer/Xcode/DerivedData/AnyProject/SourcePackages/checkouts/sift-ios/Sift/Sift.h:179:1: warning build: Umbrella header for module 'Sift' does not include header 'SiftQueue.h'

/Users/AnyUser/Library/Developer/Xcode/DerivedData/AnyProject/SourcePackages/checkouts/sift-ios/Sift/Sift.h:179:1: warning build: Umbrella header for module 'Sift' does not include header 'SiftDebug.h'

In file included from <module-includes>:1:
/Users/AnyUser/Library/Developer/Xcode/DerivedData/AnyProject/SourcePackages/checkouts/sift-ios/Sift/Sift.h:179:1: warning build: Umbrella header for module 'Sift' does not include header 'SiftUploader.h'

/Users/AnyUser/Library/Developer/Xcode/DerivedData/AnyProject/SourcePackages/checkouts/sift-ios/Sift/Sift.h:179:1: warning build: Umbrella header for module 'Sift' does not include header 'SiftIosAppStateCollector+Private.h'

/Users/AnyUser/Library/Developer/Xcode/DerivedData/AnyProject/SourcePackages/checkouts/sift-ios/Sift/Sift.h:179:1: warning build: Umbrella header for module 'Sift' does not include header 'SiftUtils.h'

/Users/AnyUser/Library/Developer/Xcode/DerivedData/AnyProject/SourcePackages/checkouts/sift-ios/Sift/Sift.h:179:1: warning build: Umbrella header for module 'Sift' does not include header 'SiftIosAppStateCollector.h'

/Users/AnyUser/Library/Developer/Xcode/DerivedData/AnyProject/SourcePackages/checkouts/sift-ios/Sift/Sift.h:179:1: warning build: Umbrella header for module 'Sift' does not include header 'SiftIosDevicePropertiesCollector.h'

/Users/AnyUser/Library/Developer/Xcode/DerivedData/AnyProject/SourcePackages/checkouts/sift-ios/Sift/Sift.h:179:1: warning build: Umbrella header for module 'Sift' does not include header 'SiftHtDictionary.h'

/Users/AnyUser/Library/Developer/Xcode/DerivedData/AnyProject/SourcePackages/checkouts/sift-ios/Sift/Sift.h:179:1: warning build: Umbrella header for module 'Sift' does not include header 'SiftTokenBucket.h'

@vincent-sift
Copy link
Contributor

@rodrigordc thanks for bringing this to our attention. We will address these issues and update this issue when we have an update. Thanks again :)

@michaeltaverne
Copy link

Seeing the same thing, would appreciate a fix. Thanks!

@samsiner
Copy link

samsiner commented Mar 7, 2023

I think this is the same or similar issue (Xcode 14.2). Would appreciate a fix:

image

@afanaian01
Copy link

Any update on this issue?

@viaskal-sift
Copy link
Contributor

@afanaian01 could you pls check if this is still the case in the latest version? Thanks!

@afanaian01
Copy link

@afanaian01 could you pls check if this is still the case in the latest version? Thanks!

Yes. We can close this.

@mthormann-dhi
Copy link

This is still happening in Sift 2.1.2 on Xcode 14.3.1. Unless I'm missing something I don't think anything is fixed.

Sift warnings

@viaskal-sift
Copy link
Contributor

@mthormann-dhi hi, could you pls try the latest version (2.1.5) and let us know if you still have the issue?

@mthormann-dhi
Copy link

mthormann-dhi commented Oct 11, 2023

Yeah still seeing it on release configuration builds. Opened project, cleaned build folder, then did an archive. Not seeing on debug builds though. (edit: Xcode 15.0)

@viaskal-sift
Copy link
Contributor

noted, I will reopen the issue then

@viaskal-sift viaskal-sift reopened this Oct 11, 2023
dbarden added a commit to dbarden/sift-ios that referenced this issue Apr 8, 2024
Creates a `include` header that will contain the headers that should be
distributed.

This avoid copying all the headers and solves the compile time error
shown in SiftScience#114
@afanaian01
Copy link

We're seeing this again. Xcode 15.4 - sift 2.1.8

Screenshot 2024-05-22 at 9 32 45 AM

@lobotony
Copy link

lobotony commented Oct 2, 2024

Still getting this with Xcode 15.3 and SDK 2.2.1.
The issue is easy to reproduce if you wrap the Sift SDK inside a Swift library package.
We're suppressing the warnings for with with swiftSettings: [.unsafeFlags(["-suppress-warnings"])], but I'd prefer if you'd take care of it in the SDK. Thanks!

apolwork pushed a commit to apolwork/sift-ios that referenced this issue Nov 8, 2024
@apolwork
Copy link

@afanaian01, @lobotony

Could you try this PR (#153). It should resolve your issues with Umbrella headers

esingin-sift added a commit that referenced this issue Nov 25, 2024
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

No branches or pull requests

8 participants