Horton

Re: Cluster.pm psql() undefined $$stderr

Andrew Dunstan <andrew@dunslane.net>

View thread View on postgr.es
From: Andrew Dunstan <andrew@dunslane.net>
To: Oleg Tselebrovskiy <o.tselebrovskiy@postgrespro.ru>
Cc: pgsql-hackers@postgresql.org
Date: 2025-06-27T19:33:20Z
Lists: pgsql-hackers 
On 2025-06-16 Mo 12:23 AM, Oleg Tselebrovskiy wrote:
>> 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 don't think we need your test script, and it looks rather ugly anyway.

Given we've got this far without it being a significant issue, I don't 
think there's much need to backpatch it.


cheers


andrew

--
Andrew Dunstan
EDB: https://www.enterprisedb.com

Commits

  1. Avoid uninitialized value error in TAP tests' Cluster->psql