| From: | Michail Nikolaev <michail(dot)nikolaev(at)gmail(dot)com> |
|---|---|
| To: | Juan José Santamaría Flecha <juanjo(dot)santamaria(at)gmail(dot)com> |
| Cc: | Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>, "Tang, Haiying" <tanghy(dot)fnst(at)cn(dot)fujitsu(dot)com>, Thomas Munro <thomas(dot)munro(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: BUG #16108: Colorization to the output of command-line has unproperly behaviors at Windows platform |
| Date: | 2020-02-22 20:08:45 |
| Message-ID: | CANtu0oic8bkGZ6uSA6NPp=dKCwpBd22yk2tJmUKCX5PVVAzMDA@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-bugs pgsql-hackers |
Hello.
> The patch about making color by default [1] introduces the function terminal_supports_color(), that I think is relevant for this issue. Please find attached a new version based on that idea.
I am not sure it is good idea to mix both patches because it adds some
confusion and makes it harder to merge each.
Maybe is it better to update current patch the way to reuse some
function later in [1]?
Also, regarding comment
> It is disabled by default, so it must be enabled to use color outpout.
It is not true for new terminal, for example. Maybe it is better to
rephrase it to something like: "Check if TV100 support if enabled and
attempt to enable if not".
| From | Date | Subject | |
|---|---|---|---|
| Next Message | PG Bug reporting form | 2020-02-23 06:05:48 | BUG #16273: InitDB Memory leak |
| Previous Message | Tom Lane | 2020-02-21 23:33:31 | Re: BUG #16272: Index expression can refer to wrong attributes if index is created via CREATE TABLE LIKE |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Daniel Gustafsson | 2020-02-22 21:22:27 | Re: Ought to use heap_multi_insert() for pg_attribute/depend insertions? |
| Previous Message | Jeff Davis | 2020-02-22 19:59:59 | Re: Memory-Bounded Hash Aggregation |