I'd love to get another review of this by a Reviewer. Someone, please? Thanks, /Staffan
On 16 mar 2012, at 09:25, Rickard Bäckman wrote: > Staffan, > > nice fixes, looks good to me. > > /R > > On Mar 16, 2012, at 9:10 AM, Staffan Larsen wrote: > >> Please review the following test fixes. >> >> Webrev: http://cr.openjdk.java.net/~sla/7154114/webrev.01/ >> >> The bugs are not available on the public server yet, but below is the text >> of the bugs. I intend to submit both fixes as one push. >> >> Fixes have been verified in JPRT. >> >> Thanks, >> /Staffan >> >> >> --------- >> 7154113: jcmd, jps and jstat tests failing when there are unknown Java >> processes on the system >> >> Some tests in sun/tools/{jcmd,jps,jstat,jstatd} fails when there are Java >> processes in the system for which we cannot determine the main class. >> >> These processes show up in jps and jcmd as just the pid, no name. This can >> happen if processes uses custom launchers or if they are launched with other >> JVMs. >> >> The tests should handle this gracefully. >> --------- >> >> --------- >> 7154114: jstat tests failing on non-english locales >> >> Some tests in sun/tools/{jstat,jstatd} fails when running on a non-english >> locale. Output from jstat has , instead of . as decimal delimiter on a >> swedish locale. >> -------- >
