The Version table provides details related to the release that this issue/RFE will be addressed.
Unresolved : Release in which this issue/RFE will be addressed. Resolved: Release in which this issue/RFE has been resolved. Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.
We currently use libxml2 version 2.7.2. We should update to the latest 2.9.7 version.
Comments
Changeset: d4868b8fd76f
Author: arajkumar
Date: 2018-02-14 07:01 -0800
URL: http://hg.openjdk.java.net/openjfx/10-dev/rt/rev/d4868b8fd76f
8197463: Update libxml2 to version 2.9.7
Reviewed-by: kcr, mbilla
14-02-2018
Build + sanity test looks good on all three platforms.
+1
14-02-2018
Sanity testing looks good on windows and OSX.
+1
14-02-2018
Addressed the build issue on Ubuntu 16.04 in http://cr.openjdk.java.net/~arajkumar/8197463/webrev.01
14-02-2018
The diffs to the build and libxml2.md file look fine to me.
With your patch applied, it builds fine on Mac, Windows, and Oracle Linux, but not on Ubuntu 16.04. I traced the problem to this line in the newly added libxslt/linux/config.h file:
/* Define to 1 if you have the <ansidecl.h> header file. */
#define HAVE_ANSIDECL_H 1
This is defined on Windows (which is fine), but not defined on Mac (which is also fine as that file is not present). I suspect it should not be defined on Linux, either. A test build on Ubuntu with that line deleted ran fine. Can you comment on this?
13-02-2018
Please review patch available at http://cr.openjdk.java.net/~arajkumar/8197463/webrev.00
Compressed changeset: http://cr.openjdk.java.net/~arajkumar/8197463/webrev.00/libxml2-2.9.7v3.diff.gz
Cmake only changes for review: http://cr.openjdk.java.net/~arajkumar/8197463/webrev.00/cmake-changes
Changes:
1. Merged libxml2-2.9.7 changes from https://git.gnome.org/browse/libxml2/tag/?h=v2.9.7
2. Enabled libxml compilation on Linux and macOS
3. Enabled libxslt compilation on Linux and macOS