Request for review (S) 7009828: Fix for 6938627 breaks visualvm monitoring when -Djava.io.tmpdir is defined
Alan.Bateman at oracle.com
Wed Jan 12 11:24:08 PST 2011
Coleen Phillimore wrote:
> Summary: Change get_temp_directory() back to /tmp and %TEMP% like it
> always was and where the tools expect it to be.
> There is more information in the bug, but essentially changing
> get_temp_directory() to use java.io.tmpdir breaks a lot of things.
> open webrev at http://cr.openjdk.java.net/~coleenp/7009828/
> bug link at http://bugs.sun.com/view_bug.do?bug_id=7009828
This looks okay to me but does mean we'll need to partially back-out
6950927  from jdk7. I can do that for you. It will mean that the test
sun/management/jmxremote/bootstrap/JvmstatCountersTest.java will fail
until we meet up in a promoted build but I think we can live with that
(or just add the @ignore until both changes come together).
More information about the hotspot-runtime-dev