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

Drop usage of the dead rnpm repository #148

Merged
merged 1 commit into from
Apr 17, 2017
Merged

Conversation

dantman
Copy link
Contributor

@dantman dantman commented Apr 7, 2017

rnpm has been replaced by react-native link, the rnpm package is no longer maintained and is out of sync with the code that actually calls postlink.

`rnpm` has been replaced by `react-native link`, the `rnpm` package is no longer maintained and is out of sync with the code that actually calls postlink.
@codesinghanoop
Copy link

react-native link does not work in case of android.

@dantman
Copy link
Contributor Author

dantman commented Apr 12, 2017

How so? react-native link is rnpm. rnpm has died and been merged into react-native itself.

@itinance
Copy link
Contributor

This PR would solve "react-native link" for android: #146
But the issue here is AFAIK, that this lib uses 2 different approaches ("android native" vs javascript) on android side and the automatic link process can only handle one of them

@dantman
Copy link
Contributor Author

dantman commented Apr 12, 2017

This PR has nothing to do with Android!!! Other than the fact that the fatal error breaks linking on all platforms.

All this PR does is fix the broken rnpm-postlink.js script, which only applies changes to the iOS project.

@itinance
Copy link
Contributor

@dantman my comment was about the comment of @codesinghanoop

@andpor andpor merged commit 23227bf into andpor:master Apr 17, 2017
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 this pull request may close these issues.

4 participants