JDK-8087537 : Build scripts should only use JDK_HOME, not JAVA_HOME
  • Type: Bug
  • Component: javafx
  • Sub-Component: build
  • Affected Version: 8u20
  • Priority: P4
  • Status: Open
  • Resolution: Unresolved
  • Submitted: 2014-04-18
  • Updated: 2018-09-05
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.
Other
tbdUnresolved
Related Reports
Relates :  
Relates :  
Description
The issue was discovered after a fix for RT-35197.

$ JDK_HOME=/usr/java/8 gradle 
 Defining Closed Properties 
:buildSrc:clean 
:buildSrc:generateGrammarSource 
:buildSrc:compileJava 
/net/lin/export/anthony/ws/g-28-twoTransparentWindows-RT-36716/rt/buildSrc/src/main/java/com/sun/scenario/effect/compiler/backend/sw/sse/SSEBackend.java:85: error: ')' expected 
        Comparator<Variable> c = (v0, v1) -> v0.getName().compareTo(v1.getName()); 
                                    ^ 
/net/lin/export/anthony/ws/g-28-twoTransparentWindows-RT-36716/rt/buildSrc/src/main/java/com/sun/scenario/effect/compiler/backend/sw/sse/SSEBackend.java:85: error: ';' expected 
        Comparator<Variable> c = (v0, v1) -> v0.getName().compareTo(v1.getName()); 
                                        ^ 
2 errors 
:buildSrc:compileJava FAILED 
Comments
Moving to tbd_major; this might be needed as part of JDK-8161704 in which case it will need to be pulled back in (or just folded into that issue).
08-08-2016

Bumping priority to P3 since we will need this fixed for Jigsaw.
29-10-2015

Copied the following comment by Anthony from RT-35197: "On a side note, our gradle script checks the minimum JDK_HOME version and requires it to be 8b115+. But the check is only performed after the SSEBackend.java compilation. Perhaps we should move the JDK version check a bit earlier. Also, I think it makes sense to require using 8 GA (b132) instead of b115 now. "
22-04-2014