Re: libedit history seems to be misbehaving / broken

From: Bruce Momjian <bruce(at)momjian(dot)us>
To: Tomas Vondra <tomas(at)vondra(dot)me>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: libedit history seems to be misbehaving / broken
Date: 2024-11-01 00:50:59
Message-ID: ZyQl8036DsVWUWGD@momjian.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Sun, Oct 27, 2024 at 09:07:44PM +0100, Tomas Vondra wrote:
> On 10/27/24 20:03, Tom Lane wrote:
> > FWIW, I don't observe any particular misbehavior with the very ancient
> > libedit that macOS ships. On Fedora 39, I notice something related to
> > what you say: it seems like the "\q" ending a session gets repeated
> > into .psql_history by the next session. I'm surprised that it's not
> > exactly the same as your results though, because it seems to be the
> > same source version:
> >
> > $ rpm -q libedit
> > libedit-3.1-53.20240808cvs.fc39.x86_64
> >
>
> That's probably because I usually terminate psql by Ctrl-D, not by
> typing "\q". But yeah, if I use "\q" it gets added twice.
>
> > Didn't try the too-many-lines behavior, but it looks like that
> > is implemented totally by history_truncate_file(), so if that's
> > busted it's surely their fault.
> >
>
> Sounds likely. What surprises me a bit, I haven't found any reports of
> particularly similar bugs ... I'd have expected other people to hit this
> too, but who knows.

I wonder if our previous libedit workarounds aren't needed anymore and
are causing the bugs.

--
Bruce Momjian <bruce(at)momjian(dot)us> https://momjian.us
EDB https://enterprisedb.com

When a patient asks the doctor, "Am I going to die?", he means
"Am I going to die soon?"

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Bruce Momjian 2024-11-01 01:10:01 Re: Remove useless casts to (void *)
Previous Message Michael Paquier 2024-11-01 00:50:02 Re: Making error message more user-friendly with spaces in a URI