Open majelbstoat opened 2 years ago
Yes, it presumes output is lines of strings. Internally, it does just buffer data, but then calls like https://github.com/go-cmd/cmd/blob/v1.4.0/cmd.go#L334 presume string lines. I'm not sure it's feasible to adapt to binary output because the string line presumption is baked into the design.
Thanks for the clarification. Instead of a replacement, would the addition of a status.StdoutReader
which just returned an io.Reader over the internal data, and/or something similar on an OutputStream be feasible, or are there internals that would break if the data was accessed in that way?
I'm using exec.Cmd() to run graphics magick to convert an image, reading from stdin and writing to stdout.
I've been using go-cmd for all my other commands, but for this use case, it doesn't seem possible, because go-cmd treats all its outputs as strings. Is that correct? I'd love to be doing other work while I wait for the conversion to finish.