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: 2024-11-23
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
Blocks :  
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
I think this is no longer an issue. I haven't seen any evidence that it is using JAVA_HOME for any of the build and test steps when JDK_HOME is set to a different JDK. We run a weekly build and test using the latest promoted JDK, by setting JDK_HOME to that JDK (and using an earlier JDK set as JAVA_HOME to run gradle), and it is working as expected. The Description is talking about the javafx.graphics classes being compiled by buildSrc, which is no longer done. It might still be worth taking a quick look, but I think this can be closed as "Cannot reproduce".
23-11-2024

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