RFR: 8336941 : Update libxslt to 1.1.42
Kevin Rushforth
kcr at openjdk.org
Fri Aug 2 18:19:44 UTC 2024
On Fri, 2 Aug 2024 06:36:22 GMT, Hima Bindu Meda <hmeda at openjdk.org> wrote:
> Updated libxslt to v1.1.42. Verified build on all platforms. No issue seen
Looks good, although I notice one difference between the upstream libxslt repo and this PR in a now-unmaintained config file:
diff -u -w -r $JFX/modules/javafx.web/src/main/native/Source/ThirdParty/libxslt/src/libxslt.m4 $LIBXML/libxslt.m4
--- $JFX/modules/javafx.web/src/main/native/Source/ThirdParty/libxslt/src/libxslt.m4 2023-07-27 15:08:29.124274800 -0700
+++ $LIBXML/libxslt.m4 2024-08-02 10:18:29.007649600 -0700
@@ -5,6 +5,8 @@
# Configure paths for GLIB
# Owen Taylor 97-11-3
#
+# Deprecated and unmaintained. Use pkg-config or xslt-config.
+#
# Modified to work with libxslt by Thomas Schraitle 2002/10/25
# Fixed by Edward Rudd 2004/05/12
Do we use `libxslt.m4` when we run configure to generate the platform-specific headers? If not, I recommend that you remove it from our repo. If we do, please add the two lines from the above patch so that we match the upstream.
Note that the deprecation comment was added by:
https://gitlab.gnome.org/GNOME/libxslt/-/commit/200ee26982b4c8cbe165555ae36aa684a744ccf4
so we will eventually need to stop using it if we still are.
-------------
PR Review: https://git.openjdk.org/jfx/pull/1527#pullrequestreview-2216151444
More information about the openjfx-dev
mailing list