<AWT Dev> [9] Request for Review: 8003900: X11 dependencies should be removed from Mac OS X build.
Artem Ananiev
artem.ananiev at oracle.com
Fri May 16 07:57:13 UTC 2014
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-
>>>
>>
>
More information about the awt-dev
mailing list