pgsql: Add ./configure check for "lz4" command

From: Michael Paquier <michael(at)paquier(dot)xyz>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Add ./configure check for "lz4" command
Date: 2022-02-14 01:42:06
Message-ID: E1nJQNS-0001cC-FB@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Add ./configure check for "lz4" command

Some environments may compile with --with-lz4 while the command "lz4"
goes missing, causing two failures in the TAP tests of pg_verifybackup
(008_untar.pl and 010_client_untar.pl) as the code assumed that the
command always existed with a hardcoded value in src/Makefile.global.
Rather than this method, this adds a ./configure check based on
PGAC_PATH_PROGS() to find automatically the command and get an absolute
path to it.

Both tests need to be adjusted for the case where the command does not
exist, actually, as Makefile.global would set now LZ4 to an empty value
in this case. The TAP tests of pg_receivewal already do that.

Per report from buildfarm member copperhead, as an effect of dab2984.
The origin of the failure is actually babbbb5 that did not centralize
the check for the existence of a "lz4" command at ./configure to shave a
few cycles. Note that one just needs to tweak an environment to move
"lz4" out of the way to reproduce the problem, which is what I did to
test this change.

Per discussion with Robert Haas, Tom Lane, Andres Freund and myself.

Discussion: https://postgr.es/m/Ygc51WVAFGocSu4h@paquier.xyz

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/1d084fba1bd144538dfea22c322f1c148c821148

Modified Files
--------------
configure | 55 +++++++++++++++++++++++++++
configure.ac | 1 +
src/Makefile.global.in | 2 +-
src/bin/pg_verifybackup/t/008_untar.pl | 5 ++-
src/bin/pg_verifybackup/t/010_client_untar.pl | 5 ++-
5 files changed, 63 insertions(+), 5 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Michael Paquier 2022-02-14 01:53:04 Re: pgsql: Add suport for server-side LZ4 base backup compression.
Previous Message Alexander Korotkov 2022-02-14 01:17:59 pgsql: Fix memory leak in IndexScan node with reordering