Re: libcurl in libpq.pc

From: Jacob Champion <jacob(dot)champion(at)enterprisedb(dot)com>
To: Christoph Berg <myon(at)debian(dot)org>
Cc: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Wolfgang Walther <walther(at)technowledgy(dot)de>, Timo Röhling <roehling(at)debian(dot)org>
Subject: Re: libcurl in libpq.pc
Date: 2025-09-30 23:15:28
Message-ID: CAOYmi+mMbenOeXrQQnuudU77dJW1QQFZVrNcdAy4ksbRx8ap=Q@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, Sep 23, 2025 at 4:30 PM Jacob Champion
<jacob(dot)champion(at)enterprisedb(dot)com> wrote:
> On my machine, pkgconf appears to recognize Requires.internal, but
> unfortunately it doesn't seem to ignore the internal requirements for
> --cflags. So... I don't get it.

To avoid having this hang in the air forever: there appears to be
considerable confusion and/or disagreement [1] about the meaning of an
internal dependency. Probably not too surprising, since the proposal I
linked above was implemented in pkgconf before the final patchset was
proposed to pkg-config. So it just kind of does whatever it does
today, I guess. (This behavior is problematic for Meson as well [2].)

The original pkg-config maintainers in that freedesktop thread appear
to have responded to the Debian use case (where you want to minimize
the number of needed developer dependencies) with EDONTCARE. Not much
I can do about that. I think that moving this forward is going to need
multiple interested communities politely lobbying pkgconf to change
behavior.

I'm not currently planning to change the .pc file for 18.1, since
there's a way forward for Debian (albeit one we really don't like),
and the current behavior was crafted to meet a requested use case.
IMO, changes we make here should not regress what was tested in [3]
unless we really cannot bear the cost. We need a working
Requires.internal to make everyone happy.

Thanks,
--Jacob

[1] https://github.com/pkgconf/pkgconf/issues/318
[2] https://github.com/mesonbuild/meson/pull/12227
[3] https://postgr.es/m/9badbeeb-a432-48d4-8710-c8254a54d428%40technowledgy.de

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Michael Paquier 2025-09-30 23:37:40 Re: [PATCH] Add tests for Bitmapset
Previous Message Michael Paquier 2025-09-30 23:05:16 Re: relfilenode statistics