Re: IPC::Run::time[r|out] vs our TAP tests - Mailing list pgsql-hackers

From Dagfinn Ilmari Mannsåker
Subject Re: IPC::Run::time[r|out] vs our TAP tests
Date
Msg-id 878r1sj7bz.fsf@wibble.ilmari.org
Whole thread Raw
In response to Re: IPC::Run::time[r|out] vs our TAP tests  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: IPC::Run::time[r|out] vs our TAP tests
List pgsql-hackers
Tom Lane <tgl@sss.pgh.pa.us> writes:

> Erik Wienhold <ewie@ewie.name> writes:
>> Libedit 20191025-3.1 is the first version where ":{?VERB<tab>" works as
>> expected.  The previous release 20190324-3.1 still produces the escaped
>> output that Michael found.  That narrows down the changes to everything
>> between [1] (changed on 2019-03-24 but not included in 20190324-3.1) and
>> [2] (both inclusive).
>
> Hm.  I just installed NetBSD 8.2 in a VM, and it passes this test:
>
> # +++ tap install-check in src/bin/psql +++
> t/001_basic.pl ........... ok     
> t/010_tab_completion.pl .. ok    
> t/020_cancel.pl .......... ok   
> All tests successful.
>
> So it seems like the bug does not exist in any currently-supported
> NetBSD release.  Debian has been known to ship obsolete libedit
> versions, though.

Both the current (bokworm/12) and previous (bullseye/11) versions of
Debian have new enough libedits to not be affected by this bug:

libedit    | 3.1-20181209-1 | oldoldstable   | source
libedit    | 3.1-20191231-2 | oldstable      | source
libedit    | 3.1-20221030-2 | stable         | source
libedit    | 3.1-20230828-1 | testing        | source
libedit    | 3.1-20230828-1 | unstable       | source
libedit    | 3.1-20230828-1 | unstable-debug | source

But in bullseye they decided that OpenSSL is a system library as far as
the GPL is concerned, so are linking directly to readline.

And even before then their psql wrapper would LD_PRELOAD readline
instead if installed, so approximately nobody actually ever used psql
with libedit on Debian.

>             regards, tom lane

- ilmari



pgsql-hackers by date:

Previous
From: Nathan Bossart
Date:
Subject: Re: Allow non-superuser to cancel superuser tasks.
Next
From: "Daniel Verite"
Date:
Subject: Re: Fixing backslash dot for COPY FROM...CSV