One more thing: src/solaris/classes/sun/awt/X11/XWM.java contains a
patch from bsd-port [1] which is unrelated to Mac OS or 7u4-specific
changes. I think that this change might have been added when porting
from macosx-port repo to the 7u-dev one. On the one hand, the change
itself is somewhat harmless, on the other hand we don't have a
corresponding CR for this change...
I think there might be some other pieces from the bsd port that don't
actually belong to the JDK mainline.
[1] http://hg.openjdk.java.net/bsd-port/bsd-port/jdk/rev/aa821b5df19d
--
best regards,
Anthony
On 2/23/2012 4:44 PM, Michael McMahon wrote:
Anthony,
Yes, sorry about that, when checking the webrev before sending it, I
concentrated on
the modified files, rather than the new ones. So, any new files in the
webrev that are
not clearly macos related need to be treated with some suspicion. I'll
probably
be generating a new webrev soon with changes to jprt configuration and
the langtools
changes mentioned by Jim, but let me know anything else you notice in
the meantime.
Thanks,
Michael
On 23/02/12 12:27, Anthony Petrov wrote:
Hi Michael,
I've noticed your webrev adds files belonging to the legacy MToolkit
(e.g. src/solaris/native/sun/awt/awt_*). These files have been removed
explicitly from JDK 8 with a fix for 6996291.
I think we should examine the list of hg add'ed files carefully to see
if they were really added in 7u-dev or not.
--
best regards,
Anthony
On 2/23/2012 1:50 AM, Michael McMahon wrote:
This is the webrev for the Mac OS changes for JDK8. It is pretty much
a straight merge of the current jdk7u-dev changes. It builds and
passes most
JCK and regression tests with only a small number of failures.
http://cr.openjdk.java.net/~michaelm/7113349/jdk8/webrev.1/
There are three separate webrevs at the link above for the three
separate
repositories affected (the top-level, corba and jdk). Obviously most
of the changes
are in the jdk webrev. Comments welcome!
Thanks,
Michael.
