Re: [PATCH] Fix missing comma in Requires.private with a Make macro - Mailing list pgsql-hackers

From Fabrízio de Royes Mello
Subject Re: [PATCH] Fix missing comma in Requires.private with a Make macro
Date
Msg-id CAFcNs+pysLX5TciGuJWT9449qUthk=Ys7MrXAGkkNNA9=_GMpA@mail.gmail.com
Whole thread Raw
In response to [PATCH] Fix missing comma in Requires.private with a Make macro  (Jacob Champion <jacob.champion@enterprisedb.com>)
List pgsql-hackers


On Wed, May 7, 2025 at 2:17 PM Jacob Champion <jacob.champion@enterprisedb.com> wrote:
Hello!

Wolfgang reported over in [1] that I've missed a comma when appending
to the PKG_CONFIG_REQUIRES_PRIVATE list, making libpq.pc look ugly:

    Requires.private: libssl, libcrypto libcurl

pkg-config itself appears to be papering over my mistake (a quick code
inspection suggests it treats commas as equivalent to whitespace?) but
here's a patch to fix it. I went with a macro inspired by
Makefile.global's add_to_path, to try to ease this for whoever comes
next.


LGTM

 
Tested with GNU Make 3.81 (the compilation of which was slightly
painful; does anyone want to debate pulling that minimum version up
sometime soon?) and 4.3.


Not sure if all animals have a minimum 4.3 make version. Do you?

--
Fabrízio de Royes Mello

pgsql-hackers by date:

Previous
From: Nathan Bossart
Date:
Subject: Re: add --sequence-data to pg_dumpall
Next
From: Peter Geoghegan
Date:
Subject: Re: Adding skip scan (including MDAM style range skip scan) to nbtree