-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
File "FirebaseCore-umbrella.h" generated when using Carthage causing warnings #5209
Comments
Thanks for the report. We'll fix for the next release. In the meantime, a workaround is to manually delete 'FirebaseCore-umbrella.h' and any other unreferenced umbrella.h files. |
I am seeing this as well for |
The 6.21.0 regression has been fixed for the two binary distributions - Carthage and Zipfile - and will be available in the next release. @donnywdavis I don't see the issue for CocoaPods installs. If you're seeing it there, please create another issue with the template filled out. |
@paulb777 I see you merged a PR fixing this issue but I still see the following warnings while running the app: warning: Swift error in fallback scratch context: error: :1:1: error: umbrella header for module 'FirebaseCore' does not include header 'FirebaseCore-umbrella.h' :1:9: note: in file included from :1: error: /Users/user/GitHub/project/Carthage/Build/iOS/Firebase.framework/Headers/Firebase.h:15:9: error: could not build module 'FirebaseCore' error: :1:1: error: umbrella header for module 'FirebaseAuth' does not include header 'FirebaseAuth-umbrella.h' error: :1:1: error: umbrella header for module 'FirebaseDatabase' does not include header 'FirebaseDatabase-umbrella.h' error: :1:1: error: umbrella header for module 'FirebaseDynamicLinks' does not include header 'FirebaseDynamicLinks-umbrella.h' error: :1:1: error: umbrella header for module 'FirebaseFirestore' does not include header 'FirebaseFirestore-umbrella.h' error: :1:1: error: umbrella header for module 'FirebaseFunctions' does not include header 'FirebaseFunctions-umbrella.h' error: :1:1: error: umbrella header for module 'FirebaseInstanceID' does not include header 'FirebaseInstanceID-umbrella.h' error: :1:1: error: umbrella header for module 'FirebaseMessaging' does not include header 'FirebaseMessaging-umbrella.h' error: :1:1: error: umbrella header for module 'FirebaseRemoteConfig' does not include header 'FirebaseRemoteConfig-umbrella.h' error: :1:1: error: umbrella header for module 'FirebaseStorage' does not include header 'FirebaseStorage-umbrella.h' error: could not build Objective-C module 'Firebase'Any ideas? Sorry to hijack this thread, let me know if I need to create a new Issue for this. |
@gbejarano01 Thanks for the report. I've just compared the 6.21.0 and 6.22.0 releases and verified that the offending umbrella.h files are no longer there. Please make sure that you've run |
Thanks @paulb777 !! yeah, last time I run Carthage it pulled 6.21.0, wil will try pulling 6.22.0. Thanks! |
That worked @paulb777 Thanks for the quick response! |
[REQUIRED] Step 1: Describe your environment
[REQUIRED] Step 2: Describe the problem
When adding Firebase v6.21.0 to your project using Carthage, a header file called "FirebaseCore-umbrella.h" is created in FirebaseCore. This will trigger a warning in Xcode, as it is not included in the header file defined in the module.modulemap of FirebaseCore.
This does not happen when using v6.19.0 of Firebase.
Warning in Xcode is:
umbrella header for module 'FirebaseCore' does not include header 'FirebaseCore-umbrella.h'
Steps to reproduce:
Follow steps in Carthage.md to install setup Firebase using Carthage, by adding this line to the Carthage file:
binary "https://dl.google.com/dl/firebase/ios/carthage/FirebaseAnalyticsBinary.json" == 6.21.0
The text was updated successfully, but these errors were encountered: