Created attachment 206263 [details] pgsql96-contrib XML The attached patch adds an XML option rather than forcing libxslt/libxml2 in environments that don't need/want them. I don't have any personal interest in the older pgsql versions but I see no reason they wouldn't apply there too.
Created attachment 206265 [details] Add pg11 This works with pgsql1 too.
There's no specific need to update the patch here to reflect it, but consider this specific to postgresql12-contrib (I don't want to mess with 9.6 or 11 while the default version is being changed). If this patch times out next week, I'll be committing it ony to 12-contrib.
Oh, yes please! And for postgresql10 too? Thanks.
Hi Loïc, At this point my intention is only to apply it to pgsqll2. There's work underway to make 11 the default, and I don't want to interfere with that. Even though it's skipping 10, I don't want anything up to 11 to diverge from where the pgsql team is expecting it. If the 12 change goes over well, I will push the pgsql team to backport it.
A commit references this bug: Author: adamw Date: Sat Aug 17 16:56:19 UTC 2019 New revision: 509159 URL: https://svnweb.freebsd.org/changeset/ports/509159 Log: Add an XML option to postgresql12-contrib Avoids libxml2/libxslt for systems that don't need XML support. Although there was substantial interest in the PR to backport to 10, I've only done pgsql12 for now, as I don't want to interfere with the effort to make pgsql11 the default. If things work well here, it can be backported to earlier versions as well. PR: 239638 Approved by: maintainer timeout (2 weeks) Changes: head/databases/postgresql12-contrib/Makefile head/databases/postgresql12-server/pkg-plist-contrib
So, now that pgsql11 has become the default, can I get an up-or-down vote on backporting this to < 12?
A commit references this bug: Author: adamw Date: Mon Dec 30 18:04:41 UTC 2019 New revision: 521556 URL: https://svnweb.freebsd.org/changeset/ports/521556 Log: postgresql{96,10,11}-contrib: Add an XML option Avoids the libxml/libxslt2 dependency chain for systems that don't need the XML datatype plugin. PR: 239638 Approved by: maintainer timeout (pgsql team, nearly 5 months) Changes: head/databases/postgresql10-contrib/Makefile head/databases/postgresql10-server/pkg-plist-contrib head/databases/postgresql11-contrib/Makefile head/databases/postgresql11-server/pkg-plist-contrib head/databases/postgresql96-contrib/Makefile head/databases/postgresql96-server/pkg-plist-contrib
I'm closing out this PR, though the fact that it went 5 months without any response from the pgsql team is something we'll have to talk about separately.
Mmm, it seems the XML option was removed in https://svnweb.freebsd.org/ports/head/databases/postgresql12-server/Makefile?view=markup&pathrev=511971#l134