From: | Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com> |
---|---|
To: | Christoph Berg <myon(at)debian(dot)org> |
Cc: | Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: PGCOLOR? (Re: pgsql: Unified logging system for command-line programs) |
Date: | 2019-06-06 09:08:25 |
Message-ID: | CAD21AoDoAZBTf=29PT-cUtFvxYEXi6nAPSzAjCc94q5UXH_6Ug@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
On Tue, Apr 9, 2019 at 9:01 PM Christoph Berg <myon(at)debian(dot)org> wrote:
>
> Re: Peter Eisentraut 2019-04-09 <d483cdb6-db98-9b2f-7f2b-eed0f4bd975d(at)2ndquadrant(dot)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
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2019-06-06 13:11:10 | pgsql: Add forgotten translatable string in pg_checksums.c |
Previous Message | Peter Eisentraut | 2019-06-06 08:28:36 | pgsql: pg_waldump: Fix invalid option handling |
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2019-06-06 09:14:45 | Re: coverage additions |
Previous Message | Michael Paquier | 2019-06-06 09:01:21 | Re: Why does pg_checksums -r not have a long option? |