-
Notifications
You must be signed in to change notification settings - Fork 53
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
Use std streams instead of Tempfile? #43
Comments
Yeah, this seems like a reasonable way to go. Being able to pipe the output somewhere for inspection might be handy, but the test suite should also raise all of that in a reasonable fashion anyway. |
I was thinking about this more in terms of perf. gains, not a user-facing feature. |
I've been playing with it a bit, it is possible to do that even without require 'open3'
command = "java -jar lib/closure-compiler-20170626.jar"
stdin, stdout, stderr, wait_thr = Open3::popen3(command)
stdin.write("const foo = () => { return { foo: 321, bar: ['a', 'b', 'c'] } };")
stdin.close
stdout.read gives me
|
also, it seems like |
Basically it should be possible to just write something into stdin via echo:
The text was updated successfully, but these errors were encountered: