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

[Daniel D. Daugherty]

On 6/1/2010 3:21 AM, Alan Bateman wrote:

David Holmes wrote:

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

Thanks David. I'll fix up these comments before pushing this.

Thumbs up on the JDK repo changes include the "/tmp" comment updates.

Dan