Great, go ahead and push the fix.
--Sean
On 8/27/15 7:49 PM, Pete Brunet wrote:
JPRT jdk_lang tests ran clean, 7 builds passed, 8 tests passed.
http://scaaa637.us.oracle.com//archive/2015/08/2015-08-27-204226.pbrunet.dev
command line:
jprt submit -stree . -email [email protected] -c JDK-8134456
-buildenv SKIP_BOOT_CYCLE=false -noqa -ot '.*product.*' -onlytests
'.*jdk_lang'
Pete
On 8/27/15 4:59 PM, Pete Brunet wrote:
Hi Sean, I might need to make a change but this is the JTREG output from
my local run
jtreg -jdk:./build/windows-x86_64-normal-server-release/images/jdk
./jdk/test/java/lang/SecurityManager/
Execution successful
java/lang/SecurityManager/CheckPackageAccess.java: Make sure all
restricted packages listed in the package.access property in the
java.security file are blocked
java/lang/SecurityManager/CheckPackageMatching.java: Check the matching
implemented by SecurityManager.checkPackageAccess
java/lang/SecurityManager/CheckSecurityProvider.java: Consolidate
java.security files into one file with modifications
java/lang/SecurityManager/NoAWT.java: Sanity check that SecurityManager
methods that used to check AWTPermission now check for AllPermission
so it looks like there are no problems. What do you think?
I'll report back once my JPRT run finishes.
Pete
On 8/27/15 4:50 PM, Sean Mullan wrote:
This looks good to me, if the JPRT results are good. However, does
this also fix the failure in CheckPackageMatching? I would have
expected you needed an additional fix - see my subsequent comment:
https://bugs.openjdk.java.net/browse/JDK-8134456?focusedCommentId=13837407&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13837407
Thanks,
Sean
On 08/27/2015 05:02 PM, Pete Brunet wrote:
Please review a patch for a test change which is needed to fix an issue
caused by JDK-8051626. Tier 1 tests are being impacted.
http://cr.openjdk.java.net/~ptbrunet/JDK-8134456/webrev.00/
Pete
