On Tue, Apr 9, 2019 at 9:01 PM Christoph Berg <m...@debian.org> wrote: > > Re: Peter Eisentraut 2019-04-09 > <d483cdb6-db98-9b2f-7f2b-eed0f4bd9...@2ndquadrant.com> > > I'm okay with changing it. As you indicate, I chose the name so that it > > doesn't look like a libpq variable. There are some other PG_ variables > > throughout the code, but those appear to be mostly for internal use. > > Also, there is GCC_COLORS, LS_COLORS, etc. But perhaps this wisdom will > > be lost on users who just read the man page and get confused. ;-) > > Do we need two variables to control this? I was only looking at > PG_COLOR, and noticed PG_COLORS only later. Keeping PG_COLORS aligned > with {GCC,LS}_COLORS makes sense. How about removing PG_COLOR, and > making "auto" the default? (Maybe we could still support "PG_COLORS=off") >
I think the if we keep two variables user can set the same value to both GCC_COLORS and PG_COLORS. Rather I think it's a problem that there is no documentation of PG_COLORS. Thoughts? Regards, -- Masahiko Sawada NIPPON TELEGRAPH AND TELEPHONE CORPORATION NTT Open Source Software Center