RFR: 8253795: Implementation of JEP 391: macOS/AArch64 Port [v3]

[David Holmes]

I don't know why the Skara tools decided to associate my comment with 
Alan Hayward's comment as they are not at all related. :(

David

On 27/01/2021 4:50 pm, David Holmes wrote:
> On Tue, 26 Jan 2021 12:34:11 GMT, Alan Hayward <github.com+4146708+a74nh at openjdk.org> wrote:
> 
>>>> AIUI, the configure line needs passing a prebuilt JavaNativeFoundation framework
>>>> ie:
>>>> `--with-extra-ldflags='-F /Applications/Xcode.app/Contents/SharedFrameworks/ContentDeliveryServices.framework/Versions/A/itms/java/Frameworks/'`
>>>>
>>>> Otherwise there will be missing _JNFNative* functions.
>>>>
>>>> Is this the long term plan? Or will eventually the required code be moved into JDK and/or the xcode one automatically get picked up by the configure scripts?
>>>
>>> There is ongoing work by P. Race to eliminate dependence on JNF at all
>>
>>>> AIUI, the configure line needs passing a prebuilt JavaNativeFoundation framework
>>>> ie:
>>>> `--with-extra-ldflags='-F /Applications/Xcode.app/Contents/SharedFrameworks/ContentDeliveryServices.framework/Versions/A/itms/java/Frameworks/'`
>>>> Otherwise there will be missing _JNFNative* functions.
>>>> Is this the long term plan? Or will eventually the required code be moved into JDK and/or the xcode one automatically get picked up by the configure scripts?
>>>
>>> There is ongoing work by P. Race to eliminate dependence on JNF at all
>>
>> Ok, that's great.
>> In the meantime is it worth adding something to the MacOS section of doc/building.* ?
>> (I pieced together the required line from multiple posts in a mailing list)
> 
> Hi Anton,
> 
> Just to add weight to comments already made by @coleenp and @stefank , I also find the W^X coding support to be too intrusive and polluting of the shared code. I would much rather see this support pushed down as far as possible, to minimise the impact and to use ifdef'd code for macos/Aarch64 (via MACOS_AARCH64_ONLY macro) rather than providing empty methods.
> 
> Thanks,
> David
> 
> -------------
> 
> PR: https://git.openjdk.java.net/jdk/pull/2200
>