-
Notifications
You must be signed in to change notification settings - Fork 211
[React Native 0.48]: Error: Redefinition of RCTMethodInfo#157
[React Native 0.48]: Error: Redefinition of RCTMethodInfo #157exentrich wants to merge 1 commit into
Conversation
Build failed without this fix
joncursi
commented
Sep 11, 2017
Confirmed working on my end. I am using @exentrich 's patch for now to get around this:
"react-native-oauth": "git+ssh://git@github.com/exentrich/react-native-oauth#patch-1",
Kottidev
commented
Sep 20, 2017
can you accept this PR ?
karlAlnebratt
commented
Oct 6, 2017
Experiencing same issue.
"react": "16.0.0-beta.5",
"react-native": "0.49.1",
"react-native-oauth": "2.2.0"
I'm using patch-package to fix this issue until this is released.
tqc
commented
Oct 8, 2017
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.
alesanabriav7
commented
Nov 23, 2017
works for me! thanks @exentrich
melihmucuk
commented
Dec 21, 2017
Im running on react-native v0.51.0 and it doesnt work with current npm version. Please merge this pull request to fix it.
mraible
commented
Jan 29, 2018
I ran into this today, using the latest and greatest react-native code (^0.52.2). I can confirm that using the following solved the problem:
"react-native-oauth": "git+ssh://git@github.com/exentrich/react-native-oauth#patch-1",
Compulsor-zz
commented
Mar 19, 2018
Can this PLEASE get merged. For crying out loud.
cranberyxl
commented
Mar 19, 2018
We forked it and are using the patch mentioned above.
sb8244
commented
Jun 1, 2018
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 ^
CapitanRedBeard
commented
Jul 23, 2018
@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