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 :  
Description
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
13-02-2018