I agree with the patch. Works in my OSes
On 7/10/25 2:54 AM, Jacob Champion <jacob.champion@enterprisedb.com> wrote:
> On Wed, Jul 9, 2025 at 12:42 PM Tom Lane <tgl@sss.pgh.pa.us> wrote:
> > Nah, let's keep them. We do document for at least some libraries
> > how to manually specify the include and link options without
> > depending on pkg-config. If someone tries that with libcurl,
> > it'd be good to have sanity checks on the results.
>
> Sounds good, thanks for the review!
>
> On Wed, Jul 9, 2025 at 11:39 AM Jacob Champion
> <jacob.champion@enterprisedb.com> wrote:
> > Here is a draft patch for Ivan's reported issue; I still need to put
> > it through its paces with some more unusual setups, but I want to get
> > cfbot on it.
>
> On HEAD, Rocky 9 fails to build with a custom Curl PKG_CONFIG_PATH and
> no libcurl-devel installed. With this patch, that build now succeeds,
> and it still succeeds after libcurl-devel is reinstalled, with the
> compiler tests continuing to use the custom libcurl and not the
> system's.
>
> So I'll give Ivan a little time in case he'd like to test/review
> again, but otherwise I plan to push it this week.
>
> Thanks,
> --Jacob
>