Project

Profile

Help

Bug #5824

closed

SaxonJ 12 does not run on Java 8

Added by Michael Kay almost 2 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Sprint/Milestone:
-
Start date:
2023-01-16
Due date:
% Done:

0%

Estimated time:
Legacy ID:
Applies to branch:
12, trunk
Fix Committed on Branch:
12, trunk
Fixed in Maintenance Release:
Platforms:
Java

Description

We say in the documentation that we're targeting Java 11 but that the code should still run on Java 8. However, it is reported that it doesn't.

Note: we are using at least one Java 9 feature, namely the java.lang.ref.Cleaner class.

Actions #1

Updated by Norm Tovey-Walsh almost 2 years ago

The source and target compatibility flags were accidentally set to "11" not "1.8" in the build scripts.

However, I assume that only fixes the build issue and it would still fail on Java 8 because of the Java 9 dependency...

Actions #2

Updated by Michael Kay almost 2 years ago

I have moved all the dependencies on java.lang.ref.Cleaner into one place (CleanerProxy.java) which is designed to have fallback behaviour if the Cleaner class is not available. The fallback action is not yet tested.

Also noted, there are also "soft" Java 9 dependencies in JavaPlatform.java, with fallback action for Java 8.

Actions #3

Updated by Michael Kay almost 2 years ago

  • Status changed from New to In Progress
Actions #4

Updated by Debbie Lockett almost 2 years ago

I have added a note on the website (https://www.saxonica.com/products/latest.xml#saxonJ12) where we mention Java versions, so that it now says:

SaxonJ 12 is built and tested using Java 11, but should still be usable with Java 8 or later. (However, due to bug #5824, note that SaxonJ 12.0 does not run on Java 8.)

I decided against adding this note in the documentation (because it's a code bug and we don't generally update the documentation to highlight code bugs). But let me know if you think I should. (The relevant documentation pages are: about/gettingstarted/gettingstartedjava, about/installationjava/prerequisites, changes/installation.)

Actions #5

Updated by Norm Tovey-Walsh almost 2 years ago

  • Status changed from In Progress to Resolved

This has been fixed and tested, I believe.

Actions #6

Updated by Norm Tovey-Walsh over 1 year ago

  • Status changed from Resolved to Closed
  • Fix Committed on Branch 12, trunk added
  • Fixed in Maintenance Release 12.1 added

I'm pretty sure this was fixed for the 12.1 release.

Please register to edit this issue

Also available in: Atom PDF