-
Notifications
You must be signed in to change notification settings - Fork 29.3k
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
Failed at the electron-mksnapshot@1.6.0 install script #30775
Comments
here is my debug file. The error is at the bottom of the file. Regards, |
Just as an afterthought , I thought I might just add , I tried install this in another fresh sample project of mine and it got installed without any problem. The latest version of electron-mksnapshot is 1.7.1 . Hence I tried changing dependency version in package.json of vscode . Same Error persists. |
Is this with npm 5.x? Could you try with 4.x? This might be addressed by #30759. |
I'm running into this issue. I ran npm 5.3.0
npm 4.6.1
|
I got I'm getting a different error from |
@firelizzard18 I met the same error with the npm@5.3.0. After checking some dependencies, I removed the |
Installed 4.6.1 . Didn't work initially. Took a fresh vscode source and built again and voila ! .. well what do you know !... That worked. |
I got it to work, without changing my global NPM version:
|
Shouldn't take that much back and forth. You could also retry with npm 5.x since #30759 was merged. |
Thanks @firelizzard18 following your procedure I fixed the issue I faced @chrmarti , I pull latest still with 5.X I have no luck |
@Jeyanthinath What are the error messages you get? Does it work with npm 4? |
@chrmarti It works with npm at install stage , but while brining up vscode it failed saying it needs to be compiled with wrong version But when I goes with 5.3 it is not getting installed |
@Jeyanthinath I'm not sure we have looked into npm 5 support in detail. Could you try with npm 4 and first |
@chrmarti After cleaning with |
@joaomoreno @chrmarti |
@marswong do you have a VPN, I mean the reason maybe the GFW when doing the post install. |
@loatheb |
@marswong it sounds like you're running into a different issue. The issue
described by myself and other commenters has nothing to do with ripgrep.
…On Sun, Jul 30, 2017 at 18:57 Mars Wong ***@***.***> wrote:
@loatheb <https://github.com/loatheb>
maybe there is nothing to do with the network. I just don't know why
***@***.*** postinstall just execute node ./dist/postinstall.js,
for there is node dist directory on workspace.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#30775 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AA1pzxkLeKdYMAb1r2JYNqC6BzsiyqWxks5sTTSigaJpZM4OZGor>
.
|
@marswong Can you update to the latest in |
@chrmarti it seems that the latest code works today. thank you :) |
What are the suggested steps to install using NPM@4.61 after installing locally? |
@devonryder You can try the commands in my above comment (#30775 (comment)). |
@chrmarti Same error, my global NPM is 5.3.0. |
@devonryder You could change the last line in |
@chrmarti What should this line look like? I'd be changing the line |
@devonryder I guess so, does that not work? |
Downloaded vscode source . Following their instructions to build , when I ran
Failed at the electron-mksnapshot@1.6.0 install script.
the console also shows the below:
I have installed extract-zip v 1.6.5 , this is mentioned in dependency. No Idea what is going wrong here. I deleted the node_modules and build file and checked again. Does not seem to work. I have asked a question at stackoverflow too.
The text was updated successfully, but these errors were encountered: