* Erik Joelsson:

> A minor mistake in JDK-8244036 is causing the javac server to never be 
> used, which is rather severly increasing build times.
>
> Before that change, the global variable ENABLE_SJAVAC was used to 
> determine if the server should be activated. After the change, the 
> global variable has changed names to ENABLE_JAVAC_SERVER. The 
> conditional in JavaCompilation.gmk tries to check the value of the local 
> variable/parameter $1_ENABLE_JAVAC_SERVER instead, but this parameter is 
> never set or given a default. I don't think we need a local 
> variable/parameter for this as there is currently no need to disable the 
> server for any specific SetupJavaCompilation call, so I my fix is to 
> just refer directly to the global variable from configure/spec.gmk instead.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8244210
>
> Webrev: http://cr.openjdk.java.net/~erikj/8244210/webrev.01/webrev/

I'm happy to report that this brings back the build times to normal.

Reply via email to