RFR: 7113349 JDK8 port to macosx
Anthony Petrov
anthony.petrov at oracle.com
Thu Feb 23 05:13:50 PST 2012
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.
>
More information about the jdk8-dev
mailing list