RFR: 8042088: Sjavac spawns external processes in a unnecessarily complex and platform dependent way

Jonathan Gibbons jonathan.gibbons at oracle.com
Fri May 2 22:01:17 UTC 2014


The code to use NUL or /dev/null is somewhat icky. Is that actually the 
behavior you want?
How about using ProcessBuilder.Redirect.INHERIT instead?

-- Jon

On 04/30/2014 06:33 AM, Andreas Lundblad wrote:
> Hi compiler-dev,
>
> Please review this small fix for JDK-8042088 which changes how sjavac spawns background processes.
>
> Description:
> Sjavac starts a background server through a shell (cmd on windows, and /bin/sh on other systems). This patch changes this so that the background service is launched directly instead.
>
> Link to webrev:
> http://cr.openjdk.java.net/~alundblad/8042088
>
> Link to bug report:
> https://bugs.openjdk.java.net/browse/JDK-8042088
>
> -- Andreas



More information about the compiler-dev mailing list