Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Duplicate RCTMethodInfo #137

Closed
iegik opened this issue Sep 4, 2017 · 19 comments
Closed

Duplicate RCTMethodInfo #137

iegik opened this issue Sep 4, 2017 · 19 comments

Comments

@iegik
Copy link

iegik commented Sep 4, 2017

Was:

#if __has_include("RCTBridgeModule.h")
  #import "RCTBridgeModule.h"
#else
  #import <React/RCTBridgeModule.h>
#endif

Should:

#if __has_include(<React/RCTBridgeModule.h>)
  #import <React/RCTBridgeModule.h>
#else
  #import "RCTBridgeModule.h"
#endif
@iegik iegik mentioned this issue Sep 4, 2017
Closed
@amitbravo
Copy link

If I do this , I got #import "RCTDevMenu.h" not found error :(

@iegik
Copy link
Author

iegik commented Sep 5, 2017

Add following into Header Search Paths

$(inherited) (non-recursive)
${PODS_DEV_ROOT}/React (recursive)
$(BUILT_PRODUCTS_DIR)/usr/local/include (recursive)

@iegik
Copy link
Author

iegik commented Sep 10, 2017

#138 (comment)

@iegik iegik mentioned this issue Sep 10, 2017
Closed
hellogerard added a commit to hellogerard/react-native-permissions that referenced this issue Sep 11, 2017
@hellogerard
Fix build for RN 0.48
4508974 
- See zoontek#137
@hellogerard hellogerard mentioned this issue Sep 11, 2017
Merged
@MoOx
Copy link
Collaborator

MoOx commented Sep 15, 2017

#138 (comment)

@isAlmogK
Copy link

Has this been fixed

@benvium
Copy link

benvium commented Sep 18, 2017

@AlmogRnD It's not working for me - I've tried targeting the commit with the merged fix above (package.json "react-native-permissions": "git+ssh://[email protected]/yonahforst/react-native-permissions.git#e7454a9c62e773a8d5ba5c6dc20843beb56c3c81",) and I'm still getting the same error on RN 48.3 :-(

samratshekhar added a commit to samratshekhar/react-native-calendar-events that referenced this issue Sep 22, 2017
@samratshekhar
Fix build for RN 0.48
bbf9f7b 
zoontek/react-native-permissions#137
@samratshekhar samratshekhar mentioned this issue Sep 22, 2017
Merged
@exKAZUu exKAZUu mentioned this issue Sep 23, 2017
Merged
@yonahforst
Copy link
Collaborator

fixed in #146

@MoOx
Copy link
Collaborator

MoOx commented Sep 27, 2017

Any plan to cut a release?

@yonahforst
Copy link
Collaborator

@MoOx just published 1.0.1 🎉

@MoOx
Copy link
Collaborator

MoOx commented Sep 28, 2017

Thank you!

@matart15 matart15 mentioned this issue Oct 2, 2017
Closed
@Pianist038801
Copy link

@legik Thanks for sharing!

wmcmahan pushed a commit to wmcmahan/react-native-calendar-events that referenced this issue Nov 13, 2017
@samratshekhar @wmcmahan
Fix build for RN 0.48 (#94)
cedd915 
zoontek/react-native-permissions#137
@paullinator paullinator mentioned this issue Dec 31, 2017
Closed
@snarang2
Copy link

@iegik - where do we change the import priority?
I mean which file?

@iegik
Copy link
Author

iegik commented Feb 27, 2018

@snarang2 in *.h file, checkout this PR https://github.com/vinnyjth/react-native-permissions/pull/1/files

@AliaMYH
Copy link

AliaMYH commented Mar 21, 2018 •
edited
Loading

Changing the priority works for me but then i get a linker error:

ld: library not found for -lPods-GroceriesApp
clang: error: linker command failed with exit code 1 (use -v to see invocation)

How do I handle that ?

Actually it's quite weird, when I open .xcworkspace as opposed to .xcodeproj, I get 27 linker errors, clearly I've installed something incorrectly.

thebradbain added a commit to thebradbain/react-native-oauth that referenced this issue Apr 7, 2018
@thebradbain
Fix duplicate RCTMethodInfo import (facebook/react-native#15775) (zoo…
02c6c1d 
…ntek/react-native-permissions#137)
@rgabs rgabs mentioned this issue May 11, 2018
Merged
@chandanyadav7
Copy link

i also have the same issue , In which file we need to change this

ruen89 referenced this issue in XiteTV/react-native-tvos-controller Aug 7, 2018
@ruen89
Update ReactNativeTvosController.h
2ed5a72
@suijunqiang
Copy link

The issue reproduced on the react-native 0.56.0

@TheSixer
Copy link

@suijunqiang Same issue here on the react-native 0.56.0. Has the problem been solved?

@suijunqiang
Copy link

@thesixth correct, it can be fixed as @iegik said.

@suijunqiang suijunqiang mentioned this issue Aug 21, 2018
Open
shalin-jasani added a commit to shalin-jasani/react-native-oauth that referenced this issue Sep 10, 2018
@shalin-jasani
Merge fullstackreact/react-native-oauth master branch into shalin-jas…
6e4a1c4 
…ani/fullstackreact/react-native-oauth master branch (#1)

* Added fullstackreact#171
and my NSString *clientID fix, which might not be as good as
https://github.com/fullstackreact/react-native-oauth/blob/2f8c8d1483526bbc8a6ca72183c6d11a71538ad3/ios/OAuthManager/OAuthManager.m

* Added fullstackreact#171
and my NSString *clientID fix, which might not be as good as
https://github.com/fullstackreact/react-native-oauth/blob/2f8c8d1483526bbc8a6ca72183c6d11a71538ad3/ios/OAuthManager/OAuthManager.m

* Added fullstackreact#171
and my NSString *clientID fix, which might not be as good as
https://github.com/fullstackreact/react-native-oauth/blob/2f8c8d1483526bbc8a6ca72183c6d11a71538ad3/ios/OAuthManager/OAuthManager.m

* Merge PR 121, fix user agent, fix full screen webview

* Changed if/else statement to avoid React error.

* Fix duplicate RCTMethodInfo import (facebook/react-native#15775) (zoontek/react-native-permissions#137)

* Fix duplicate React library import error conflict w/certain pods

* Pass back response headers over javascript bridge

* Dispatch safariViewController on main queue

The safariViewController dispatch was occuring on another thread.
This sometimes caused app crashes when the view was presented,
in particular if the keyboard had been presented via a TextInput or
other component. The resulting crash complained about
_cachedSystemAnimationFence and the main thread. This has been with
other React Native apps that load a viewController.

Dispatching to present the viewController on the main thread fixes this
issue.

* Fix build issue

* Remove deprecated @OverRide

* Fix: Duplicate RCTMethodInfo while building iOS app

* Fix error for redefinition of RCTMethodInfo

* Ignored dist/

* 2.1.16

* 2.1.17

* 2.1.18
@PimDeWitte PimDeWitte mentioned this issue Sep 21, 2018
Merged
jdegger added a commit to jdegger/react-native-flic-nonpbf that referenced this issue Sep 24, 2018
@jdegger
Change import order of RCTBridge conditional
c9fa961 
Was not able to build without changing this to this format. See zoontek/react-native-permissions#137 for reference.
@jdegger jdegger mentioned this issue Sep 24, 2018
Merged
@leechance
Copy link

@suijunqiang how did you solved this issue?

TalentDev00 added a commit to TalentDev00/workspace-react-native-OAuth that referenced this issue Jan 10, 2020
@TalentDev00
Fix duplicate RCTMethodInfo import (facebook/react-native#15775) (zoo…
83ed82b 
…ntek/react-native-permissions#137)
albanqoku pushed a commit to albanqoku/react-native-calendar that referenced this issue Feb 26, 2021
@samratshekhar @albanqoku
Fix build for RN 0.48 (#94)
cdc99dd 
zoontek/react-native-permissions#137
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests