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.

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

Is Examples/VideoPlayer completely broken now? #117

Closed
PublicParadise opened this issue Dec 20, 2015 · 9 comments
Closed

Is Examples/VideoPlayer completely broken now? #117

PublicParadise opened this issue Dec 20, 2015 · 9 comments

Comments

@PublicParadise
Copy link
Contributor

Run this:

git clone https://github.com/brentvatne/react-native-video.git
cd react-native-video
npm install
cd Example/VideoPlayer
npm install

You'll get:

npm ERR! git rev-list -n1 feature/android-support: fatal: ambiguous argument 'feature/android-support': unknown revision or path not in the working tree.
npm ERR! git rev-list -n1 feature/android-support: Use '--' to separate paths from revisions, like this:
npm ERR! git rev-list -n1 feature/android-support: 'git <command> [<revision>...] -- [<file>...]'
npm ERR! git rev-list -n1 feature/android-support: 
npm ERR! git rev-list -n1 feature/android-support: fatal: ambiguous argument 'feature/android-support': unknown revision or path not in the working tree.
...

Those errors are probably coming from this line in package.json:

"react-native-video": "brentvatne/react-native-video#feature/android-support"
@brentvatne
Copy link
Contributor

oh, you can change that to master in the package.json -- we deleted that branch. PR welcome, thanks!

@PublicParadise
Copy link
Contributor Author

@brentvatne Oh, there is more...

On ios I ran this:

git clone https://github.com/brentvatne/react-native-video.git
cd react-native-video
react-native-video.git
cd react-native-video
npm install
cd Example/VideoPlayer
npm install react-native
open VideoPlayer.xcodeproj

In XCode I see that the project is incoherent:

screen shot 2015-12-20 at 4 25 01 pm

I am seeing two problems:

  • VideoPlayer.xcodeproj includes node_modules/react-native-video/RCTVideo.xcodeproj, which should reference the source files from .../react-native-video/Example/VideoPlayer/../../RCTVideo.h etc.
  • what's the deal with libicucore.dylib?

@PublicParadise
Copy link
Contributor Author

@brentvatne Wait, there is more!

After fixing the incorrect references in Examples/VideoPlayer/node_modules/react-native-video/RCTVideo.xcodeproj and hitting run I am getting this puppy:

screen shot 2015-12-20 at 4 34 14 pm

That looks like index.ios.js hasn't been updated in a long time. I suspect this error is about magus/react-native-facebook-login#31

@PublicParadise
Copy link
Contributor Author

@brentvatne What has happened to this project? It was in a fairly good shape the last time I played with it (~July 2015). My PR from June (#68) is still in the freezer?

Has this project become abandonware?

@PublicParadise
Copy link
Contributor Author

In the XCode terminal I am seeing:

2015-12-20 16:43:26.634 [error][tid:main] Unable to resolve module react-native-video from /Users/bparadie/git/research/react/react-native-video/Examples/VideoPlayer/index.ios.js: Invalid directory /Users/node_modules/react-native-video

Adding these lines to index.ios.js didn't help:

  NativeModules,
  requireNativeComponent,

@PublicParadise
Copy link
Contributor Author

My environment:

OSX 10.10.5
XCode 7.2
npm --version # 2.14.15
node --version # v5.3.0
watchman --version #3.8.0

@PublicParadise
Copy link
Contributor Author

@brentvatne Ok, I got the example not working again.
Here are the steps:

  • fix package.json:
  "dependencies": {
    "react-native": "^0.16.0",
    "react-native-video": "0.6.1"
  }
  • fix node_modules/react-native-video/RCTVideo.xcodeproj, let sources point to ../..

@PublicParadise
Copy link
Contributor Author

I updated #68 and added a fix for package.json.
Any takers?

@PublicParadise
Copy link
Contributor Author

@brentvatne There were a bunch of things wrong with the VideoPlayer example. Just adding a version number to the react-native-video dependency in package.json wasn't enough, because that doesn't allow you to test VideoPlayer against your local changes.
Either way, I cleaned everything up in #118.

Now, may I hope that this PR will be looked at? I am thinking of facebook/react-native#1546 where I spent a lot of time fine tuning the PR and nobody looked at it until I had to assume that there was no interest in it. So I closed it...

isair added a commit that referenced this issue Feb 1, 2016
- Updated package.json to use local react-native-video in a simple manner.
- Added shared scheme for target.
- Updated poject settings.
- Now using react-native 0.18.1.

Closes #117
isair added a commit that referenced this issue Feb 1, 2016
- The test script just does an eslint check. (Closes #121)
- Refactored JS code to pass the check.
- Fixed VideoPlayer example. (Closes #117)
  - Updated package.json to use local react-native-video in a simple manner.
  - Added shared scheme for target.
  - Updated poject settings.
  - Now using react-native 0.18.1.
@isair isair closed this as completed in #136 Feb 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants