[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-serviceability-dev
Subject:    6950927: Testcase failure sun/management/jmxremote/bootstrap/JvmstatCountersTest.java
From:       David.Holmes () oracle ! com (David Holmes)
Date:       2010-05-31 23:40:37
Message-ID: 4C0448F5.6090800 () oracle ! com
[Download RAW message or body]

Hi Alan,

Looks okay to me. There are a couple of /tmp references left in the 
comments:

Linux:
268     // found it looks in /tmp.
276             // Not found, so try /tmp

Solaris:
196     // directory or /tmp.
222     // target VM's working directory (or /tmp), and the SIGQUIT 
handler checks

Cheers,
David

Alan Bateman said the following on 06/01/10 04:39:
> 
> I need a reviewer to a small fix to the attach API. A few weeks back, 
> Coleen pushed changes to hotspot so that it uses the value of the 
> java.io.tmpdir property as the temporary directory [1]. The jdk side to 
> this is that the attach API also needs to use this property as otherwise 
> tools can't attach when the target VM isn't using the default temporary 
> file directory. The webrev to restore world order is here:
>   http://cr.openjdk.java.net/~alanb/6950927/webrev/
> 
> Thanks,
> Alan.
> 
> http://hg.openjdk.java.net/jdk7/hotspot-rt/hotspot/rev/a2ea687fdc7c
> 

[prev in list] [next in list] [prev in thread] [next in thread]