Decode also err output of external tool subprocess

riptano / ccm

A script to easily create and destroy an Apache Cassandra cluster on localhost

Apache License 2.0

1.22k stars 303 forks source link

Closed spodkowinski closed 6 years ago

spodkowinski commented 6 years ago

Byte values for err channel should be decoded as well, see 0a5824508

cc @aweisberg

aweisberg commented 6 years ago

+1 in for a penny in for a pound. So who is going to update the dtests instances where we decode?

ptnapoleon commented 6 years ago

Is this safe to merge or do you want me to wait for some corresponding dtest changes?

aweisberg commented 6 years ago

it's not safe, but a lot of these instances are already broken on decoding stdout anyways. So... umm... sure merge it?