RFR: JDK-8055922: Work around sjavac limitation with public api tracking cross modules

Erik Joelsson erik.joelsson at oracle.com
Tue Aug 26 09:58:45 UTC 2014


Updated webrev: http://cr.openjdk.java.net/~erikj/8055922/webrev.root.02/

Some of the demos failed to compile because the javac_state file did not 
contain any public api and this caused grep to exit with code 1, which 
failed the build. I made exit code 1 for this grep line not fail the build.

/Erik

On 2014-08-25 16:50, Erik Joelsson wrote:
> Hello,
>
> Please review this little workaround for a current shortcoming in 
> Sjavac. See bug for more details. With this change, Sjavac will start 
> acting correctly again and not miss any files that need to be 
> recompiled. The approximation is course however, we should still fix 
> https://bugs.openjdk.java.net/browse/JDK-8054717 to properly only 
> recompile what is really necessary. Still, this workaround does add 
> value compared to running without sjavac. With this change and 
> JDK-8014510 (also on review) I'm starting to feel confident enough to 
> make sjavac default for building jdk9.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8055922
> Webrev: http://cr.openjdk.java.net/~erikj/8055922/webrev.root.01/
>
> /Erik



More information about the compiler-dev mailing list