Re: Need reviewer: Windows use of /safeseh

2011-02-16 Thread Kelly O'Hair
Point is that the option /SAFESEH is accepted on Windows 64bit and does not complain that it does not apply. And even though it always uses table driven exception handling on 64bit, dumpbin output doesn't indicate it does. Which is fine, I can work around that. The latest webrev makes this a

Re: Need reviewer: Windows use of /safeseh

2011-02-16 Thread Kelly O'Hair
I still need a reviewer on this. -kto On Feb 15, 2011, at 2:05 PM, Kelly O'Hair wrote: Need reviewer: Windows use of /safeseh Just an addition of /safeseh during links and a check to verify that the dll and exe files all have a Safe Exception Handler Table. See http://msdn.microsoft.com/en

Re: Need reviewer: Windows use of /safeseh

2011-02-16 Thread Alan Bateman
Kelly O'Hair wrote: I still need a reviewer on this. -kto I went through the changes and they look good to me. -Alan. On Feb 15, 2011, at 2:05 PM, Kelly O'Hair wrote: Need reviewer: Windows use of /safeseh Just an addition of /safeseh during links and a check to verify that the dll and

hg: jdk7/build/corba: 7013964: openjdk LICENSE file needs rebranding

2011-02-16 Thread kelly . ohair
Changeset: a9335b646c1c Author:ohair Date: 2011-02-16 13:29 -0800 URL: http://hg.openjdk.java.net/jdk7/build/corba/rev/a9335b646c1c 7013964: openjdk LICENSE file needs rebranding Reviewed-by: darcy, katleman, jjg ! LICENSE

hg: jdk7/build: 7013964: openjdk LICENSE file needs rebranding

2011-02-16 Thread kelly . ohair
Changeset: da55264ff2fb Author:ohair Date: 2011-02-16 13:29 -0800 URL: http://hg.openjdk.java.net/jdk7/build/rev/da55264ff2fb 7013964: openjdk LICENSE file needs rebranding Reviewed-by: darcy, katleman, jjg ! LICENSE

hg: jdk7/build/hotspot: 7013964: openjdk LICENSE file needs rebranding

2011-02-16 Thread kelly . ohair
Changeset: cdef89d821bd Author:ohair Date: 2011-02-16 13:30 -0800 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/cdef89d821bd 7013964: openjdk LICENSE file needs rebranding Reviewed-by: darcy, katleman, jjg ! LICENSE

hg: jdk7/build/jaxp: 7013964: openjdk LICENSE file needs rebranding

2011-02-16 Thread kelly . ohair
Changeset: a57220f22751 Author:ohair Date: 2011-02-16 13:29 -0800 URL: http://hg.openjdk.java.net/jdk7/build/jaxp/rev/a57220f22751 7013964: openjdk LICENSE file needs rebranding Reviewed-by: darcy, katleman, jjg ! LICENSE

hg: jdk7/build/jaxws: 7013964: openjdk LICENSE file needs rebranding

2011-02-16 Thread kelly . ohair
Changeset: 9e3904866cab Author:ohair Date: 2011-02-16 13:29 -0800 URL: http://hg.openjdk.java.net/jdk7/build/jaxws/rev/9e3904866cab 7013964: openjdk LICENSE file needs rebranding Reviewed-by: darcy, katleman, jjg ! LICENSE

hg: jdk7/build/jdk: 7013964: openjdk LICENSE file needs rebranding

2011-02-16 Thread kelly . ohair
Changeset: 0355c60c2da4 Author:ohair Date: 2011-02-16 13:29 -0800 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/0355c60c2da4 7013964: openjdk LICENSE file needs rebranding Reviewed-by: darcy, katleman, jjg ! LICENSE

hg: jdk7/build/langtools: 7013964: openjdk LICENSE file needs rebranding

2011-02-16 Thread kelly . ohair
Changeset: 22ea7d483794 Author:ohair Date: 2011-02-16 13:30 -0800 URL: http://hg.openjdk.java.net/jdk7/build/langtools/rev/22ea7d483794 7013964: openjdk LICENSE file needs rebranding Reviewed-by: darcy, katleman, jjg ! LICENSE

hg: jdk7/deploy/corba: 4 new changesets

2011-02-16 Thread igor . nekrestyanov
Changeset: 30ecf5c90a30 Author:mfang Date: 2011-02-10 11:07 -0800 URL: http://hg.openjdk.java.net/jdk7/deploy/corba/rev/30ecf5c90a30 7014477: pt_BR corba resource bundle is missing in jdk7 build Reviewed-by: ohair ! make/common/Defs.gmk Changeset: c08dff674e53 Author:mfang Dat

hg: jdk7/deploy/jdk: 8 new changesets

2011-02-16 Thread igor . nekrestyanov
Changeset: 7bb09178ffc7 Author:ohair Date: 2011-02-10 20:45 -0800 URL: http://hg.openjdk.java.net/jdk7/deploy/jdk/rev/7bb09178ffc7 7012644: Regression: jdk/make/common/shared/Defs-windows.gmk has problems on cygwin 7018835: Debug build issues in jdk makefiles Reviewed-by: ksrini !

hg: jdk7/deploy/langtools: 4 new changesets

2011-02-16 Thread igor . nekrestyanov
Changeset: a21c7f194d31 Author:mfang Date: 2011-02-10 16:51 -0800 URL: http://hg.openjdk.java.net/jdk7/deploy/langtools/rev/a21c7f194d31 7017734: jdk7 message drop 1 translation integration Reviewed-by: ogino, yhuang ! src/share/classes/com/sun/tools/apt/resources/apt_ja.properties

hg: jdk7/build/jdk: 7010594: Add /SAFESEH to links on windows to verify safe exceptions

2011-02-16 Thread kelly . ohair
Changeset: 9b2631288894 Author:ohair Date: 2011-02-16 14:33 -0800 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/9b2631288894 7010594: Add /SAFESEH to links on windows to verify safe exceptions Reviewed-by: alanb ! make/common/Defs-windows.gmk ! make/common/shared/Defs-windows.

Request for review of "Nice report of build times"

2011-02-16 Thread Fredrik Öhrström
As part of our future work in speeding up the build process of OpenJDK I think it is necessary that it is easy to see the actual build times. This is a suggested patch that reports the build times for the subrepos in a nice manner after the build is complete. It looks like this: # Build time

Re: Request for review of "Nice report of build times"

2011-02-16 Thread mark . reinhold
> Date: Wed, 16 Feb 2011 17:19:43 +0100 > From: fredrik.ohrst...@oracle.com > As part of our future work in speeding up the build process of OpenJDK > I think it is necessary that it is easy to see the actual build times. > > This is a suggested patch that reports the build times for the subrepos