Sorry, I forgot to mention that the same problem holds true for the test java/util/logging/AnonLoggerWeakRefLeak.sh as well. And it can be fixed in the same way as proposed for LoggerWeakRefLeak.sh.
Volker On Wed, Dec 18, 2013 at 5:44 PM, Volker Simonis <volker.simo...@gmail.com> wrote: > Hi, > > the test 'java/util/logging/LoggerWeakRefLeak.sh' checks for the > availability of the '-histo:live' option in jmap which is only > available if the SA is available. > > However I don't think that it should be an error if the option isn't > supported because of a missing SA implementation on a certain > platfrom. > > I suggest the following fix: > > --- a/test/java/util/logging/LoggerWeakRefLeak.sh Tue Dec 17 > 19:01:21 2013 +0100 > +++ b/test/java/util/logging/LoggerWeakRefLeak.sh Wed Dec 18 > 17:40:46 2013 +0100 > @@ -83,9 +83,9 @@ > fi > > if [ "$status" != 0 ]; then > - echo "ERROR: 'jmap $jmap_option' is not supported so this test" > - echo "ERROR: cannot work reliably. Aborting!" > - exit 2 > + echo "WARNING: 'jmap $jmap_option' is not supported on this platform" > + echo "WARNING: so this test cannot work reliably. Aborting!" > + exit 0 > fi > fi > > I want to put this fix into the patch for "8028537: PPC64: Updated > jdk/test scripts to understand the AIX os and environment" so I don't > think we need a special BugID for it. > > What do you think? > > Regards, > Volker
