JDK-8197463 : Update libxml2 to version 2.9.7
  • Type: Bug
  • Component: javafx
  • Sub-Component: web
  • Affected Version: 8u162,10,openjfx11
  • Priority: P1
  • Status: Resolved
  • Resolution: Fixed
  • Submitted: 2018-02-09
  • Updated: 2020-01-31
  • Resolved: 2018-02-14
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.

To download the current JDK release, click here.
JDK 10 JDK 8
10 b44Fixed 8u171Fixed
Related Reports
Relates :  
Relates :  
We currently use libxml2 version 2.7.2. We should update to the latest 2.9.7 version.

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

Build + sanity test looks good on all three platforms. +1

Sanity testing looks good on windows and OSX. +1

Addressed the build issue on Ubuntu 16.04 in http://cr.openjdk.java.net/~arajkumar/8197463/webrev.01

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?

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