On 3/5/12 5:13 PM, Kurchi Hazra wrote:
jdk8 build is breaking if SKIP_BOOT_CYCLE is set to false due to some changes 
made
as a part of warnings cleanup in sun.rmi.* (see 7146763) that are incompatible
with the
code in the corba respository. This CR is simply to revert back the changes
made to files
in sun/rmi/rmic for the time being.

Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7151348 (Not available
yet)

Webrev: http://cr.openjdk.java.net/~khazra/7151348/webrev.00/

OK, looks good.

(Of course, that's what I said during the last round of review, which ended up breaking the build.) :-)

Actually I think I see the problem. In the original changeset, sun/rmi/rmic/Main.java was changed as follows:

- environmentClass = envClass;
+ environmentClass = BatchEnvironment.class.asSubclass(environmentClass);

This should have been

+ environmentClass = BatchEnvironment.class.asSubclass(envClass);

I don't know for sure that this causes the problem but it sure seems suspicious.

Anyway, go ahead push these changes as they stand. We'll work on the modified rmic changes later, and push them after having done more boot cycle testing.

Thanks!

s'marks

Reply via email to