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

fsevents install script failing #48

Open
Sentoj opened this issue Jun 13, 2016 · 9 comments
Open

fsevents install script failing #48

Sentoj opened this issue Jun 13, 2016 · 9 comments
Labels

Comments

@Sentoj
Copy link

Sentoj commented Jun 13, 2016

Hi, I am trying to get F8 app to build for Windows Phone. When running npm install, I get the following error:

npm-install-failure

After I ran it as administrator, it managed to create npm-debug.log (attached)

npm-debug.zip

Any idea what the problem could be?

N.B: I am running that on a Windows 10 machine and I have node 6.2.1 and npm 3.9.3 (as shown in the attached screenshot).

Any help would be appreciated,
Hisham

@mpodwysocki
Copy link

@Hochman85 You cannot install via npm using the Git Bash as it causes errors. You need to install using the normal command prompt cmd.exe.

We had earlier documentation on this and will make sure that gets updated. Thanks for the bug report!

@Sentoj
Copy link
Author

Sentoj commented Jun 13, 2016

@mattpodwysocki Thanks for your quick reply. I am actually using cmd.exe and not Git Bash (the screenshot in my original question is from the default Windows 10 command line). Also when I installed Git 2.8.4 (64 bit), I made sure it used cmd.exe and not Git Bash.

Any idea please? If you could share the latest documentation with me that'd be great too.

Thanks!

@es128
Copy link

es128 commented Jun 13, 2016

@mpodwysocki your repo contains an npm-shrinkwrap.json which includes fsevents. The underlying problem is npm/npm#2679, but in the meantime you could probably fix this problem by not uploading a shrinkwrap to the repo so that any system-specific optional dependencies get pulled in as designed. Or if you insist on the shrinkwrap, just take care to ensure there is no fsevents entry on it.

@rozele
Copy link
Member

rozele commented Jun 13, 2016

The general guidance I would give here is to just wait until the upstream repo (https://github.com/fbsamples/f8app) gets updated to [email protected].*. Once that happens, we'll be able to use the proper react-native-windows package, and we'll have a branch / fork shortly after that works a bit more seamlessly.

@Sentoj
Copy link
Author

Sentoj commented Jun 14, 2016

@rozele Alright - thanks - I'd wait then. Where is the best place to observe such a change please? Ideally, I'd like to be notified :)

@rozele
Copy link
Member

rozele commented Jun 21, 2016

@es128 thanks for the suggestion, we've removed NPM shrinkwrap for now.

@rozele
Copy link
Member

rozele commented Jun 21, 2016

@Hochman85 I went ahead and upgraded the app to [email protected], so you should be good to go. The instructions have changes a bit, so you may want to install a fresh repo (you now only need to run npm install, no need to replace the node_modules\react-native folder). Make sure you use this fork's master branch.

@chrisahardie
Copy link

@rozele I'm still struggling with this after cloning and checking out the master branch. My package.json indicates "react-native": "0.28.0"

After doing an npm install via my command prompt - not git bash - I get the same error described above.

Please find my npm log attached - thanks for any input!

npm-debug.zip

@jurby
Copy link

jurby commented Dec 6, 2016

@chrisahardie +1 same issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants