JDK-8169511 : correct concatenation of dependencies/*/*.extra files
  • Type: Bug
  • Component: javafx
  • Sub-Component: build
  • Priority: P4
  • Status: Resolved
  • Resolution: Fixed
  • Submitted: 2016-11-09
  • Updated: 2016-11-10
  • Resolved: 2016-11-10
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 9
9Fixed
Related Reports
Relates :  
JDK-8161704 - Switch to Jigsaw-aware boot JDK for compiling FX 9
Description
The extra files currently are parsed based on the keyword exports and would ignore lines with the keyword opens.
Comments
Changeset: f213beedd572 Author: ddhill Date: 2016-11-10 09:31 -0500 URL: http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/f213beedd572 8169511: correct concatenation of dependencies/*/*.extra files Reviewed-by: kcr ! build.gradle
10-11-2016
+1
09-11-2016
ddhill@ddhill-pad:~/sb/9/graphics/rt$ hg diff diff --git a/build.gradle b/build.gradle --- a/build.gradle +++ b/build.gradle @@ -3850,7 +3850,7 @@ List<String> lines = extras.get(usename) e.eachLine { line -> line = line.trim() - if (line.contains("exports")) { + if (line.length() > 1 && Character.isLetter(line.charAt(0))) { lines << line } } @@ -3858,8 +3858,9 @@ } else { List<String> lines = [] e.eachLine { line -> - if (line.contains("exports")) { - lines << line.trim() + line = line.trim() + if (line.length() > 1 && Character.isLetter(line.charAt(0))) { + lines << line } } extras.put(usename,lines)
09-11-2016