| From: | Oleg Tselebrovskiy <o(dot)tselebrovskiy(at)postgrespro(dot)ru> |
|---|---|
| To: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
| Cc: | pgsql-hackers(at)postgresql(dot)org |
| Subject: | Re: Cluster.pm psql() undefined $$stderr |
| Date: | 2025-06-16 04:23:20 |
| Message-ID: | 378eac5de4b8ecb5be7bcdf2db9d2c4d@postgrespro.ru |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
> I think we need to do something slightly earlier than that. Attached is
> what I propose.
I agree, my patch could miss one case with undefined $$stderr.
Fixed my patch with your suggestion
> I think the short answer is no, we've already hijacked STDOUT/STDERR in
> Utils.pm to point to the log file, and you shouldn't mess with them. I
> don't think you're using SimpleTee correctly anyway, but it's really
> not
> meant for general use, only for the use in Utils.pm.
Thanks for the info!
Regards, Oleg Tselebrovskiy
| Attachment | Content-Type | Size |
|---|---|---|
| 0001_fix_undef_stderr_v2.patch | text/x-diff | 1.9 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | jian he | 2025-06-16 05:52:45 | Re: Add SPLIT PARTITION/MERGE PARTITIONS commands |
| Previous Message | Matt Smith (matts3) | 2025-06-16 04:10:40 | [PATCH] Add an ldflags_sl meson build option |