RFR: 8215369: Jcstress pollute /var/tmp with temporary files.

[Igor Ignatyev]

> On Dec 13, 2018, at 1:36 PM, Leonid Mesnik <leonid.mesnik at oracle.com> wrote:
> 
> 
> 
>> On Dec 13, 2018, at 1:24 PM, Igor Ignatyev <igor.ignatyev at oracle.com <mailto:igor.ignatyev at oracle.com>> wrote:
>> 
>> 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.
> 
> It is right after -jvmArgs. 
ok, it was just me misreading udiff. looks good and trivial to me, ship it.
> 
> 105         extraFlags.add("--jvmArgs");
> 106         extraFlags.add("-Djava.io.tmpdir=" + System.getProperty("user.dir"));
> then more --jvmArgs are added with other options in for loop.
> 
>> 
>> since this is test-only fix, I'd prefer to get it integrated into jdk/jdk12 rather than into jdk/jdk.
>> 
> ok.
> 
> Leonid
> 
>> Cheers,
>> -- Igor
>> 
>>> On Dec 13, 2018, at 12:57 PM, Leonid Mesnik <leonid.mesnik at oracle.com <mailto: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/> <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> <https://bugs.openjdk.java.net/browse/JDK-8215369 <https://bugs.openjdk.java.net/browse/JDK-8215369>>
>>> 
>>> Leonid
>> 
>