-
Notifications
You must be signed in to change notification settings - Fork 213
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
[2.14.0] iOS build error #528
Comments
Hi @Bardiamist, Can you try cleaning your project and trying to build again. This would mean you must delete the folder in which your |
Hi @poovamraj Not helped:
|
@Bardiamist I am not able to reproduce this issue. Can you try and reproduce in this sample repository. The header file you mentioned is used for interop between Swift and ObjC code and it is autogenerated. That is why I assumed clean up should fix it :\ |
Not reproducing on the fresh project but still can't update my project. In my project I can't jump into |
Hi @Bardiamist where you able to fix this by any chance or any success in creating a reproducible sample for us? |
No, waiting now for something |
Hi! I have the same issue, with version 2.14.0 |
Hi @shrekuu @SmartByt3r @Bardiamist Can you try the suggestion from this Stackoverflow Thread. It asks for the "Defines Module" setting under "Packaging" of "Build Settings" to be set to |
My target already has "Defines Module" "YES" |
Hi, @Bardiamist any progress on this issue? It would be great to have a reproducible environment. Can you try to use a build setup similar to the one which is causing the issue on the sample app and then update it to see if the issue is reproducible? |
@poovamraj Hello
No, I'm still use react-native-auth0 2.13.3
Unfortunatelly my project is big and I have no idea what could be causing this problem You can create branch based on 2.13.3 and checrry pick only related change and I can test this branch. Then we can localize the problem. |
Same issue here. @poovamraj happy to help diagnose. |
@ebg1223 can you provide a reproducible sample so that we can look into this? |
@Bardiamist The changes related to this should be out of this commit here |
@poovamraj I think I figured out the issue. I am attempting to migrate from firebase auth(I am guessing a possibly common use case).
Removing this code allows the build to complete successfully. I am assuming it is useFrameworks:static that is preventing proper generation of headers. I know there are other libraries that require this, so I am sure the community would benefit from a solution that allows building in auth0 with useFrameworks: static, but at least for now, maybe docs can include something stating that it is incompatible with this build property and any libraries requiring it. |
@ebg1223 That's a great find! I'll check this one out and get the docs ready. @Bardiamist Can you verify this as well? |
@poovamraj Thank you!! I wish i knew more about the expo native build process to give insight into the importance of this flag, or how to correct it on the native side, but I have seen this requirement in other libs besides firebase. Maybe someone more knowledgeable in this aspect can suggest a fix to allow for useFrameworks:static to maintain compatibility with other libs. |
@poovamraj Hello, yes I'm use Firebase and it's require I use bare React native project so repro steps:
Or you can clone this repro: https://github.com/Bardiamist/diff/tree/auth0-use-frameworks Looks need to add |
@poovamraj Sorry I can't reproduce this. I messed up with code base during debugging. I end up creating a new repo and test react-native-auth0 first and move the rest of the code to the new repo code incrementally. I am not using Firebase. I only have the this default in Podfile:
|
Hey all 👋 I figured out what is causing the issue! So we are referencing the file from local context. So if you change the
In A0Auth0.m in your node_modules, this should start working (There are tones of problems with Hermes and New architecture if you enable Can someone verify this? I can try to figure out a patch. |
Hi all, does your app build if you add |
@poovamraj yes, it helped
@Widcket Tried to add it to |
@Bardiamist can you please try out the branch feature/static-linking-support? |
Tried, it works
|
Not building after update from 2.13.3 to 2.14.0
Screenshot
The text was updated successfully, but these errors were encountered: