RCTWebRTCDemo icon indicating copy to clipboard operation
RCTWebRTCDemo copied to clipboard

xcode 8.2.1 build error 'React/RTCLog.h' file not found

Open pkuhonker opened this issue 8 years ago • 5 comments

e0ef36c6-10db-4b31-80c2-13b4ff540529 ff07d9cd-d234-4a23-9fb8-93eeb9b975f5

pkuhonker avatar Jan 17 '17 07:01 pkuhonker

Any update on this issue? I am also facing the same with Xcode 8.2.1

payalbaldota avatar Jan 19 '17 08:01 payalbaldota

Howdy! I just ran into this problem. The issue is with how your Header path is defined. Basically, start over and make sure to follow these directions: https://github.com/oney/react-native-webrtc/blob/master/Documentation/iOSInstallation.md

rfelixallen avatar Jan 19 '17 17:01 rfelixallen

When I did npm install, the steps mentioned in the above URL where already performed and I could see the dependencies added in the demo project automatically. Problem is with RCTWebRCT project which gives library(libRCTWebRTC.a) as product. That project is missing the dependencies and not able to build the library, hence the demo is giving the build error.

payalbaldota avatar Jan 20 '17 08:01 payalbaldota

Upgrading react-native version to 0.40.0 solved the issue. Also I changed the import statement for socket.io in main .js file. Following are the steps to solve the build errors.

  1. Clone/Download repository
  2. hit npm install on project root folder from terminal
  3. hit react-native-git-upgrade on root folder
  4. hit rm -rf node_modules
  5. hit watchman watch-del-all
  6. Change import statement "import io from 'socket.io-client/socket.io';" in main.js to "import io from 'socket.io-client/dist/socket.io';"
  7. hit npm install --save on root folder from terminal

Now you can run the demo iOS project with few signing related changes. Forked working copy https://github.com/payalbaldota/RCTWebRTCDemo

@oney Can you please merge these changes in master branch?

payalbaldota avatar Jan 24 '17 06:01 payalbaldota

@payalbaldota thank you very much.

I had checked your master branch and was going to merge directly from your branch, but noticed that there are some thing needs to be adjust:

  1. you have your own codes in MainApplication.java
  2. you remove lots of permission in AndroidManifest.xml which are necessary to run the demo

would you might create a branch for upgrading RN 40 and made a PR?

thanks again!

zxcpoiu avatar Jan 26 '17 11:01 zxcpoiu