Re: RFR: 8009280: JCE jurisdiction policy files not copied into jdk/lib/security

2013-05-07 Thread Erik Joelsson
It seems to work for me. The reason I removed things from RT_JAR_EXCLUDES that are no longer present in jdk/classes outputdir. /Erik On 2013-05-07 07:33, David Holmes wrote: Erik, Was this tested with a Profiles build? The changes to the RT_JAR_INCLUDES variable has me concerned.

Re: RFR: 8009280: JCE jurisdiction policy files not copied into jdk/lib/security

2013-05-07 Thread David Holmes
On 7/05/2013 7:22 PM, Erik Joelsson wrote: It seems to work for me. The reason I removed things from RT_JAR_EXCLUDES that are no longer present in jdk/classes outputdir. Ah okay that should be fine then. If they aren't there in the first place then no need to exclude :) Thanks, David

Re: RFR: 8009280: JCE jurisdiction policy files not copied into jdk/lib/security

2013-05-07 Thread Erik Joelsson
This patch didn't work with sjavac enabled. Back to the drawing board. /Erik On 2013-04-30 16:45, Erik Joelsson wrote: With this patch the security tests will again be runnable on the exploded jdk image. The main changes are: * The security classes are compiled separately to a different

hg: jdk8/tl/jdk: 8013086: NPE thrown by SimpleDateFormat with TimeZoneNameProvider supplied

2013-05-07 Thread naoto . sato
Changeset: e30396e22c6f Author:naoto Date: 2013-05-07 11:31 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/e30396e22c6f 8013086: NPE thrown by SimpleDateFormat with TimeZoneNameProvider supplied Reviewed-by: okutsu !

hg: jdk8/tl/jdk: 4802647: Throw required NPEs from removeAll()/retainAll()

2013-05-07 Thread mike . duigou
Changeset: fe4e9bc2186f Author:mduigou Date: 2013-05-07 12:05 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/fe4e9bc2186f 4802647: Throw required NPEs from removeAll()/retainAll() Reviewed-by: mduigou, chegar, dholmes Contributed-by: Brandon Passanisi

Re: Code review request: 8012679: Let allow_weak_crypto default to false

2013-05-07 Thread Valerie (Yu-Ching) Peng
Changes look fine. Thanks, Valerie On 05/06/13 19:47, Weijun Wang wrote: Webrev at http://cr.openjdk.java.net/~weijun/8012679/webrev.00/ Thanks Max

hg: jdk8/tl/langtools: 8004082: test/tools/javac/plugin/showtype/Test.java fails on windows: jtreg can't delete plugin.jar

2013-05-07 Thread jonathan . gibbons
Changeset: 43c2f7cb9c76 Author:jjg Date: 2013-05-07 14:27 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/43c2f7cb9c76 8004082: test/tools/javac/plugin/showtype/Test.java fails on windows: jtreg can't delete plugin.jar Reviewed-by: vromero, mcimadamore !

hg: jdk8/tl/jdk: 8012664: Add tests for java.util.stream and lambda translation

2013-05-07 Thread mike . duigou
Changeset: 6feee75b0a8b Author:briangoetz Date: 2013-05-06 11:43 -0400 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/6feee75b0a8b 8012664: Add tests for java.util.stream and lambda translation Reviewed-by: mduigou, briangoetz Contributed-by: Brian Goetz brian.go...@oracle.com,

Code Review Request for 8013069: javax.crypto tests fail with new PBE algorithm names

2013-05-07 Thread Valerie (Yu-Ching) Peng
Vinnie, Could you please help reviewing the fixes for 8013069 javax.crypto tests fail with new PBE algorithm names? Given that the current javax.crypto.Mac API doesn't provide a way to return algorithm parameters (unlike most of other crypto engine classes), I think it's better to error out