Re: PG versus libxml2 2.12.x - Mailing list pgsql-hackers

From Andrew Dunstan
Subject Re: PG versus libxml2 2.12.x
Date
Msg-id 9f835a7c-87d8-9fd8-7c21-71c938f346f2@dunslane.net
Whole thread Raw
In response to PG versus libxml2 2.12.x  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
On 2024-01-27 Sa 14:04, Tom Lane wrote:
> Buildfarm member caiman has been failing build for a couple weeks now.
> The reason turns out to be that recent libxml2 has decided to throw
> a "const" into the signature required for custom error handlers.
> (API compatibility?  What's that?)
>
> I don't mind adopting the "const" --- it's a good idea in isolation.
> The trouble is in fixing our code to work with both old and new
> libxml2 versions.  We could thrash around with a configure test or
> something, but I think the most expedient answer is just to insert
> some explicit casts, as shown in the attached.  It's possible though
> that some compilers will throw a cast-away-const warning.  I'm
> not seeing any, but ...
>
> Also, I'm seeing a deprecation warning in contrib/xml2/xpath.c
> for
>
>     xmlLoadExtDtdDefaultValue = 1;
>
> I'm not sure why that's still there, given that we disabled external
> DTD access ages ago.  I propose we just remove it.
>
> In short, I suggest the attached.
>
>             



Looks reasonable.


cheers


andrew

--
Andrew Dunstan
EDB: https://www.enterprisedb.com




pgsql-hackers by date:

Previous
From: vignesh C
Date:
Subject: Re: Race condition in FetchTableStates() breaks synchronization of subscription tables
Next
From: Kurlaev Jaroslav
Date:
Subject: RE: Finding every use of a built-in function