SiftScience / sift-ios

Sift iOS SDK
MIT License
19 stars 26 forks source link

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

Open ghost opened 2 years ago

ghost commented 2 years ago

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 commented 2 years ago

@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 commented 1 year ago

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

samsiner commented 1 year ago

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

image

afanaian01 commented 1 year ago

Any update on this issue?

viaskal-sift commented 1 year ago

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

afanaian01 commented 1 year ago

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

Yes. We can close this.

mthormann-dhi commented 1 year ago

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 commented 1 year ago

@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 commented 1 year ago

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 commented 1 year ago

noted, I will reopen the issue then

afanaian01 commented 6 months ago

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

Screenshot 2024-05-22 at 9 32 45 AM
lobotony commented 1 month ago

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 commented 1 week ago

@afanaian01, @lobotony

Could you try this PR (https://github.com/SiftScience/sift-ios/pull/153). It should resolve your issues with Umbrella headers