RFR: 8215369: Jcstress pollute /var/tmp with temporary files.
Igor Ignatyev
igor.ignatyev at oracle.com
Thu Dec 13 21:24:07 UTC 2018
Hi Leonid,
shouldn't 2nd -Djava.io.tmpdir go after '--jvmArgs' ?
> 106 extraFlags.add("-Djava.io.tmpdir=" + System.getProperty("user.dir"));
the rest looks good.
since this is test-only fix, I'd prefer to get it integrated into jdk/jdk12 rather than into jdk/jdk.
Cheers,
-- Igor
> On Dec 13, 2018, at 12:57 PM, Leonid Mesnik <leonid.mesnik at oracle.com> wrote:
>
> Hi
>
> Could you please review following fix which set property "java.io.tmpdir" for jctress harness and it's forked VMs . So any temporary files are created in test directory and not in system temp.
>
> webrev: http://cr.openjdk.java.net/~lmesnik/8215369/webrev.00/ <http://cr.openjdk.java.net/~lmesnik/8215369/webrev.00/>
> bug: https://bugs.openjdk.java.net/browse/JDK-8215369 <https://bugs.openjdk.java.net/browse/JDK-8215369>
>
> Leonid
More information about the hotspot-dev
mailing list