Re: Fwd: Remove including of link.h to improve portability

2012-03-15 Thread Jonathan Lu
On 03/13/2012 06:25 PM, Alan Bateman wrote: On 13/03/2012 10:21, Jonathan Lu wrote: Hi Alan, With Phil's advice, I've splitted the original patch into two parts and created another bug for TL repository, bug 7153343. And here's the rebased patch based on TL repository. http://cr.openjdk.java

Re: Fwd: Remove including of link.h to improve portability

2012-03-15 Thread Chris Hegarty
On 15/03/12 07:17, Jonathan Lu wrote: On 03/13/2012 06:25 PM, Alan Bateman wrote: On 13/03/2012 10:21, Jonathan Lu wrote: Hi Alan, With Phil's advice, I've splitted the original patch into two parts and created another bug for TL repository, bug 7153343. And here's the rebased patch based on

hg: jdk8/tl/jdk: 2 new changesets

2012-03-15 Thread michael . x . mcmahon
Changeset: ac5024504439 Author:michaelm Date: 2012-03-15 16:45 + URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/ac5024504439 7151898: Check for LANG in Mac OS X jdk build sanity check [macosx] Reviewed-by: ohair, smarks ! make/common/shared/Sanity.gmk Changeset: cfe2328912b3

hg: jdk8/tl/jdk: 7045655: An empty InMemoryCookieStore should not return true for removeAll

2012-03-15 Thread kurchi . subhra . hazra
Changeset: 3bfebedb549f Author:khazra Date: 2012-03-15 13:21 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/3bfebedb549f 7045655: An empty InMemoryCookieStore should not return true for removeAll Summary: CookieStore.removeAll() should return false for an empty CookieStore Re

hg: jdk8/tl/jdk: 2 new changesets

2012-03-15 Thread valerie . peng
Changeset: bdbc32b2f920 Author:valeriep Date: 2012-03-15 14:28 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/bdbc32b2f920 7130959: Tweak 7058133 fix for JDK 8 (javah makefile changes) Summary: Fixed JAVAHFLAGS setting to use -bootclasspath. Reviewed-by: wetmore ! make/sun/s

Re: [7u4] Request for approval for CR 7088989 - Improve the performance for T4 by utilizing the newly provided crypto APIs

2012-03-15 Thread Andrew Hughes
- Original Message - > On 3/14/12 9:08 AM, Andrew Hughes wrote: > > 1. java.security is altered to prioritise > > com.oracle.security.ucrypto.UcryptoProvider > > on Solaris builds. What does this mean for OpenJDK builds on > > Solaris that don't have > > this provider? > > Nothing, really

Re: [7u4] Request for approval for CR 7088989 - Improve the performance for T4 by utilizing the newly provided crypto APIs

2012-03-15 Thread Valerie (Yu-Ching) Peng
On 03/15/12 16:29, Andrew Hughes wrote: 2. Was there a reason the testcases in this commit were locked down to only work with the proprietary ucrypto provider? Are they not suitable for all providers? They would be very useful for testing the SunPKCS11/NSS provider. That's a question better s

Re: [7u4] Request for approval for CR 7088989 - Improve the performance for T4 by utilizing the newly provided crypto APIs

2012-03-15 Thread Valerie (Yu-Ching) Peng
Forgot to mention that I could move the tests out of OpenJDK if that'd help things . And all will be done following the 7u public review process. ;) Valerie On 03/15/12 17:14, Valerie (Yu-Ching) Peng wrote: On 03/15/12 16:29, Andrew Hughes wrote: 2. Was there a reason the testcases in this

Re: Fwd: Remove including of link.h to improve portability

2012-03-15 Thread Jonathan Lu
On 03/15/2012 10:42 PM, Chris Hegarty wrote: On 15/03/12 07:17, Jonathan Lu wrote: On 03/13/2012 06:25 PM, Alan Bateman wrote: On 13/03/2012 10:21, Jonathan Lu wrote: Hi Alan, With Phil's advice, I've splitted the original patch into two parts and created another bug for TL repository, bug 71

hg: jdk8/tl/jdk: 7153343: Dependency on non-POSIX header file causes portability problem

2012-03-15 Thread littlee
Changeset: c4e66dc3222d Author:littlee Date: 2012-03-16 10:47 +0800 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/c4e66dc3222d 7153343: Dependency on non-POSIX header file causes portability problem Summary: Remove the unneccessary link.h Reviewed-by: alanb, chegar Contributed-by