[8u] RFR: 8195153: [test] runtime/6981737/Test6981737.java shouldn't check 'java.vendor' and 'java.vm.vendor' properties

Andrew Hughes gnu.andrew at redhat.com
Fri Mar 1 19:46:01 UTC 2019


On Fri, 1 Mar 2019 at 16:12, Severin Gehwolf <sgehwolf at redhat.com> wrote:
>
> Hi,
>
> Could somebody please review this 8u backport for a test which checks
> the java.vendor and java.vm.vendor property? The JDK 8 patch is the
> same as JDK 10's version modulo contextual changes and reshuffeling.
>
> the JDK 10 patch does not apply cleanly because there is no
> Runtime.version() in JDK 8 (that is part of the patch context).
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8195153
> webrev: http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8195153/jdk8/01/webrev/
>
> Testing: Test fails with AdoptOpenJDK's JDK 8 build, and passes after
> the fix.
>
> Thanks,
> Severin
>

Looks good to me.

I think we also need https://bugs.openjdk.java.net/browse/JDK-8189761 to make
this really worthwhile.

Thanks,
-- 
Andrew :)

Senior Free Java Software Engineer
Red Hat, Inc. (http://www.redhat.com)

Web Site: http://fuseyism.com
Twitter: https://twitter.com/gnu_andrew_java
PGP Key: ed25519/0xCFDA0F9B35964222 (hkp://keys.gnupg.net)
Fingerprint = 5132 579D D154 0ED2 3E04  C5A0 CFDA 0F9B 3596 4222


More information about the jdk8u-dev mailing list