Code review request: 8010531: Add BadKdc* tests to problem list for solaris-sparcv9

2013-03-22 Thread Weijun Wang
Please take a look at http://cr.openjdk.java.net/~weijun/8010531/webrev.00/ The tests have been failed a lot recently. Put them in problem list now. Thanks Max

Re: Code review request: 8010531: Add BadKdc* tests to problem list for solaris-sparcv9

2013-03-22 Thread Alan Bateman
On 22/03/2013 09:49, Weijun Wang wrote: Please take a look at http://cr.openjdk.java.net/~weijun/8010531/webrev.00/ The tests have been failed a lot recently. Put them in problem list now. Thanks Max Looks fine to me but curious that this is Solaris 64-bit only (or is it Solaris 11 only?)

Re: Code review request: 8010531: Add BadKdc* tests to problem list for solaris-sparcv9

2013-03-22 Thread Seán Coffey
I was wondering the same. Is this sparc only or is solaris x86 impacted also ? regards, Sean. On 22/03/2013 10:00, Alan Bateman wrote: On 22/03/2013 09:49, Weijun Wang wrote: Please take a look at http://cr.openjdk.java.net/~weijun/8010531/webrev.00/ The tests have been failed a lot

Re: Code review request: 8010531: Add BadKdc* tests to problem list for solaris-sparcv9

2013-03-22 Thread Weijun Wang
All recent failures are on sparc. Since SQE only run 64 bit, I cannot confirm about 32 bit. They do run tests on solaris-i586 and tests run fine there. Definitely not only Solaris 11. Multiple Solaris 10 failures observed. Thanks Max On 3/22/13 6:02 PM, Seán Coffey wrote: I was wondering

hg: jdk8/tl/jdk: 8010531: Add BadKdc* tests to problem list for solaris-sparcv9

2013-03-22 Thread weijun . wang
Changeset: 93cd7052d306 Author:weijun Date: 2013-03-22 19:59 +0800 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/93cd7052d306 8010531: Add BadKdc* tests to problem list for solaris-sparcv9 Reviewed-by: alanb ! test/ProblemList.txt

hg: jdk8/tl/langtools: 5 new changesets

2013-03-22 Thread maurizio . cimadamore
Changeset: cc38a6723663 Author:mcimadamore Date: 2013-03-22 12:38 + URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/cc38a6723663 8009649: Lambda back-end should generate invokespecial for method handles referring to private instance methods Summary: Private lambda

Re: Code review request: 8009970: Several LoginModule classes need extra permission to load AuthResources

2013-03-22 Thread Sean Mullan
This change looks fine to me. --Sean On 03/13/2013 03:13 AM, Weijun Wang wrote: http://cr.openjdk.java.net/~weijun/8009970/webrev.00 I was checking for krb5 permissions and noticed this. There is really no need for a permission to access AuthResources strings in these login modules. Also

hg: jdk8/tl/langtools: 7080464: langtools regression test failures when assertions are enabled

2013-03-22 Thread joe . darcy
Changeset: f4500abff1fd Author:darcy Date: 2013-03-22 10:08 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/f4500abff1fd 7080464: langtools regression test failures when assertions are enabled Reviewed-by: jjg ! test/tools/javac/api/TestJavacTaskScanner.java !

code review request: 7171982 Cipher getParameters() throws RuntimeException: Cannot find SunJCE provider

2013-03-22 Thread Anthony Scarpino
Hi all, I need a code review for below webrev. The changes are to have SunJCE call itself, using it's current instance, for checking such things as parameters, instead of searching through the provider list or creating a one time instance.

Re: Code review request: 8009875: Provide a default udp_preference_limit for krb5.conf

2013-03-22 Thread Valerie (Yu-Ching) Peng
Look fine to me. Thanks, Valerie On 03/20/13 18:57, Weijun Wang wrote: Please review the code change at http://cr.openjdk.java.net/~weijun/8009875/webrev.00/ A default udp_preference_limit value is defined. Note: The fix in KDC.java is trivial and not related to this bug. Thanks Max

hg: jdk8/tl/jdk: 2 new changesets

2013-03-22 Thread weijun . wang
Changeset: 3470101fae58 Author:weijun Date: 2013-03-23 11:49 +0800 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/3470101fae58 8009970: Several LoginModule classes need extra permission to load AuthResources Reviewed-by: mullan !