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

open is not recognized as an internal or external command #13

Closed
sreich opened this issue Jan 29, 2016 · 6 comments
Closed

open is not recognized as an internal or external command #13

sreich opened this issue Jan 29, 2016 · 6 comments

Comments

@sreich
Copy link

sreich commented Jan 29, 2016

└─[$] diff2html.cmd -i file -- difftest.diff [14:09:15] 'open' is not recognized as an internal or external command,
operable program or batch file.
child_process.js:507
throw err;
^

Error: Command failed: open /tmp/diff.html
'open' is not recognized as an internal or external command,
operable program or batch file.

at checkExecSyncError (child_process.js:464:13)
at Object.execSync (child_process.js:504:13)
at Utils.runCmd (C:\Users\sreich\AppData\Roaming\npm\node_modules\diff2html-cli\src\utils.js:58:25)
at Diff2HtmlInterface.preview (C:\Users\sreich\AppData\Roaming\npm\node_modules\diff2html-cli\src\cli.js:106:11)
at onOutput (C:\Users\sreich\AppData\Roaming\npm\node_modules\diff2html-cli\src\main.js:157:9)
at Diff2HtmlInterface.getOutput (C:\Users\sreich\AppData\Roaming\npm\node_modules\diff2html-cli\src\cli.js:77:14)
at onInput (C:\Users\sreich\AppData\Roaming\npm\node_modules\diff2html-cli\src\main.js:145:7)
at FSReqWrap.readFileAfterClose [as oncomplete] (fs.js:380:3)

ideas on what's wrong?

@rtfpessoa
Copy link
Owner

I have some OS specific stuff in the code.
I will try to remove it. But I still have no way to test it.
If you can do some testing after the fix I would appreciate.

@sreich
Copy link
Author

sreich commented Jan 29, 2016

sounds good. how will i update to your changes? i installed using npm.

java guy, never used node

@rtfpessoa
Copy link
Owner

As soon as I release something you just will have to do npm update -g diff2html-cli.
On Fri, Jan 29, 2016 at 9:35 PM Shaun Reich notifications@github.com
wrote:

sounds good. how will i update to your changes? i installed using npm.

java guy, never used node


Reply to this email directly or view it on GitHub
#13 (comment)
.

@rtfpessoa
Copy link
Owner

@sreich can you update with npm install -g diff2html-cli@1.4.0-windows1
Since it is a pre-release I am not sure the update will take you there.

Then report back it is fixed. If can can check other features and report here, it would be really appreciated.

@sreich
Copy link
Author

sreich commented Feb 1, 2016

awesome! works perfectly now.

@rtfpessoa
Copy link
Owner

Closing this then 😄

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

No branches or pull requests

2 participants