[PATCH] Prefer TMPDIR over hard coded /tmp

Alan Bateman Alan.Bateman at oracle.com
Thu Apr 26 06:24:43 UTC 2018



On 26/04/2018 01:45, Brian Burkhalter wrote:
> On Apr 25, 2018, at 5:38 PM, Brian Burkhalter <brian.burkhalter at oracle.com> wrote:
>
>>> The attached patch changed the behavior to use the content of the environment variable TMPDIR, if present. Note that this does not change where the hsperf* folders are created.
>> The source change looks OK to me aside from the copyright year and that in the second line here
> Note that I am only saying that the patch looks OK, not that we want to use it. There have been a number of issues filed in this area over time, for example https://bugs.openjdk.java.net/browse/JDK-6205979. There are arguments against this change.
>
Right, it's come a few times. In principle it seems reasonable but it 
will likely have a tail of issues. One concern is that it will cause 
issues with the attach mechanism as that depends on the tool side 
knowing the target VM's tmp dir. There's also the transition issue where 
old JDKs will be using /tmp, new JDKs using whatever TMPDIR is set too. 
This is one of these issues where most of the work will be shaking out 
the knock on impact (that will be needed for the CSR and release note at 
least).

-Alan


More information about the core-libs-dev mailing list