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
