node-maven doesn't work on Windows

headcr4sh / node-maven

Maven for Node.js

Apache License 2.0

18 stars 14 forks source link

node-maven doesn't work on Windows #3

Closed headcr4sh closed 9 years ago

headcr4sh commented 9 years ago

We use "child_process.spawn" to fork a sub-process to execute Maven. mvn.bat on Windows can't be invoked like that... see https://github.com/joyent/node/issues/2318 for further details.

Possible solutions:

Use "cmd /c /s" when using Windows
Use cross-spawn insteads of child_process.spawn