Re: psql tab completion versus Debian's libedit

From: Andres Freund <andres(at)anarazel(dot)de>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: pgsql-hackers(at)lists(dot)postgresql(dot)org, pgbf(at)twiska(dot)com
Subject: Re: psql tab completion versus Debian's libedit
Date: 2022-02-03 02:10:29
Message-ID: 20220203021029.aw5ivy7u4us6msfm@alap3.anarazel.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi,

On 2022-02-01 16:30:11 -0500, Tom Lane wrote:
> I chased down the failure that kittiwake has been showing since
> 02b8048ba [1].

I just rebased my meson branch across the commit d33a81203e9. And on freebsd
the meson based build failed in the expanded tests, while autoconf succeeded.

The failure is:

not ok 22 - complete schema-qualified name
# Failed test 'complete schema-qualified name'
# at /tmp/cirrus-ci-build/src/bin/psql/t/010_tab_completion.pl line 236.
# Actual output was "tab "
# Did not match "(?^:tab1 )"

I think this is caused by the feature flag detection being broken in the meson
branch - unrelated to your commit - ending up with falsely believing that none
of the rl_* variables exist (below for more on that aspect).

Do we care that the tests would fail when using a readline without any of the
rl_* variables? I don't know if those even exist.

The reason for meson not detecting the variables is either an "andres" or
freebsd / readline issue. The tests fail with:

/usr/local/include/readline/rltypedefs.h:71:36: error: unknown type name 'FILE'
typedef int rl_getc_func_t PARAMS((FILE *));
^
apparently the readline header on freebsd somehow has a dependency on stdio.h
being included.

Looks like it's easy enough to work around. My local copy of readline.h (8.1
on debian sid) has an explicit stdio.h include, but it looks like that's a
debian addition...

Greetings,

Andres Freund

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message David G. Johnston 2022-02-03 02:21:19 Re: Unclear problem reports
Previous Message Andres Freund 2022-02-03 01:38:26 Re: ci/cfbot: run windows tests under a timeout