-
Notifications
You must be signed in to change notification settings - Fork 5
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
command "gh tr" not working on win7 #8
Comments
Awesome! Thanks for sending those fixes @ramonPires. Could you test and let know me know if it works? |
|
@zenorocha ,still showing this error message: D:\workspaces\Node.js\Node-GH\elixir [master]> gh tr C:\Users\RAMON\AppData\Roaming\npm\gh-tr.CMD:1 (function (exports, require, module, __filename, __dirname) { :: Created by np ^ gh [error] Command not found D:\workspaces\Node.js\Node-GH\elixir [master]> But now the gh-tr.cmd file was generated correctly: :: Created by npm, please don't edit manually. @IF EXIST "%~dp0\node.exe" ( "%~dp0\node.exe" "%~dp0\node_modules\gh-travis\bin\gh-travis.js" %* ) ELSE ( node "%~dp0\node_modules\gh-travis\bin\gh-travis.js" %* ) The version of Node.js in my environment is v0.10.2 and npm is 1.2.15 . |
@henvic Did you had a chance to look at this issue? Thanks! |
Oops, my bad. I haven't. But will look at night. Thank you for remembering me. |
@ramonPires, sorry for the delay. Could you reproduce it on another machine and tell me if the same thing happens? I don't have access to a Windows computer. Do you have any idea about what is going on? I asked a friend who develops also with node.js and has access to one to take a look tomorrow. Let's see if can help us :) |
i'm running node-gh normally,but with gh-travis i got this issue:
Opening this file gh-travis.cmd,the script is different from gh.cmd and others *.cmd files inside of C:\Users\RAMON\AppData\Roaming\npm folder.
gh-tr.cmd:
gh.cmd:
mocha.cmd
Searching, i found a possible solution:
http://stackoverflow.com/questions/10396305/npm-package-bin-script-for-windows
https://github.com/isaacs/npm/issues/2547
The text was updated successfully, but these errors were encountered: