-
Notifications
You must be signed in to change notification settings - Fork 214
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
[React Native 0.48]: Error: Redefinition of RCTMethodInfo #157
base: master
Are you sure you want to change the base?
Conversation
Build failed without this fix
Confirmed working on my end. I am using @exentrich 's patch for now to get around this:
|
can you accept this PR ? |
Experiencing same issue.
I'm using patch-package to fix this issue until this is released. |
The code modified in this PR isn't actually the problem. The error comes up because there are multiple versions of the react native headers in the search path - the older version exists, but isn't the one you are actually using. Assuming you have a fairly standard setup, deleting node_modules/react-native and reinstalling should work - a lot of files got moved to different locations around 0.48, so the old ones may not get overwritten/removed with an upgrade. |
Is @tqc 's solution working for anyone els ? I'm only able to solve this with the patch. |
Following fullstackreact#157
works for me! thanks @exentrich |
Working with 0.51.0 too. This should be merged a few days ago when I started debugging this one 💃 |
Im running on react-native v0.51.0 and it doesnt work with current npm version. Please merge this pull request to fix it. |
I ran into this today, using the latest and greatest react-native code (
|
Can this PLEASE get merged. For crying out loud. |
We forked it and are using the patch mentioned above. |
This is still occurring and the only solution right now is to the use the forked patch. Is this going to be merged? Is there a better solution? |
TODO: create fork from react-native-oauth with fixes - fullstackreact/react-native-oauth#121 - fullstackreact/react-native-oauth#157 - fullstackreact/react-native-oauth#158 - fullstackreact/react-native-oauth#208 ^
@auser Can you merge this in and cut a new version? Seems this is working for everyone and has been a problem for everyone |
Build failed without this fix