<AWT Dev> [9] Request for Review: 8003900: X11 dependencies should be removed from Mac OS X build.

[Petr Pchelko]

Hello, David.

I've pushed your fix to the client repo:
http://hg.openjdk.java.net/jdk9/client/jdk/rev/56024e323104

With best regards. Petr.

On 16 мая 2014 г., at 11:57, Artem Ananiev <artem.ananiev at oracle.com> wrote:

> 
> I'm fine with the fix.
> 
> Thanks,
> 
> Artem
> 
> On 5/15/2014 9:36 PM, Petr Pchelko wrote:
>> Hello, David.
>> 
>> The fix looks good to me.
>> Artem was interested in the review, so we may want to wait for his final vote..
>> 
>> I can push the changeset for you. Please ping me when you decide it’s ready to go.
>> 
>> Thank you.
>> With best regards. Petr.
>> 
>> On May 15, 2014, at 8:23 PM, David DeHaven <david.dehaven at oracle.com> wrote:
>> 
>>> 
>>> Can I get another reviewer on this? I need a sponsor to push too.. (Anthony?)
>>> 
>>> -DrD-
>>> 
>>>> 
>>>>> src/solaris/native/sun/awt/awt.h
>>>>>> 113 #if !defined(HEADLESS) && defined(XAWT)
>>>>>> 114 extern Display         *awt_display; /* awt_GraphicsEnv.c */
>>>>> 
>>>>> You forgot to update this XAWT usage. Otherwise the fix looks fine.
>>>> 
>>>> Whoops! Updated in-place.
>>>> 
>>>> -DrD-
>>>> 
>>> 
>>