News for hackers. Stuff that matters.
Re:Show HN: Grep with colours written in Go by wruza on Wednesday June 06, 2018 @12:32AM (#17244359)
What if someone needs colors, but not colors that util maker used? What if two or more tools make different use of colors? With terminal aware of that problem you can remap or erase specific colors, but maybe it’s wiser to not create a problem at all, since just making text bold may be enough for practical needs.

Edit: missing not

Re:Show HN: Grep with colours written in Go by giancarlostoro on Wednesday June 06, 2018 @05:45AM (#17245648)
Sounds like a better standard is being able to define color schemas for CLI utilities. Would be pretty awesome if this were possible, though many utilities vary in what type of output is colored a certain way sadly.

Another workaround is to merely pipe the output to a file in certain cases.

Re:Show HN: Grep with colours written in Go by megous on Wednesday June 06, 2018 @08:59AM (#17246706)
> Another workaround is to merely pipe the output to a file in certain cases.

If that works for a certain tool, it's author really has no excuse to not implement NO_COLOR env as well.

Re:Show HN: Grep with colours written in Go by mbrumlow on Wednesday June 06, 2018 @09:22AM (#17246907)
Piping the output does not guarantee the removal of the color codes. This is yet another standard some programs have chosen to implement.

The program checks if the output is a tty, if it is not a tty it will drop color information as it is either a pipe or a file. The escape codes would look ugly in a file and could get in your way with some text processing programs.

Even if the program did drop color when not sending output to a tty it still is a crappy way to remove color as the pipe would need another program as a receiver such as pager.

Re:Show HN: Grep with colours written in Go by burntsushi on Wednesday June 06, 2018 @09:34AM (#17247006)
> Even if the program did drop color when not sending output to a tty it still is a crappy way to remove color as the pipe would need another program as a receiver such as pager.

You don't need a pager. cat works. Compare

    grep --color=auto hacker /usr/share/dict/words
with

    grep --color=auto hacker /usr/share/dict/words | cat
The same applies to GNU ls, which actually changes more than just colors depending on whether it's printing to a tty or not.

Programs that emit colors by default when not sending output to a tty are buggy.

A wild card in this mix is Windows. On UNIX-like systems, detecting a tty is simple. On Windows, it is... not so simple. But the OP's tool, blush, doesn't appear to support Windows at all anyway. (Which is totally cool. I very much understand why you might not.)

Re:Show HN: Grep with colours written in Go by mbrumlow on Wednesday June 06, 2018 @02:29PM (#17249709)
Yeah, I did not mean to imply you needed a pager, just something to receive the output, a pager is just one choice.

I guess the program implemented the tty method you could always alias your stuff to something like blush="/bin/blush | cat" if you decided you did not like color.

While a part of me says piping programs to get the result you want is the *nix way. Another part of me feels like it would just be a dirty hack to avoid color.

Re:Show HN: Grep with colours written in Go by tom_ on Wednesday June 06, 2018 @10:11AM (#17247263)
I don't think it matters so much on Windows, because the colour information is sent out of band. When you call SetConsoleTextAttribute on the STD_OUTPUT_HANDLE, and that handle is a file, the call simply fails.
Re:Show HN: Grep with colours written in Go by burntsushi on Wednesday June 06, 2018 @10:34AM (#17247452)
I am not speaking without experience. If you support Windows, you probably need to support non-console environments like cygwin and msys. Moreover, Windows 10 has opt-in support for ANSI style coloring. At some point, coloring via the console APIs will probably stop being used.
Re:Show HN: Grep with colours written in Go by megous on Friday June 08, 2018 @02:58AM (#17263220)
That's the point. If the program is already doing isatty() and thus has code for color/no-color output already, just adding getenv("NO_COLOR") is a no-brainer.