Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(Apple Silicon) aarch64 OpenJDK detect and install #273

Merged
merged 8 commits into from
Mar 7, 2023
Merged
Changes from 5 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 20 additions & 2 deletions app/assets/js/assetguard.js
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,8 @@ const ConfigManager = require('./configmanager')
const DistroManager = require('./distromanager')
const isDev = require('./isdev')

const isARM64 = process.arch === "arm64";
dscalzi marked this conversation as resolved.
Show resolved Hide resolved

// Classes

/** Class representing a base asset. */
Expand Down Expand Up @@ -150,6 +152,7 @@ class Util {
* @param {string} desired The desired version.
* @param {string} actual The actual version.
*/

static mcVersionAtLeast(desired, actual){
const des = desired.split('.')
const act = actual.split('.')
Expand Down Expand Up @@ -302,7 +305,8 @@ class JavaGuard extends EventEmitter {
break
}

const url = `https://corretto.aws/downloads/latest/amazon-corretto-${major}-x64-${sanitizedOS}-jdk.${ext}`
const arch = this. ? 'aarch64' : 'x64'
const url = `https://corretto.aws/downloads/latest/amazon-corretto-${major}-${arch}-${sanitizedOS}-jdk.${ext}`

return new Promise((resolve, reject) => {
request.head({url, json: true}, (err, resp) => {
Expand Down Expand Up @@ -495,6 +499,8 @@ class JavaGuard extends EventEmitter {
let vendorName = props[i].split('=')[1].trim()
this.logger.debug(props[i].trim())
meta.vendor = vendorName
} else if (props[i].indexOf('os.arch') > -1) {
meta.isARM = props[i].split('=')[1].trim() === 'aarch64'
}
}

Expand Down Expand Up @@ -866,6 +872,9 @@ class JavaGuard extends EventEmitter {
* @param {string} dataDir The base launcher directory.
* @returns {Promise.<string>} A Promise which resolves to the executable path of a valid
* x64 Java installation. If none are found, null is returned.
*
* Added: On the system with ARM architecture attempts to find aarch64 Java.
*
*/
async _darwinJavaValidate(dataDir){

Expand Down Expand Up @@ -894,7 +903,16 @@ class JavaGuard extends EventEmitter {
pathArr = JavaGuard._sortValidJavaArray(pathArr)

if(pathArr.length > 0){
return pathArr[0].execPath

// TODO Revise this a bit, seems to work for now. Discovery logic should
// probably just filter out the invalid architectures before it even
// gets to this point.
if (isARM64) {
return pathArr.find(({ isARM }) => isARM)?.execPath ?? null
} else {
return pathArr.find(({ isARM }) => !isARM)?.execPath ?? null
}

} else {
return null
}
Expand Down