-
Notifications
You must be signed in to change notification settings - Fork 11.8k
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
does not install on my windows 10 #96
Comments
@rfikki thanks for reporting. Seems like a problem with how windows handles paths. Can you try changing this line: https://github.com/OpenZeppelin/zeppelin-solidity/blob/master/package.json#L12
Or however windows recognizes the path? (sorry, I don't have much experience with windows development) |
@rfikki any update on this? Would like to help |
@maraoz I added that as well as tried added exec in front of the path: Neither worked unfortunately. I've been researching other ways of moving files post-npm-install to try and create a helpful pull request, but am not having much luck yet. Edit: I should note I'm executing this with PowerShell on Windows 10. |
You can't run bash (Unix/Linux) scripts on windows by default if you're not running something like wsl (Windows Subsystem for Linux) or GIT Bash. I think using os-specific scripts/commands and copying stuff to some folder outside of node_modules is a bad Idea, you should convert your bash scripts to js files and add the moving of folders as part of a global commad, for example I'm currently running on windows so I could do a pr if you want. As temp workaround you can also use |
@explodingcamera Thanks for the info. |
To get rid of the errors we were having before we started powershell as admin and run the following command line:
The result was:
To get rid of the warnings we added a package.json file with the following content to WINDOWS\System32:
Finally the open zeppelin package installed without any errors or warnings. To be tested now though.
|
After issueing the
|
This is a known issue with IPFS, sorry: #209 |
this issue could be fixed when #253 is deployed to npm, correct? |
This is obsolete as |
$ npm i zeppelin-solidity
'scripts' is not recognized as an internal or external command,
operable program or batch file.
npm WARN enoent ENOENT: no such file or directory, open 'C:\projects\truffle-dap ps\coinmaker\package.json'
npm WARN coinmaker No description
npm WARN coinmaker No repository field.
npm WARN coinmaker No README data
npm WARN coinmaker No license field.
npm ERR! Windows_NT 10.0.14393
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Users\Rocky\AppData\ Roaming\npm\node_modules\npm\bin\npm-cli.js" "i" "zeppelin-solidity"
npm ERR! node v6.7.0
npm ERR! npm v3.10.8
npm ERR! code ELIFECYCLE
npm ERR! zeppelin-solidity@1.0.0 install:
scripts/install.sh
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the zeppelin-solidity@1.0.0 install script 'scripts/install.s h'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the zeppelin-solidity pac kage,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR! scripts/install.sh
npm ERR! You can get information on how to open an issue for this project with:
npm ERR! npm bugs zeppelin-solidity
npm ERR! Or if that isn't available, you can get their info via:
npm ERR! npm owner ls zeppelin-solidity
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR! C:\projects\truffle-dapps\coinmaker\npm-debug.log
Rocky@ASUS MINGW64 /c/projects/truffle-dapps/coinmaker
$ node --version
v6.7.0
Rocky@ASUS MINGW64 /c/projects/truffle-dapps/coinmaker
$ npm --version
3.10.8
The text was updated successfully, but these errors were encountered: