Apple Silicon support: Allowing arm64 builds for Mac. Mostly removing…

… hardcoded values and making sure user-options are utilized for platform/dmg packagers. Updated tests to target electron 11. Updated mac snapshots to include multi-arch images
electron-userland · Nov 26, 2020 · 1acc32e · 1acc32e
1 parent 982b073
commit 1acc32e
Show file tree

Hide file tree

Showing 23 changed files with 160 additions and 139 deletions.
diff --git a/.circleci/config.yml b/.circleci/config.yml
@@ -11,7 +11,7 @@ jobs:
             - install-dep-cache-{{ checksum "yarn.lock" }}
       - restore_cache:
           keys:
-            - v-8.2.5-electron
+            - v-11.0.0-electron
       - run:
           command: node .yarn/releases/yarn-2.2.2.cjs install
       - run:
@@ -25,7 +25,7 @@ jobs:
       - run:
           command: node .yarn/releases/yarn-2.2.2.cjs node ./test/out/helpers/downloadElectron.js
       - save_cache:
-          key: v-8.2.5-electron
+          key: v-11.0.0-electron
           paths:
             - ~/.cache/electron
 
@@ -43,7 +43,7 @@ jobs:
             - install-dep-cache-{{ checksum "yarn.lock" }}
       - restore_cache:
           keys:
-            - v-8.2.5-electron
+            - v-11.0.0-electron
       # because in the build job we use circleci docker image and circleci restores cache to original user home
       - run:
           command: |

diff --git a/packages/app-builder-lib/src/core.ts b/packages/app-builder-lib/src/core.ts
@@ -37,9 +37,6 @@ export class Platform {
     }
 
     const archToType = new Map()
-    if (this === Platform.MAC) {
-      archs = [Arch.x64]
-    }
 
     for (const arch of (archs == null || archs.length === 0 ? [archFromString(process.arch)] : archs)) {
       archToType.set(arch, type == null ? [] : (Array.isArray(type) ? type : [type]))

diff --git a/packages/app-builder-lib/src/macPackager.ts b/packages/app-builder-lib/src/macPackager.ts
@@ -100,7 +100,7 @@ export default class MacPackager extends PlatformPackager<MacConfiguration> {
     if (!hasMas || targets.length > 1) {
       const appPath = prepackaged == null ? path.join(this.computeAppOutDir(outDir, arch), `${this.appInfo.productFilename}.app`) : prepackaged
       nonMasPromise = (prepackaged ? Promise.resolve() : this.doPack(outDir, path.dirname(appPath), this.platform.nodeName as ElectronPlatformName, arch, this.platformSpecificBuildOptions, targets))
-        .then(() => this.packageInDistributableFormat(appPath, Arch.x64, targets, taskManager))
+        .then(() => this.packageInDistributableFormat(appPath, arch, targets, taskManager))
     }
 
     for (const target of targets) {

diff --git a/packages/app-builder-lib/src/packager.ts b/packages/app-builder-lib/src/packager.ts
@@ -185,10 +185,6 @@ export class Packager {
 
     function processTargets(platform: Platform, types: Array<string>) {
       function commonArch(currentIfNotSpecified: boolean): Array<Arch> {
-        if (platform === Platform.MAC) {
-          return currentIfNotSpecified ? [Arch.x64] : []
-        }
-
         const result = Array<Arch>()
         return result.length === 0 && currentIfNotSpecified ? [archFromString(process.arch)] : result
       }

diff --git a/packages/app-builder-lib/src/platformPackager.ts b/packages/app-builder-lib/src/platformPackager.ts
@@ -487,7 +487,7 @@ export abstract class PlatformPackager<DC extends PlatformSpecificBuildOptions>
 
   private computeArtifactName(pattern: any, ext: string, arch: Arch | null | undefined): string {
     const archName = arch == null ? null : getArtifactArchName(arch, ext)
-    return this.expandMacro(pattern, this.platform === Platform.MAC ? null : archName, {
+    return this.expandMacro(pattern, archName, {
       ext
     })
   }

diff --git a/packages/app-builder-lib/src/targets/targetFactory.ts b/packages/app-builder-lib/src/targets/targetFactory.ts
@@ -13,7 +13,7 @@ export function computeArchToTargetNamesMap(raw: Map<Arch, Array<string>>, platf
     }
   }
 
-  const defaultArchs: Array<ArchType> = raw.size === 0 ? [platform === Platform.MAC ? "x64" : process.arch as ArchType] : Array.from(raw.keys()).map(it => Arch[it] as ArchType)
+  const defaultArchs: Array<ArchType> = raw.size === 0 ? [process.arch as ArchType] : Array.from(raw.keys()).map(it => Arch[it] as ArchType)
   const result = new Map(raw)
   for (const target of asArray(platformPackager.platformSpecificBuildOptions.target).map<TargetConfiguration>(it => typeof it === "string" ? {target: it} : it)) {
     let name = target.target

diff --git a/packages/dmg-builder/src/dmg.ts b/packages/dmg-builder/src/dmg.ts
@@ -3,7 +3,7 @@ import { findIdentity, isSignAllowed } from "app-builder-lib/out/codeSign/macCod
 import MacPackager from "app-builder-lib/out/macPackager"
 import { createBlockmap } from "app-builder-lib/out/targets/differentialUpdateInfoBuilder"
 import { executeAppBuilderAsJson } from "app-builder-lib/out/util/appBuilder"
-import { Arch, AsyncTaskManager, exec, InvalidConfigurationError, isEmptyOrSpaces, log, spawn } from "builder-util"
+import { Arch, AsyncTaskManager, exec, getArchSuffix, InvalidConfigurationError, isEmptyOrSpaces, log, spawn } from "builder-util"
 import { CancellationToken } from "builder-util-runtime"
 import { copyDir, copyFile, exists, statOrNull } from "builder-util/out/fs"
 import { stat } from "fs-extra"
@@ -23,15 +23,15 @@ export class DmgTarget extends Target {
   async build(appPath: string, arch: Arch) {
     const packager = this.packager
     // tslint:disable-next-line:no-invalid-template-strings
-    const artifactName = packager.expandArtifactNamePattern(packager.config.dmg, "dmg", null, "${productName}-" + (packager.platformSpecificBuildOptions.bundleShortVersion || "${version}") + ".${ext}")
+    const artifactName = packager.expandArtifactNamePattern(this.options, "dmg", arch, "${productName}-" + (packager.platformSpecificBuildOptions.bundleShortVersion || "${version}") + "-${arch}.${ext}", true)
     const artifactPath = path.join(this.outDir, artifactName)
     await packager.info.callArtifactBuildStarted({
       targetPresentableName: "DMG",
       file: artifactPath,
       arch,
     })
 
-    const volumeName = sanitizeFileName(this.computeVolumeName(this.options.title))
+    const volumeName = sanitizeFileName(this.computeVolumeName(arch, this.options.title))
 
     const tempDmg = await createStageDmg(await packager.getTempFile(".dmg"), appPath, volumeName)
 
@@ -115,15 +115,17 @@ export class DmgTarget extends Target {
     await exec("codesign", args)
   }
 
-  computeVolumeName(custom?: string | null): string {
+  computeVolumeName(arch: Arch, custom?: string | null): string {
     const appInfo = this.packager.appInfo
     const shortVersion = this.packager.platformSpecificBuildOptions.bundleShortVersion || appInfo.version
+    const archString = getArchSuffix(arch)
 
     if (custom == null) {
-      return `${appInfo.productFilename} ${shortVersion}`
+      return `${appInfo.productFilename} ${shortVersion}${archString}`
     }
 
     return custom
+      .replace(/\${arch}/g, archString)
       .replace(/\${shortVersion}/g, shortVersion)
       .replace(/\${version}/g, appInfo.version)
       .replace(/\${name}/g, appInfo.name)

diff --git a/packages/electron-builder/src/builder.ts b/packages/electron-builder/src/builder.ts
@@ -33,10 +33,6 @@ export function normalizeOptions(args: CliOptions): BuildOptions {
 
   function processTargets(platform: Platform, types: Array<string>) {
     function commonArch(currentIfNotSpecified: boolean): Array<Arch> {
-      if (platform === Platform.MAC) {
-        return args.x64 || currentIfNotSpecified ? [Arch.x64] : []
-      }
-
       const result = Array<Arch>()
       if (args.x64) {
         result.push(Arch.x64)
@@ -202,7 +198,7 @@ export function coerceTypes(host: any): any {
 export function createTargets(platforms: Array<Platform>, type?: string | null, arch?: string | null): Map<Platform, Map<Arch, Array<string>>> {
   const targets = new Map<Platform, Map<Arch, Array<string>>>()
   for (const platform of platforms) {
-    const archs = platform === Platform.MAC ? [Arch.x64] : (arch === "all" ? [Arch.x64, Arch.ia32] : [archFromString(arch == null ? process.arch : arch)])
+    const archs = (arch === "all" ? (platform === Platform.MAC ? [Arch.x64, Arch.arm64] : [Arch.x64, Arch.ia32]) : [archFromString(arch == null ? process.arch : arch)])
     const archToType = new Map<Arch, Array<string>>()
     targets.set(platform, archToType)
 

diff --git a/test/fixtures/test-app-build-sub/electron-builder.yml b/test/fixtures/test-app-build-sub/electron-builder.yml
@@ -1,4 +1,4 @@
-electronVersion: 8.2.5
+electronVersion: 11.0.0
 appId: org.electron-builder.testApp
 compression: store
 npmRebuild: false

diff --git a/test/fixtures/test-app-one/package.json b/test/fixtures/test-app-one/package.json
@@ -8,7 +8,7 @@
   "author": "Foo Bar <foo@example.com>",
   "license": "MIT",
   "build": {
-    "electronVersion": "8.2.5",
+    "electronVersion": "11.0.0",
     "appId": "org.electron-builder.testApp",
     "compression": "store",
     "npmRebuild": false,

diff --git a/test/fixtures/test-app-yarn-several-workspace/packages/test-app/package.json b/test/fixtures/test-app-yarn-several-workspace/packages/test-app/package.json
@@ -7,7 +7,7 @@
   "author": "Foo Bar <foo@example.com>",
   "license": "MIT",
   "build": {
-    "electronVersion": "8.2.5",
+    "electronVersion": "11.0.0",
     "appId": "org.electron-builder.testApp",
     "compression": "store",
     "npmRebuild": false,

diff --git a/test/fixtures/test-app-yarn-workspace-version-conflict/packages/test-app/package.json b/test/fixtures/test-app-yarn-workspace-version-conflict/packages/test-app/package.json
@@ -7,7 +7,7 @@
   "author": "Foo Bar <foo@example.com>",
   "license": "MIT",
   "build": {
-    "electronVersion": "8.2.5",
+    "electronVersion": "11.0.0",
     "appId": "org.electron-builder.testApp",
     "compression": "store",
     "npmRebuild": false,

diff --git a/test/fixtures/test-app-yarn-workspace/packages/test-app/package.json b/test/fixtures/test-app-yarn-workspace/packages/test-app/package.json
@@ -7,7 +7,7 @@
   "author": "Foo Bar <foo@example.com>",
   "license": "MIT",
   "build": {
-    "electronVersion": "8.2.5",
+    "electronVersion": "11.0.0",
     "appId": "org.electron-builder.testApp",
     "compression": "store",
     "npmRebuild": false,

diff --git a/test/fixtures/test-app/package.json b/test/fixtures/test-app/package.json
@@ -1,7 +1,7 @@
 {
   "private": true,
   "build": {
-    "electronVersion": "8.2.5",
+    "electronVersion": "11.0.0",
     "appId": "org.electron-builder.testApp",
     "compression": "store",
     "npmRebuild": false,

diff --git a/test/snapshots/PublishManagerTest.js.snap b/test/snapshots/PublishManagerTest.js.snap
@@ -169,6 +169,7 @@ Object {
   "CFBundlePackageType": "APPL",
   "CFBundleShortVersionString": "1.1.0",
   "LSApplicationCategoryType": "your.app.category.type",
+  "LSRequiresNativeExecution": true,
   "NSAppTransportSecurity": Object {
     "NSAllowsLocalNetworking": true,
     "NSExceptionDomains": Object {
@@ -273,6 +274,7 @@ Object {
   "CFBundlePackageType": "APPL",
   "CFBundleShortVersionString": "1.1.0",
   "LSApplicationCategoryType": "your.app.category.type",
+  "LSRequiresNativeExecution": true,
   "NSAppTransportSecurity": Object {
     "NSAllowsLocalNetworking": true,
     "NSExceptionDomains": Object {
@@ -349,6 +351,7 @@ Object {
   "CFBundlePackageType": "APPL",
   "CFBundleShortVersionString": "1.1.0",
   "LSApplicationCategoryType": "your.app.category.type",
+  "LSRequiresNativeExecution": true,
   "NSAppTransportSecurity": Object {
     "NSAllowsLocalNetworking": true,
     "NSExceptionDomains": Object {

diff --git a/test/snapshots/filesTest.js.snap b/test/snapshots/filesTest.js.snap
@@ -53,6 +53,7 @@ Array [
   "lib/net45/v8_context_snapshot.bin",
   "lib/net45/vk_swiftshader.dll",
   "lib/net45/vk_swiftshader_icd.json",
+  "lib/net45/vulkan-1.dll",
   "lib/net45/locales/en-US.pak",
   "lib/net45/resources/app.asar",
   "lib/net45/resources/platformSpecificR",

diff --git a/test/snapshots/linux/debTest.js.snap b/test/snapshots/linux/debTest.js.snap
@@ -24,12 +24,12 @@ Array [
   "/opt/Test App ßW/chrome-sandbox",
   "/opt/Test App ßW/chrome_100_percent.pak",
   "/opt/Test App ßW/chrome_200_percent.pak",
-  "/opt/Test App ßW/crashpad_handler",
   "/opt/Test App ßW/icudtl.dat",
   "/opt/Test App ßW/libEGL.so",
   "/opt/Test App ßW/libffmpeg.so",
   "/opt/Test App ßW/libGLESv2.so",
   "/opt/Test App ßW/libvk_swiftshader.so",
+  "/opt/Test App ßW/libvulkan.so",
   "/opt/Test App ßW/LICENSE.electron.txt",
   "/opt/Test App ßW/LICENSES.chromium.html",
   "/opt/Test App ßW/resources.pak",
@@ -99,12 +99,12 @@ Array [
   "/opt/Test App ßW/chrome-sandbox",
   "/opt/Test App ßW/chrome_100_percent.pak",
   "/opt/Test App ßW/chrome_200_percent.pak",
-  "/opt/Test App ßW/crashpad_handler",
   "/opt/Test App ßW/icudtl.dat",
   "/opt/Test App ßW/libEGL.so",
   "/opt/Test App ßW/libffmpeg.so",
   "/opt/Test App ßW/libGLESv2.so",
   "/opt/Test App ßW/libvk_swiftshader.so",
+  "/opt/Test App ßW/libvulkan.so",
   "/opt/Test App ßW/LICENSE.electron.txt",
   "/opt/Test App ßW/LICENSES.chromium.html",
   "/opt/Test App ßW/resources.pak",
@@ -186,12 +186,12 @@ Array [
   "/opt/Test App ßW/chrome-sandbox",
   "/opt/Test App ßW/chrome_100_percent.pak",
   "/opt/Test App ßW/chrome_200_percent.pak",
-  "/opt/Test App ßW/crashpad_handler",
   "/opt/Test App ßW/icudtl.dat",
   "/opt/Test App ßW/libEGL.so",
   "/opt/Test App ßW/libffmpeg.so",
   "/opt/Test App ßW/libGLESv2.so",
   "/opt/Test App ßW/libvk_swiftshader.so",
+  "/opt/Test App ßW/libvulkan.so",
   "/opt/Test App ßW/LICENSE.electron.txt",
   "/opt/Test App ßW/LICENSES.chromium.html",
   "/opt/Test App ßW/resources.pak",
@@ -271,12 +271,12 @@ Array [
   "/opt/Test App ßW/chrome-sandbox",
   "/opt/Test App ßW/chrome_100_percent.pak",
   "/opt/Test App ßW/chrome_200_percent.pak",
-  "/opt/Test App ßW/crashpad_handler",
   "/opt/Test App ßW/icudtl.dat",
   "/opt/Test App ßW/libEGL.so",
   "/opt/Test App ßW/libffmpeg.so",
   "/opt/Test App ßW/libGLESv2.so",
   "/opt/Test App ßW/libvk_swiftshader.so",
+  "/opt/Test App ßW/libvulkan.so",
   "/opt/Test App ßW/LICENSE.electron.txt",
   "/opt/Test App ßW/LICENSES.chromium.html",
   "/opt/Test App ßW/resources.pak",
@@ -357,12 +357,12 @@ Array [
   "/opt/Test App ßW/chrome-sandbox",
   "/opt/Test App ßW/chrome_100_percent.pak",
   "/opt/Test App ßW/chrome_200_percent.pak",
-  "/opt/Test App ßW/crashpad_handler",
   "/opt/Test App ßW/icudtl.dat",
   "/opt/Test App ßW/libEGL.so",
   "/opt/Test App ßW/libffmpeg.so",
   "/opt/Test App ßW/libGLESv2.so",
   "/opt/Test App ßW/libvk_swiftshader.so",
+  "/opt/Test App ßW/libvulkan.so",
   "/opt/Test App ßW/LICENSE.electron.txt",
   "/opt/Test App ßW/LICENSES.chromium.html",
   "/opt/Test App ßW/resources.pak",
@@ -471,12 +471,12 @@ Array [
   "/opt/foo/chrome-sandbox",
   "/opt/foo/chrome_100_percent.pak",
   "/opt/foo/chrome_200_percent.pak",
-  "/opt/foo/crashpad_handler",
   "/opt/foo/icudtl.dat",
   "/opt/foo/libEGL.so",
   "/opt/foo/libffmpeg.so",
   "/opt/foo/libGLESv2.so",
   "/opt/foo/libvk_swiftshader.so",
+  "/opt/foo/libvulkan.so",
   "/opt/foo/LICENSE.electron.txt",
   "/opt/foo/LICENSES.chromium.html",
   "/opt/foo/resources.pak",