plasma-umass / doppio

Breaks the browser language barrier (includes a plugin-free JVM).
http://plasma-umass.github.io/doppio-demo
MIT License
2.16k stars 175 forks source link

Separate stdout from stderr. #203

Closed int3 closed 10 years ago

int3 commented 11 years ago

This is useful mostly for the node frontend. It will prevent us from dumping both error messages and actual output into the tools/preload file. (Also see #202.)

jvilk commented 11 years ago

Agreed. In the browser, we can simply pass in the stdout function for stderr to keep the same browser behavior.

jvilk commented 10 years ago

This should be a part of our node emulation; the browser frontend would register a stdout/stderr callback with the node emulation layer.

I might hack it into BrowserFS, but it's somewhat outside the scope of that library.

perimosocordiae commented 10 years ago

I agree that it's not a good fit for BFS.

jvilk commented 10 years ago

I'll eventually need to figure out a scheme for neatly combining node emulation libraries for different tasks (e.g. file system, sockets, etc).

jvilk commented 10 years ago

Actually, it kindof does make sense -- stdout is a Writable Stream, which I'll need to add to BFS eventually.

jvilk commented 10 years ago

BrowserFS 0.3.2 now contains stdout/stderr/stdin emulation that we can take advantage of. No need to have a hack that passes in callbacks to the JVM at runtime; configure the environment and be done with it.

perimosocordiae commented 10 years ago

Marking this as blocked on the upgrade to BFS 0.3.2

jvilk commented 10 years ago

Upgrade complete. We can begin transitioning code to use process.sdout/stderr/stdin.

perimosocordiae commented 10 years ago

Does this mean we can close #274?

jvilk commented 10 years ago

No. Read the last comment there -- we have failing tests.

We're blocked again, btw. This time on this BrowserFS bug (whoops): https://github.com/jvilk/BrowserFS/issues/82

jvilk commented 10 years ago

I just merged in our stdout/stderr/stdin branch to master, so this is fixed.