TEST: java/util/logging/LoggerWeakRefLeak.sh

Volker Simonis volker.simonis at gmail.com
Wed Dec 18 08:44:58 PST 2013


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


More information about the quality-discuss mailing list