hg: jdk7/build/langtools: 32 new changesets

2009-08-18 Thread xiomara . jayasena
Changeset: 4ac89259512f Author:xdono Date: 2009-08-13 12:11 -0700 URL: http://hg.openjdk.java.net/jdk7/build/langtools/rev/4ac89259512f Added tag jdk7-b69 for changeset ce9bcdcb7859 ! .hgtags Changeset: 777a3efad0d5 Author:jjg Date: 2009-07-28 10:36 -0700 URL: http:

hg: jdk7/build/jdk: 86 new changesets

2009-08-18 Thread xiomara . jayasena
Changeset: 36c8ddbe9bc5 Author:xdono Date: 2009-08-13 12:11 -0700 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/36c8ddbe9bc5 Added tag jdk7-b69 for changeset 226b20019b1f ! .hgtags Changeset: 12e479399ced Author:dl Date: 2009-07-28 13:24 -0700 URL: http://hg.op

hg: jdk7/build/jaxp: 11 new changesets

2009-08-18 Thread xiomara . jayasena
Changeset: 8287833daabc Author:xdono Date: 2009-08-13 12:11 -0700 URL: http://hg.openjdk.java.net/jdk7/build/jaxp/rev/8287833daabc Added tag jdk7-b69 for changeset a4ab0d6ded63 ! .hgtags Changeset: 1687f5192ce7 Author:asaha Date: 2009-06-22 13:56 -0700 URL: http://h

hg: jdk7/build/hotspot: 4 new changesets

2009-08-18 Thread xiomara . jayasena
Changeset: 5021b9893d0a Author:xdono Date: 2009-08-13 12:11 -0700 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/5021b9893d0a Added tag jdk7-b69 for changeset 54fd4d923296 ! .hgtags Changeset: 185d256992c3 Author:asaha Date: 2009-08-07 11:31 -0700 URL: http:

hg: jdk7/build/corba: 4 new changesets

2009-08-18 Thread xiomara . jayasena
Changeset: 07b3e9ba5085 Author:xdono Date: 2009-08-13 12:11 -0700 URL: http://hg.openjdk.java.net/jdk7/build/corba/rev/07b3e9ba5085 Added tag jdk7-b69 for changeset 8120d308ec4e ! .hgtags Changeset: f3f572ea0cf2 Author:asaha Date: 2009-08-07 11:31 -0700 URL: http://

hg: jdk7/build: 4 new changesets

2009-08-18 Thread xiomara . jayasena
Changeset: 51a71c2c4b80 Author:xdono Date: 2009-08-13 12:11 -0700 URL: http://hg.openjdk.java.net/jdk7/build/rev/51a71c2c4b80 Added tag jdk7-b69 for changeset 82e6c820c51a ! .hgtags Changeset: 4cad5a3f5038 Author:asaha Date: 2009-08-07 11:31 -0700 URL: http://hg.ope

Re: [PATCH FOR REVIEW]: Make source/target options explicit for JDK bootstrap tools

2009-08-18 Thread Andrew John Hughes
2009/8/18 Kelly O'Hair : > 6873059: Explicitly use -source 6 -target 6 when compiling with the boot jdk > javac > > You can use this same bugid with multiple repositories. > Thanks. Pushed: http://hg.openjdk.java.net/jdk7/build/jdk/rev/80368890a2a0 I'll push the other three patches under this bu

hg: jdk7/build/jdk: 6873059: Explicitly use -source 6 -target 6 when compiling with the boot jdk javac

2009-08-18 Thread ahughes
Changeset: 80368890a2a0 Author:andrew Date: 2009-08-18 19:50 +0100 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/80368890a2a0 6873059: Explicitly use -source 6 -target 6 when compiling with the boot jdk javac Summary: The bootstrap javac currently uses the default source and t

Re: [PATCH FOR REVIEW]: Make source/target options explicit for JDK bootstrap tools

2009-08-18 Thread Kelly O'Hair
6873059: Explicitly use -source 6 -target 6 when compiling with the boot jdk javac You can use this same bugid with multiple repositories. -kto Andrew John Hughes wrote: 2009/8/18 Kelly O'Hair : This one looks fine to me. -kto Andrew John Hughes wrote: 2009/8/18 Andrew John Hughes : Thi

Re: [PATCH FOR REVIEW]: Make source/target options explicit for JDK bootstrap tools

2009-08-18 Thread Andrew John Hughes
2009/8/18 Kelly O'Hair : > This one looks fine to me. > > -kto > > > Andrew John Hughes wrote: >> >> 2009/8/18 Andrew John Hughes : >>> >>> This is pretty much the same change as for CORBA, but for JDK: >>> >>> http://cr.openjdk.java.net/~andrew/ecj/04/webrev.01/ >>> >>> BOOT_SOURCE_LANGUAGE_VERSIO

Re: [PATCH FOR REVIEW]: Make source/target options explicit for JDK bootstrap tools

2009-08-18 Thread Kelly O'Hair
This one looks fine to me. -kto Andrew John Hughes wrote: 2009/8/18 Andrew John Hughes : This is pretty much the same change as for CORBA, but for JDK: http://cr.openjdk.java.net/~andrew/ecj/04/webrev.01/ BOOT_SOURCE_LANGUAGE_VERSION and BOOT_TARGET_CLASS_VERSION are added and set to 6 by d

Re: [PATCH FOR REVIEW]: Make source/target options explicit for JDK bootstrap tools

2009-08-18 Thread Andrew John Hughes
2009/8/18 Andrew John Hughes : > This is pretty much the same change as for CORBA, but for JDK: > > http://cr.openjdk.java.net/~andrew/ecj/04/webrev.01/ > > BOOT_SOURCE_LANGUAGE_VERSION and BOOT_TARGET_CLASS_VERSION are added > and set to 6 by default.  They are then used by the bootstrap javac. >

[PATCH FOR REVIEW]: Make source/target options explicit for JDK bootstrap tools

2009-08-18 Thread Andrew John Hughes
This is pretty much the same change as for CORBA, but for JDK: http://cr.openjdk.java.net/~andrew/ecj/04/webrev.01/ BOOT_SOURCE_LANGUAGE_VERSION and BOOT_TARGET_CLASS_VERSION are added and set to 6 by default. They are then used by the bootstrap javac. -- Andrew :-) Free Java Software Engineer

[PATCH FOR REVIEW]: Make source/target options explicit for JDK bootstrap tools

2009-08-18 Thread Andrew John Hughes
This is pretty much the same change as for CORBA, but for JDK: http://cr.openjdk.java.net/~andrew/ecj/04/webrev.01/ BOOT_SOURCE_LANGUAGE_VERSION and BOOT_TARGET_CLASS_VERSION are added and set to 6 by default. They are then used by the bootstrap javac. -- Andrew :-) Free Java Software Engineer

Re: [PATCH FOR REVIEW]: Make source/target options explicit for CORBA bootstrap tools

2009-08-18 Thread Andrew John Hughes
2009/8/18 Jonathan Gibbons : > Andrew, > > If this is a patch for jdk7, it does not appear to be a patch to a recent > copy > of 7. It's against b69 which is the latest release (from Friday). The patches are against the IcedTea forest so builds can be tested with IcedTea as well. Specifically, y

Re: [PATCH FOR REVIEW]: Make source/target options explicit for CORBA bootstrap tools

2009-08-18 Thread Kelly O'Hair
I also prefer Jon's suggested change. I was going to suggest the same thing. -kto Jonathan Gibbons wrote: Andrew, If this is a patch for jdk7, it does not appear to be a patch to a recent copy of 7. Specifically, you do not seem to have the recent changeset to set the source/target used to

Re: [PATCH FOR REVIEW]: Make source/target options explicit for CORBA bootstrap tools

2009-08-18 Thread Jonathan Gibbons
Andrew, If this is a patch for jdk7, it does not appear to be a patch to a recent copy of 7. Specifically, you do not seem to have the recent changeset to set the source/target used to compile JDK to 7. [1] While your patch does not directly conflict with any edits in that patch, and while

[PATCH FOR REVIEW]: Make source/target options explicit for CORBA bootstrap tools

2009-08-18 Thread Andrew John Hughes
Currently the javac calls for building the bootstrap tools (not the classes for the final JDK, which correctly now use source and target 7) don't set an explicit source and target version. The webrev: http://cr.openjdk.java.net/~andrew/ecj/02/webrev.01/ sets these to 6 explicitly, as happens in