On 8/7/20 3:11 PM, Daniel D. Daugherty wrote:
On 8/7/20 6:03 PM, Chris Plummer wrote:
Hello,

Please review the following:

https://bugs.openjdk.java.net/browse/JDK-8241951
http://cr.openjdk.java.net/~cjplummer/8241951/webrev.00/index.html

test/lib/jdk/test/lib/util/CoreUtils.java
    L139:                 if (Platform.getOsVersionMajor() == 10 && Platform.getOsVersionMinor() >= 15) {
        Should the major version check be ">= 10"?
If I'm going to check for >= 10, then it needs to be something like :

   if (major > 10 || (major == 10 && minor >= 15)


        Platform.isSignedOSX() would return true for signed 11.X. Dunno.

Yes, it would return true for 11.x if signed, which is what we would want.

Chris

Thumbs up.

Dan



We need to disable SA core file testing on OSX 10.15.* and later when the binary is signed because OSX will no longer produce core files with this configuration.

thanks,

Chris



Reply via email to