feat: Apple Silicon Universal Support

packages/app-builder-lib/package.json

@@ -63,7 +63,8 @@
     "read-config-file": "6.0.0",
     "sanitize-filename": "^1.6.3",
     "semver": "^7.3.4",
-    "temp-file": "^3.3.7"
+    "temp-file": "^3.3.7",
+    "@electron/universal": "1.0.4"
   },
   "///": "babel in devDependencies for proton tests",
   "devDependencies": {

packages/app-builder-lib/src/macPackager.ts

@@ -18,6 +18,7 @@ import { PkgTarget, prepareProductBuildArgs } from "./targets/pkg"
 import { createCommonTarget, NoOpTarget } from "./targets/targetFactory"
 import { isMacOsHighSierra } from "./util/macosVersion"
 import { getTemplatePath } from "./util/pathManager"
+import { promisify } from "util"
 
 export default class MacPackager extends PlatformPackager<MacConfiguration> {
   readonly codeSigningInfo = new Lazy<CodeSigningInfo>(() => {
@@ -91,6 +92,41 @@ export default class MacPackager extends PlatformPackager<MacConfiguration> {
     }
   }
 
+  protected async doPack(outDir: string, appOutDir: string, platformName: ElectronPlatformName, arch: Arch, platformSpecificBuildOptions: MacConfiguration, targets: Array<Target>): Promise<any> {
+    switch (arch) {
+      default: {
+        return super.doPack(outDir, appOutDir, this.platform.nodeName as ElectronPlatformName, arch, this.platformSpecificBuildOptions, targets);
+      }
+      case Arch.universal: {
+        const x64Arch = Arch.x64;
+        const x64AppOutDir = appOutDir + '-' + Arch[x64Arch];
+        await super.doPack(outDir, x64AppOutDir, platformName, x64Arch, platformSpecificBuildOptions, targets, false);
+        const arm64Arch = Arch.arm64;
+        const arm64AppOutPath = appOutDir + '-' + Arch[arm64Arch];
+        await super.doPack(outDir, arm64AppOutPath, platformName, arm64Arch, platformSpecificBuildOptions, targets, false);
+        const framework = this.info.framework
+        log.info({
+          platform: platformName,
+          arch: Arch[arch],
+          [`${framework.name}`]: framework.version,
+          appOutDir: log.filePath(appOutDir),
+        }, `packaging`)
+        const appFile = `${this.appInfo.productFilename}.app`;
+        const { makeUniversalApp } = require('@electron/universal');
+        await makeUniversalApp({
+          x64AppPath: path.join(x64AppOutDir, appFile),
+          arm64AppPath: path.join(arm64AppOutPath, appFile),
+          outAppPath: path.join(appOutDir, appFile),
+        });
+        const rmdir = promisify(require('fs').rmdir);
+        await rmdir(x64AppOutDir, { recursive: true });
+        await rmdir(arm64AppOutPath, { recursive: true });
+        await this.doSignAfterPack(outDir, appOutDir, platformName, arch, platformSpecificBuildOptions, targets);
+        break;
+      }
+    }
+  }
+
   async pack(outDir: string, arch: Arch, targets: Array<Target>, taskManager: AsyncTaskManager): Promise<any> {
     let nonMasPromise: Promise<any> | null = null
 

packages/app-builder-lib/src/platformPackager.ts

@@ -158,7 +158,7 @@ export abstract class PlatformPackager<DC extends PlatformSpecificBuildOptions>
     }
   }
 
-  protected async doPack(outDir: string, appOutDir: string, platformName: ElectronPlatformName, arch: Arch, platformSpecificBuildOptions: DC, targets: Array<Target>) {
+  protected async doPack(outDir: string, appOutDir: string, platformName: ElectronPlatformName, arch: Arch, platformSpecificBuildOptions: DC, targets: Array<Target>, sign: boolean = true) {
     if (this.packagerOptions.prepackaged != null) {
       return
     }
@@ -241,11 +241,26 @@ export abstract class PlatformPackager<DC extends PlatformSpecificBuildOptions>
 
     const isAsar = asarOptions != null
     await this.sanityCheckPackage(appOutDir, isAsar, framework)
-    await this.signApp(packContext, isAsar)
+    if (sign) {
+      await this.doSignAfterPack(outDir, appOutDir, platformName, arch, platformSpecificBuildOptions, targets);
+    }
+  }
 
-    const afterSign = resolveFunction(this.config.afterSign, "afterSign")
+  protected async doSignAfterPack(outDir: string, appOutDir: string, platformName: ElectronPlatformName, arch: Arch, platformSpecificBuildOptions: DC, targets: Array<Target>) {
+    const asarOptions = await this.computeAsarOptions(platformSpecificBuildOptions);
+    const isAsar = asarOptions != null;
+    const packContext = {
+      appOutDir,
+      outDir,
+      arch,
+      targets,
+      packager: this,
+      electronPlatformName: platformName
+    };
+    await this.signApp(packContext, isAsar);
+    const afterSign = resolveFunction(this.config.afterSign, "afterSign");
     if (afterSign != null) {
-      await Promise.resolve(afterSign(packContext))
+      await Promise.resolve(afterSign(packContext));
     }
   }
 

packages/builder-util/src/arch.ts

@@ -1,8 +1,8 @@
 export enum Arch {
-  ia32, x64, armv7l, arm64
+  ia32, x64, armv7l, arm64, universal
 }
 
-export type ArchType = "x64" | "ia32" | "armv7l" | "arm64"
+export type ArchType = "x64" | "ia32" | "armv7l" | "arm64" | "universal"
 
 export function toLinuxArchString(arch: Arch, targetName: string): string {
   switch (arch) {
@@ -38,7 +38,8 @@ export function archFromString(name: string): Arch {
       return Arch.arm64
     case "armv7l":
       return Arch.armv7l
-
+    case "universal":
+      return Arch.universal
     default:
       throw new Error(`Unsupported arch ${name}`)
   }

packages/electron-builder/src/builder.ts

@@ -19,6 +19,7 @@ export interface CliOptions extends PackagerOptions, PublishOptions {
   ia32?: boolean
   armv7l?: boolean
   arm64?: boolean
+  universal?: boolean
 
   dir?: boolean
 }
@@ -46,6 +47,9 @@ export function normalizeOptions(args: CliOptions): BuildOptions {
       if (args.ia32) {
         result.push(Arch.ia32)
       }
+      if (args.universal) {
+        result.push(Arch.universal)
+      }
 
       return result.length === 0 && currentIfNotSpecified ? [archFromString(process.arch)] : result
     }
@@ -120,6 +124,7 @@ export function normalizeOptions(args: CliOptions): BuildOptions {
   delete result.x64
   delete result.armv7l
   delete result.arm64
+  delete result.universal
 
   let config = result.config
 
@@ -198,7 +203,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 = (arch === "all" ? (platform === Platform.MAC ? [Arch.x64, Arch.arm64] : [Arch.x64, Arch.ia32]) : [archFromString(arch == null ? process.arch : arch)])
+    const archs = (arch === "all" ? (platform === Platform.MAC ? [Arch.x64, Arch.arm64, Arch.universal] : [Arch.x64, Arch.ia32]) : [archFromString(arch == null ? process.arch : arch)])
     const archToType = new Map<Arch, Array<string>>()
     targets.set(platform, archToType)
 
@@ -259,6 +264,11 @@ export function configureBuildCommand(yargs: yargs.Argv): yargs.Argv {
       description: "Build for arm64",
       type: "boolean",
     })
+    .option("universal", {
+      group: buildGroup,
+      description: "Build for universal",
+      type: "boolean",
+    })
     .option("dir", {
       group: buildGroup,
       description: "Build unpacked dir. Useful to test.",

test/snapshots/mac/macPackagerTest.js.snap

@@ -145,6 +145,21 @@ Object {
         "size": "@size",
       },
     },
+    Object {
+      "arch": "universal",
+      "file": "TestApp-1.1.0-mac-universal.dmg",
+      "updateInfo": Object {
+        "sha512": "@sha512",
+        "size": "@size",
+      },
+    },
+    Object {
+      "file": "TestApp-1.1.0-mac-universal.dmg.blockmap",
+      "updateInfo": Object {
+        "sha512": "@sha512",
+        "size": "@size",
+      },
+    },
     Object {
       "arch": "x64",
       "file": "TestApp-1.1.0-mac.dmg",
@@ -176,6 +191,12 @@ Object {
             "size": "@size",
             "url": "TestApp-1.1.0-mac-arm64.zip",
           },
+          Object {
+            "blockMapSize": "@blockMapSize",
+            "sha512": "@sha512",
+            "size": "@size",
+            "url": "TestApp-1.1.0-mac-universal.zip",
+          },
           Object {
             "sha512": "@sha512",
             "size": "@size",
@@ -186,6 +207,11 @@ Object {
             "size": "@size",
             "url": "TestApp-1.1.0-mac-arm64.dmg",
           },
+          Object {
+            "sha512": "@sha512",
+            "size": "@size",
+            "url": "TestApp-1.1.0-mac-universal.dmg",
+          },
         ],
         "path": "TestApp-1.1.0-mac.zip",
         "releaseDate": "@releaseDate",
@@ -202,6 +228,15 @@ Object {
         "size": "@size",
       },
     },
+    Object {
+      "arch": "universal",
+      "file": "TestApp-1.1.0-mac-universal.zip",
+      "updateInfo": Object {
+        "blockMapSize": "@blockMapSize",
+        "sha512": "@sha512",
+        "size": "@size",
+      },
+    },
     Object {
       "arch": "x64",
       "file": "TestApp-1.1.0-mac.zip",