Skip to content
This repository has been archived by the owner on Jan 4, 2020. It is now read-only.

Proposal: continuous text output from script #134

Open
rmirabelli opened this issue Sep 29, 2017 · 3 comments
Open

Proposal: continuous text output from script #134

rmirabelli opened this issue Sep 29, 2017 · 3 comments

Comments

@rmirabelli
Copy link

Currently Marathon prints all of the script's output at the end of the script running.

I'd love to see Marathon instead use a Pipe() along with waitForDataInBackgroundAndNotify(), rather than use a print statement at script completion. This could probably be done by creating a different printer that is passed along to shellOut.

I'm happy to work on this request myself, but would appreciate input from others as to potential impacts, especially on Linux.

@jarod2d
Copy link

jarod2d commented Oct 3, 2017

Yes please! I was just struggling for a while to figure out why I couldn't get output in real time. In the mean time, is there any workaround without ditching Marathon?

@rmirabelli
Copy link
Author

I don't know of a workaround, but I'm going to start working on this over the next day or two, and it shouldn't be too challenging.

@JohnSundell
Copy link
Owner

I'd really love to see this happen 👍 It will require adding async/continuous output to ShellOut, here's an issue for that: JohnSundell/ShellOut#11

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

No branches or pull requests

3 participants