Review Request: JDK-8011602 - jobjc build failure on Mac

Dan Xu dan.xu at
Fri Apr 5 16:21:47 PDT 2013

Right. Even the generated native headers from jobjc are saved into its 
own directory.

Thank you for your review!


On 04/05/2013 04:15 PM, Mandy Chung wrote:
> Thumbs up.  Thanks for fixing it quickly.  The macosx jobjc build has 
> its own special build logic that brought some surprise when I merged 
> it with jigsaw at one time.
> Mandy
> On 4/5/2013 3:35 PM, Dan Xu wrote:
>> Hi All,
>> Please review the change to fix the build issue on mac platformat 
>> It removes the 
>> un-necessary @Native annotation from 
>> src/macosx/native/jobjc/src/core/java/com/apple/jobjc/ 
>> Therefore, the objc compilation will not dependon the new 
>> java.lang.annotation.Native interfacethat is only introduced in jdk8.
>> For the normal build process, even though @Native annotationis added 
>> to many other java classes to replace @GenerateNativeHeader, the 
>> langtool has the capability to handle that when building with jdk7. 
>> But on Mac platform, objc compilation is a special process, where the 
>> magic of langtool to handle new jdk8 interface in old jdk7 cannot be 
>> applied. Since already has a native method, 
>> getNativeFFITypePtrForCode(), declared, @Native is not necessary to 
>> be present, and it is safe to remove them. The other change in file, 
>> src/share/classes/sun/java2d/opengl/, is only a format 
>> correction. I have tested this fix in jprt with boot jdk as jdk7 
>> across all platforms. Thanks!
>> webreve:
>> -Dan

More information about the macosx-port-dev mailing list