Re: 001_password.pl fails with --without-readline

From: Soumya S Murali <soumyamurali(dot)work(at)gmail(dot)com>
To: Oleg Tselebrovskiy <o(dot)tselebrovskiy(at)postgrespro(dot)ru>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: 001_password.pl fails with --without-readline
Date: 2026-01-16 10:29:19
Message-ID: CAMtXxw9PVPyiHY6y4CxS4L92wDqpOp6DsLpC4sAsqT5WO963xA@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi,

On Fri, Jan 16, 2026 at 12:53 PM Oleg Tselebrovskiy
<o(dot)tselebrovskiy(at)postgrespro(dot)ru> wrote:
>
> Thanks for your answer!
>
> Personally, I can’t think of any more test scenarios that could be
> tested here (maybe some versions of readline library on some exotic
> OS could break, but that seems a little far fetched), so, if you see
> no further problems with the patches, you could change
> the commitfest entry status to "Ready for Committer", so we could get
> another opinion on patches and, if possible, get them committed.
>

Thank you for the clarification.
The patches seem correct based on the testing I have done so far and
for me I did not observe any further issues in both readline and
non-readline configurations. Since I am one of the reviewers and not
the only one. I consider my review as only one data point and I would
feel more comfortable if the patches also receive input from other
reviewers who may have more experience with this part of the code or
different testing environments.
I am happy to add my test results and observations to the CommitFest
entry to assist further review. Once others have had a chance to look
and review, it would likely be more appropriate to move the entry
forward.

Regards,
Soumya

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Tatsuro Yamada 2026-01-16 10:37:00 Re: [PATCH] psql: add \dcs to list all constraints
Previous Message David Rowley 2026-01-16 10:09:58 Re: [PATCH] remove incorrect comment in pg_resetwal.c