-
Notifications
You must be signed in to change notification settings - Fork 129
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
Building iOS to a real device fails #10
Comments
Probably related to this line https://github.com/mikehardy/rnfbdemo/blob/main/patches/react-native%2B0.69.0.patch#L101 |
Oh no! @raghavyadavm can you test that patch on any other project to see how it affects thing? If that patch is bad that means we have a generic problem with the upcoming react-native 0.69.1 as that patch will be included in the main release. This could be a big deal |
@mikehardy I created a new project using 69.1 and got the same error on a real device
|
Thanks again for bringing this up - I tagged you in the related react-native discussions for well-earned credit, of course unsubscribe if it's bothersome (and, apologies) - I dropped that part of the patch and make sure hermes is not enabled in make-demo.sh now, so it works with react-native 0.69.0 - I'll have to reverse that part of the patch as a forward-port to react-native 0.69.1, I'll do that in a bit |
Turns out (from the related PR on react-native) that this may always be incompatible with bitcode but apparently bitcode is deprecated in Xcode 14 anyway. So, it is perhaps best to simply disable it, then there's no problem. In Podfile in postinstall this will do it, I will probably update make-demo.sh to do this when I get a chance installer.pods_project.targets.each do |target|
target.build_configurations.each do |config|
config.build_settings['ENABLE_BITCODE'] = 'NO'
end
end |
Great thanks for the workaround. I will update my project with these settings |
Disabling Bitcode started giving me errors with other libraries like Reanimated. I ended up using framework with the option |
Yes - other libraries may have issues as well, it really is a big change, thanks for reporting what worked, that will help me and others going forward! |
I create a PR to update the demo.sh with the above changes #11 |
This workaround solved the issue for me, but I also had to disable Bitcode for release builds in my project's XCode config. |
Hmm - perhaps instead of Line 202 in 041c169
@raghavyadavm text of error messages vs images are strongly preferred if possible, I'm traveling and mobile right now most of the time so images are rough on me to see all pinch+zoom :-) - it might be best to post that in the react-native main repo as it looks like you've stumbled on "the next thing..." after getting the basic build+link to work. Hopefully it's an easy fix - you might try pinging @Kudo as he was the creator of the original fix that made Hermes build at all (the first, important step) and the error may be familiar to them, or...it could be aware from areas any of us are familiar with 😅 |
hi there! sorry being late for this response. i finally get a chance to investigate this problem and it turns out to me that the root cause is that due to React-Core podspec does not have a dependency to React-hermes, so here the import is not available. and the runtime is jsc but not hermes. i need some time to think a proper solution for the issue. just to update my investigation here. |
I always appreciate your investigations and efforts @Kudo - thanks for even trying, sincerely. I posted something related here on the road to 0.70 discussion - reactwg/react-native-releases#26 (reply in thread) I've exposed the issue in this script (at least) by adding exposure of the
|
nice finding for it's jsc not hermes by the |
This worked for me too :) |
I tried all of the solutions mentioned above, but still no luck. Edit: upgraded all other js modules and did "pod install" afterwards. Works now. I guess running "pod install" is a necessary step, so should be mentioned maybe in the resolution. Thanks. |
When building the rnfbdemo project to real iOS device it fails with the below error
The text was updated successfully, but these errors were encountered: