-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
[🐛] 'FirebaseFunctions/FirebaseFunctions-Swift.h' file not found #6382
Comments
🤔 Hmm I don't recognize these, but I see them show up pretty frequently now that use_frameworks is required. Why are these here? Where does the recommendation to turn them on come from? What problem do they solve, if you cannot build currently?
Flipper is not going to work with use_frameworks - that's not fixed yet as far as I know:
Does my demo script (still on 0.69.0, sorry - but reasonably representative) work for you? https://github.com/mikehardy/rnfbdemo/blob/main/make-demo.sh Does it still work if you chop the installed modules down to just the ones you use? |
Hey, im also experience the same issue, and have a very similar @mikehardy the usage of the specific We are using these in leu of the I believe the issue may also be related to a recent issue in FirebaseUI-IOS here, which looks to have been resolved here. Keen to discuss, and if I find a working solution with Flipper, I will provide. |
@octiva i just did a release last night that has latest iOS SDK dependency so if it's released, it's available here Not sure what's going on with the storage compile fail? Dies the make-demo.sh I link everywhere work? It has storage and it works for me ? 🤔 |
@mikehardy I cant see the new release version, will this be |
Apologies thought I published but did not, job running now... |
@mikehardy I just tested I have investigated a little more, but with no success. Will be moving on from this and following a less ideal path of using the Firebase web package in the app for Storage. Will likely look to pick this back up at some point, but my gut feeling is that it is upstream of this package and likely needs to be resolved in |
I read through the changes in linked FirebaseUI area, and this is the style of change they did:
We are all 9+ though, so we just changed the import. I think cocoapods might generate that with |
@mikehardy "if I understand correctly, the projects are attempting to set $RNFirebaseAsStaticFramework to true and use_modular_headers but not set use_frameworks itself ?", yes that is exactly the case. I cannot use use_frameworks as I also use react-native-skia which does not work with frameworks. The other FB modules work OK, it is just storage that seems to have an issue. |
I think based on a thread upstream that cocoapods generates the Swift bridging header file for storage, only in response to the use frameworks toggle. This may be the fundamental reason for the new requirement to use that toggle, perhaps Paul Bausterien knows that for sure - I'm mobile and cannot tag them properly unfortunately |
RN 0.68, firebase 15.2.0, podfile:
Any ideas? 🤔 |
@Fortidude this looks like the equivalent problem from storage except for functions. The firebase-ios-sdk release notes are clear, use_frameworks is required. Are you using that flag in your podfile? @paulb777 may be able to elaborate on this specific integration need in cocoapods? I had enough time just now to properly tag him in 😅 |
While the |
@paulb777 can you show how to user
|
@ithustle you cannot get away from it I think. Flipper must be fixed to work in There is no other way I'm aware of I'm working on these things (Flipper + use_frameworks compatibility, and Hermes + use_frameworks compatibility which will hopefully be in react-native 0.70) but it will take some time, months I suspect. So I'm going to close this as a sort of "blocked by upstream" resolution. for Functions and Storage you must have The |
I ended up downgrading to firebase@14 all the firebase packages to be able to use storage with flipper. |
Can you share you package.json? |
Little frustating... I need to use Appcheck and as soon as I upgrade to version 15, I can't get a thing to work.. |
@ithustle Thing is that I need to add to my project AppCheck which depends on the latest version(15) of RN Firebase. I tried to make it as modules, but it always complains about FirebaseStorage-Swift. I tried with use_frameworks and that stopped complaining about this library but I have another library not compiling. |
Any progress here? I'm stuck with the same error specific to Firebase Storage only (@15.2) |
@juanctecdam AppCheck has been available for more than a year, it is available in v14. Firebase storage and Firebase functions require |
hi, |
@gitcoder-23 that's what you see if you do not enable |
@mikehardy Hi, Thanks. But now fatal error: 'FlipperKit/FlipperClient.h' file not found.. please help |
@gitcoder-23 you must read the instructions. Flipper is not compatible with https://rnfirebase.io/#altering-cocoapods-to-use-frameworks (big note there) |
Hi, Disabling Hermes & Flipper causes app crash on build by my side.. I'll need another solution to manage files for the moment :s Anybody found the solution ? |
That's really unexpected, if you provided any sort of details (https://stackoverflow.com/help/how-to-ask) I/we might be able to help. I can tell you that it should work though, I just re-ran my build test rig yesterday and it was fine https://github.com/mikehardy/rnfbdemo/blob/main/make-demo.sh You probably don't need to disable Hermes though, for what it's worth. Hermes appears to work well now. Flipper still definitely not working. |
I made some changes regarding your script :
And it worked ! Thanks a lot for your help ! |
Can you please post the whole of your pod file? |
@danias you can run this script any time I keep it up to date / in sync with current react-native as I work with react-native releases group to test new releases so it's always pretty fresh It generates a Podfile that works https://github.com/mikehardy/rnfbdemo/blob/main/make-demo.sh |
Is working for me with the version 14.11.1 and $FirebaseSDKVersion = '8.15.0' |
It is working for you because you have specifically downgraded to a version of react-native-firebase that includes a version of firebase-ios-sdk from before they converted that to Swift (and before they started requiring So sure, if you downgrade you can "workaround" this problem. However, you will have bought yourself other problems. Check the release notes and see if any of the changes between firebase-ios-sdk 8.15.0 and 10.2.0 (soon to be 10.3.0 here) are for issues that will cause you problems https://firebase.google.com/support/release-notes/ios |
This worked for me, thanks |
I had to remove Flipper and add use_frameworks! :linkage => :static |
Yes, as we document here: https://rnfirebase.io/#altering-cocoapods-to-use-frameworks |
Not at all working for me. I tried every solution. Please check my
|
react-native 0.70.0
that work for me. |
Yes 🥹 🙌, Finally I resolve the issue, Resolve react_native_pods.rb with node to allow for hoistingrequire Pod::Executable.execute_command('node', ['-p', use_modular_headers!platform :ios, min_ios_version_supported If you are using a
|
Don't do that, or if you do that do not come here for support.
You are attempting to use the module contrary to the documentation so no support is provided |
Hey all, I had the same problem and following the "Getting Started" instructions on this page completely solved it for me: https://rnfirebase.io/ |
Hello, checking back on this. |
No. firebase-ios-sdk requires use_frameworks, and Flipper does not work with it. Before anyone posts on this again and triggers notifications please do the research - answer these questions 1) is Flipper still the way to debug things in react-native and 2) after checking the Flipper repo does it work with use_frameworks? Check those things. Answer yes to both questions if you can, then post. |
Thanks a lot, actually you made my day, I was struggling with it for hours, that documentation also solved it for me. |
When building for iOS, I keep getting the following error:
dependencies are all at latest version
podfile
The text was updated successfully, but these errors were encountered: