Puppet should use IO#tty? to determine default for colorization
|Assignee:||Nigel Kersten||% Done:|
|Affected Puppet version:||0.24.6||Branch:|
Puppet should use the IO#tty? to set the default value for colorized output.
I’d be glad to implement this, if it’s seen as worthwhile.
#1 Updated by Luke Kanies about 5 years ago
- Status changed from Unreviewed to Needs Decision
I don’t really like this idea (as mentioned on IRC), because there are plenty of times where I don’t have a tty but still want color —
less -X -r is great at showing color but doesn’t have a tty.
The only way I’d even consider this is if it allowed an override so that you could have color even without a tty, but I’d still prefer a default color — it’s 2008, people, your apps can almost definitely handle color at this point.
#2 Updated by Thomas Bellman about 5 years ago
It may be 2008, but it is still a pain to write regexps that handle strange escape sequences when you postprocess the output, with for example grep or awk.
As for an override, I would assume that —color=true would provide that.
(Personally, I dislike output that looks like a christmas tree even for interactive use. It just gives me headaches.)
#4 Updated by Nigel Kersten about 3 years ago
- Status changed from Accepted to Closed
- Assignee set to Nigel Kersten
Clayton, still interested in fixing this? Please feel free to reopen if so.
As per the below thread, we’re more aggressively closing tickets whose state is unsure, particularly old tickets with little to no inactivity in a long time.
You are free to reopen them.