TEST: java/util/logging/LoggerWeakRefLeak.sh

[Volker Simonis]

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.simonis at 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