The Puppet Labs Issue Tracker has Moved: https://tickets.puppetlabs.com
Puppet should use IO#tty? to determine default for colorization
|Assignee:||Nigel Kersten||% Done:|
|Affected Puppet version:||0.24.6||Branch:|
Ticket tracking is now hosted in JIRA: https://tickets.puppetlabs.com
This issue is currently not available for export. If you are experiencing the issue described below, please file a new ticket in JIRA. Once a new ticket has been created, please add a link to it that points back to this Redmine ticket.
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 almost 6 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 almost 6 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 almost 4 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.