6950927: Testcase failure sun/management/jmxremote/bootstrap/JvmstatCountersTest.java

[David Holmes]

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
>