-
Notifications
You must be signed in to change notification settings - Fork 50
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
Speakable fails on Node.js v4.2.1 due to spawh() changes #13
Comments
I am having the same issue. Have you by any chance fixed it? |
@gcrev93 Some people have fix this particular issue, for example https://github.com/ajay-gandhi/node-speakable I still can't get this library to work however, possibly due to the last commit being 2 years ago |
Seems to work once that change is made. Works on node/0.12.7 and node/5.7.0 |
Yeah the fix does stop the error |
It doesn't seem to work in nodejs 7.4.0 |
Running speakable on Node.js v4.2.1 fails with
[TypeError: options argument must be an object]
. This happens when trying to spawn thesox
process (node-speakable.js, line 75) due to the options argument passed being a string.The quick fix for this is to change that line to
var rec = spawn(self.cmd, self.cmdArgs, {stdio:'pipe'});
I haven't tested this change on Node.js v0.12.x though.
The text was updated successfully, but these errors were encountered: