hg: jdk7/build/jdk: 7038711: Fix CC_VER checks for compiler options, fix use of -Wno-clobber

2011-04-21 Thread kelly . ohair
Changeset: b715eb9f3d99 Author:ohair Date: 2011-04-21 18:26 -0700 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/b715eb9f3d99 7038711: Fix CC_VER checks for compiler options, fix use of -Wno-clobber Reviewed-by: igor ! make/common/Defs-linux.gmk ! make/common/Defs-solaris.gmk !

Need reviewer: Correcting jar manifests

2011-04-21 Thread Kelly O'Hair
Need reviewer for these jar manifest changes. Multiple issues here: * Bean manifest information should only be in rt.jar, not resources.jar, jsse.jar, * Added missing basic jdk manifest information into demo jar manifests * JCE jars manifest missing company name change * Specification ve

Re: Need reviewer: CC_VER checks on compiler options

2011-04-21 Thread Dr Andrew John Hughes
On 14:33 Thu 21 Apr , Kelly O'Hair wrote: > > This started out as an annoyance around the use of -Wno-clobber on Linux when > the gcc might > not support it. Turned into fixing several CC_VER checks in the makefiles: > > 7038711: Fix CC_VER checks for compiler options, fix use of -Wno-clobbe

Re: Need reviewer: CC_VER checks on compiler options

2011-04-21 Thread Kelly O'Hair
Thanks. I'll run it through JPRT and multiple builds before integrating. Did not intend to be fixing this at this time, but I can't leave it be. Must be the altar boy in me. ;^) -kto On Apr 21, 2011, at 2:47 PM, Igor Nekrestyanov wrote: > I do not see anything obviously wrong in this change (

Need reviewer: CC_VER checks on compiler options

2011-04-21 Thread Kelly O'Hair
This started out as an annoyance around the use of -Wno-clobber on Linux when the gcc might not support it. Turned into fixing several CC_VER checks in the makefiles: 7038711: Fix CC_VER checks for compiler options, fix use of -Wno-clobber http://cr.openjdk.java.net/~ohair/openjdk7/jdk7-build-cc