RFR: 8257758: Allow building of JavaFX native libs for Apple Silicon
Kevin Rushforth
kcr at openjdk.java.net
Tue Dec 8 21:08:36 UTC 2020
On Tue, 8 Dec 2020 11:00:43 GMT, Johan Vos <jvos at openjdk.org> wrote:
> Fix for JDK-8257758
>
> In case the -PtargetArch=arm64 option is supplied to the gradle build, the
> native components that are part of the JavaFX build will target this platform.
I haven't tested it yet. I made a naming suggestion, but otherwise looks OK.
buildSrc/mac.gradle line 32:
> 30:
> 31: def TARGET_ARCH = "x86_64"
> 32: if (hasProperty('targetArch')) {
Normally, our external gradle properties are all upper-case, so `TARGET_ARCH` might be a better name for the property and `targetArch` a better name for the local variable.
buildSrc/mac.gradle line 130:
> 128:
> 129: if (hasProperty('targetArch')) {
> 130: commonParams = ["-target", "$TARGET_ARCH-apple-macos-11", commonParams]
Maybe surround the variable with curly braces to make it clearer?
-------------
PR: https://git.openjdk.java.net/jfx/pull/363
More information about the openjfx-dev
mailing list