On 02/11/2011 07:09 AM, Dr Andrew John Hughes wrote:
On 16:38 Thu 10 Feb , Gary Meyer wrote:
On Feb 9, 2011, at 6:48 PM, Jonathan Gibbons wrote:
Build folk,
The following fix should make it possible to (again) build langtools on a Mac.
Dan Smith reports:
- When building the GenStubs in OS X, the Java 6 tools classes are in
classes.jar, and this shadows $build.bootstrap.dir, which appears in the
classpath. This can be fixed with -Xbootclasspath/p.
Webrev here:
http://cr.openjdk.java.net/~jjg/7018452/webrev/
-- Jon
I don't know if you will accept me as a code reviewer, but I'm the Java Build
Engineer at Apple. These code diffs look good to me.
Gary.
~~~~~~~~~~~~~~~~~~~~~~~
Gary Meyer
Java Software Build Engineer
[email protected]
Hi Gary,
Personally, I think it's great that someone who knows the platform
has checked over this patch.
More formally, for the commit, I think it depends on if Oracle gave
you an OpenJDK username yet :-) AFAIK, the 'Reviewed-by' field in
commits has to contain OpenJDK usernames. See the (incomplete)
developer's guide at http://openjdk.java.net/guide/producingChangeset.html
This may or may not be checked by the jcheck extension Oracle run
on commits. Given it's proprietary, we really don't know what it does.
Jon, FWIW, I also think the patch looks fine.
Hope that helps,
Andrew, Gary,
Note that Kelly asked recently that all build changes should be reviewed
by members of the build team. So while is it always acceptable to have
additional reviewers, we do need Kelly or someone else in the build
group to verify the change too.
http://mail.openjdk.java.net/pipermail/build-dev/2011-February/004034.html
In this case, Kelly has already approved the change and the changeset
has been pushed.
http://mail.openjdk.java.net/pipermail/compiler-dev/2011-February/002851.html
-- Jon
