7175775: Disable SA options in jinfo/Basic.java test until SA updated for new hash and String count/offset

Staffan Larsen staffan.larsen at oracle.com
Sun Jun 10 23:54:44 PDT 2012


Looks like a good compromise to me.

Thanks!
/Staffan

On 10 jun 2012, at 11:28, Alan Bateman wrote:

> 
> sun/tools/jinfo/Basic.java is currently failing with builds of jdk8/tl because SA hasn't been updated to work with the enhanced hashing algorithm for Strings. It probably also needs requires because of the removal of the String offset/count fields. The SA work is tracked by 7175133. In the mean-time we need to add this test to the ProblemList.txt file so that it is excluded from test runs, or else modify the test so that it skips the command line options that use SA. The latter seems preferable so I propose the attached patch. This patch will of course need to be reserved once the SA changes are done.
> 
> -Alan
> 
> diff --git a/test/sun/tools/jinfo/Basic.sh b/test/sun/tools/jinfo/Basic.sh
> --- a/test/sun/tools/jinfo/Basic.sh
> +++ b/test/sun/tools/jinfo/Basic.sh
> @@ -43,7 +43,8 @@ set +e
> 
> failed=0
> 
> -runSA=true
> +# Skip SA options for now, see 7175133
> +runSA=false
> 
> if [ $isMacos = true ]; then
>     runSA=false
> 



More information about the serviceability-dev mailing list