fix: ${arch} missing from app-update.yml

Close #1389

Author: develar

packages/electron-builder-squirrel-windows/src/squirrelWindows.ts

@@ -36,15 +36,15 @@ export default class SquirrelWindowsTarget extends Target {
 
     await buildInstaller(<SquirrelOptions>distOptions, installerOutDir, setupFileName, packager, appOutDir)
 
-    packager.dispatchArtifactCreated(path.join(installerOutDir, setupFileName), this, `${appInfo.name}-Setup-${version}${archSuffix}.exe`)
+    packager.dispatchArtifactCreated(path.join(installerOutDir, setupFileName), this, arch, `${appInfo.name}-Setup-${version}${archSuffix}.exe`)
 
     const packagePrefix = `${appInfo.name}-${convertVersion(version)}-`
-    packager.dispatchArtifactCreated(path.join(installerOutDir, `${packagePrefix}full.nupkg`), this)
+    packager.dispatchArtifactCreated(path.join(installerOutDir, `${packagePrefix}full.nupkg`), this, arch)
     if (distOptions.remoteReleases != null) {
-      packager.dispatchArtifactCreated(path.join(installerOutDir, `${packagePrefix}delta.nupkg`), this)
+      packager.dispatchArtifactCreated(path.join(installerOutDir, `${packagePrefix}delta.nupkg`), this, arch)
     }
 
-    packager.dispatchArtifactCreated(path.join(installerOutDir, "RELEASES"), this)
+    packager.dispatchArtifactCreated(path.join(installerOutDir, "RELEASES"), this, arch)
   }
 
   async computeEffectiveDistOptions(): Promise<SquirrelOptions> {

packages/electron-builder/src/macPackager.ts

@@ -214,7 +214,7 @@ export default class MacPackager extends PlatformPackager<MacOptions> {
 
       const pkg = path.join(outDir!, this.expandArtifactNamePattern(masOptions, "pkg"))
       await this.doFlat(appPath, pkg, masInstallerIdentity, keychainName)
-      this.dispatchArtifactCreated(pkg, null, `${this.appInfo.name}-${this.appInfo.version}.pkg`)
+      this.dispatchArtifactCreated(pkg, null, Arch.x64, `${this.appInfo.name}-${this.appInfo.version}.pkg`)
     }
   }
 

packages/electron-builder/src/packagerApi.ts

@@ -76,6 +76,7 @@ export interface BuildInfo {
 export interface ArtifactCreated {
   readonly packager: PlatformPackager<any>
   readonly target: Target | null
+  readonly arch: Arch | null
 
   readonly file?: string
   readonly data?: Buffer

packages/electron-builder/src/platformPackager.ts

@@ -89,12 +89,10 @@ export abstract class PlatformPackager<DC extends PlatformSpecificBuildOptions>
     return this.info.prepackaged || path.join(outDir, `${this.platform.buildConfigurationKey}${getArchSuffix(arch)}${this.platform === Platform.MAC ? "" : "-unpacked"}`)
   }
 
-  dispatchArtifactCreated(file: string, target: Target | null, safeArtifactName?: string) {
+  dispatchArtifactCreated(file: string, target: Target | null, arch: Arch | null, safeArtifactName?: string) {
     this.info.dispatchArtifactCreated({
-      file: file,
-      safeArtifactName: safeArtifactName,
+      file, safeArtifactName, target, arch,
       packager: this,
-      target: target,
     })
   }
 

packages/electron-builder/src/publish/PublishManager.ts

@@ -1,6 +1,6 @@
 import BluebirdPromise from "bluebird-lst"
 import { createHash } from "crypto"
-import { Platform, PlatformSpecificBuildOptions, Target } from "electron-builder-core"
+import { Arch, Platform, PlatformSpecificBuildOptions, Target } from "electron-builder-core"
 import { CancellationToken } from "electron-builder-http/out/CancellationToken"
 import { BintrayOptions, GenericServerOptions, GithubOptions, githubUrl, PublishConfiguration, PublishProvider, S3Options, s3Url, UpdateInfo, VersionInfo } from "electron-builder-http/out/publishOptions"
 import { asArray, debug, isEmptyOrSpaces, isPullRequest, safeStringifyJson } from "electron-builder-util"
@@ -73,7 +73,7 @@ export class PublishManager implements PublishContext {
         return
       }
 
-      const publishConfigs = await getPublishConfigsForUpdateInfo(packager, await getPublishConfigs(packager, null))
+      const publishConfigs = await getPublishConfigsForUpdateInfo(packager, await getPublishConfigs(packager, null, event.arch), event.arch)
       if (publishConfigs == null || publishConfigs.length === 0) {
         return
       }
@@ -96,7 +96,7 @@ export class PublishManager implements PublishContext {
   private async artifactCreated(event: ArtifactCreated) {
     const packager = event.packager
     const target = event.target
-    const publishConfigs = event.publishConfig == null ? await getPublishConfigs(packager, target == null ? null : target.options) : [event.publishConfig]
+    const publishConfigs = event.publishConfig == null ? await getPublishConfigs(packager, target == null ? null : target.options, event.arch) : [event.publishConfig]
 
     const eventFile = event.file
     if (publishConfigs == null) {
@@ -184,7 +184,7 @@ export class PublishManager implements PublishContext {
   }
 }
 
-export async function getPublishConfigsForUpdateInfo(packager: PlatformPackager<any>, publishConfigs: Array<PublishConfiguration> | null): Promise<Array<PublishConfiguration> | null> {
+export async function getPublishConfigsForUpdateInfo(packager: PlatformPackager<any>, publishConfigs: Array<PublishConfiguration> | null, arch: Arch | null): Promise<Array<PublishConfiguration> | null> {
   if (publishConfigs === null) {
     return null
   }
@@ -195,7 +195,7 @@ export async function getPublishConfigsForUpdateInfo(packager: PlatformPackager<
     // default publish config is github, file should be generated regardless of publish state (user can test installer locally or manage the release process manually)
     const repositoryInfo = await packager.info.repositoryInfo
     if (repositoryInfo != null && repositoryInfo.type === "github") {
-      const resolvedPublishConfig = await getResolvedPublishConfig(packager, {provider: repositoryInfo.type}, false)
+      const resolvedPublishConfig = await getResolvedPublishConfig(packager, {provider: repositoryInfo.type}, arch, false)
       if (resolvedPublishConfig != null) {
         return [resolvedPublishConfig]
       }
@@ -206,7 +206,7 @@ export async function getPublishConfigsForUpdateInfo(packager: PlatformPackager<
 
 async function writeUpdateInfo(event: ArtifactCreated, _publishConfigs: Array<PublishConfiguration>) {
   const packager = event.packager
-  const publishConfigs = await getPublishConfigsForUpdateInfo(packager, _publishConfigs)
+  const publishConfigs = await getPublishConfigsForUpdateInfo(packager, _publishConfigs, event.arch)
   if (publishConfigs == null || publishConfigs.length === 0) {
     return
   }
@@ -236,6 +236,7 @@ async function writeUpdateInfo(event: ArtifactCreated, _publishConfigs: Array<Pu
 
       packager.info.dispatchArtifactCreated({
         file: updateInfoFile,
+        arch: null,
         packager: packager,
         target: null,
         publishConfig: publishConfig,
@@ -273,13 +274,15 @@ async function writeWindowsUpdateInfo(event: ArtifactCreated, version: string, o
       packager: packager,
       target: null,
       publishConfig: githubPublishConfig,
+      arch: null,
     })
   }
 
   const genericPublishConfig = publishConfigs.find(it => it.provider === "generic" || it.provider === "s3")
   if (genericPublishConfig != null) {
     packager.info.dispatchArtifactCreated({
       file: updateInfoFile,
+      arch: null,
       packager: packager,
       target: null,
       publishConfig: genericPublishConfig,
@@ -347,7 +350,7 @@ export function computeDownloadUrl(publishConfig: PublishConfiguration, fileName
   return `${baseUrl}/${encodeURI(fileName)}`
 }
 
-export async function getPublishConfigs(packager: PlatformPackager<any>, targetSpecificOptions: PlatformSpecificBuildOptions | null | undefined): Promise<Array<PublishConfiguration> | null> {
+export async function getPublishConfigs(packager: PlatformPackager<any>, targetSpecificOptions: PlatformSpecificBuildOptions | null | undefined, arch: Arch | null): Promise<Array<PublishConfiguration> | null> {
   let publishers
 
   // check build.nsis (target)
@@ -385,7 +388,7 @@ export async function getPublishConfigs(packager: PlatformPackager<any>, targetS
 
     if (serviceName != null) {
       debug(`Detect ${serviceName} as publish provider`)
-      return [(await getResolvedPublishConfig(packager, {provider: serviceName}))!]
+      return [(await getResolvedPublishConfig(packager, {provider: serviceName}, arch))!]
     }
   }
 
@@ -394,7 +397,7 @@ export async function getPublishConfigs(packager: PlatformPackager<any>, targetS
   }
 
   debug(`Explicit publish provider: ${safeStringifyJson(publishers)}`)
-  return await (<Promise<Array<PublishConfiguration>>>BluebirdPromise.map(asArray(publishers), it => getResolvedPublishConfig(packager, typeof it === "string" ? {provider: it} : it)))
+  return await (<Promise<Array<PublishConfiguration>>>BluebirdPromise.map(asArray(publishers), it => getResolvedPublishConfig(packager, typeof it === "string" ? {provider: it} : it, arch)))
 }
 
 function sha256(file: string) {
@@ -423,22 +426,21 @@ function getCiTag() {
   return tag != null && tag.length > 0 ? tag : null
 }
 
-function expandPublishConfig(options: any, packager: PlatformPackager<any>): void {
+function expandPublishConfig(options: any, packager: PlatformPackager<any>, arch: Arch | null): void {
   for (const name of Object.keys(options)) {
     const value = options[name]
     if (typeof value === "string") {
-      const expanded = packager.expandMacro(value, null)
+      const expanded = packager.expandMacro(value, arch)
       if (expanded !== value) {
         options[name] = expanded
       }
     }
   }
 }
-  
 
-async function getResolvedPublishConfig(packager: PlatformPackager<any>, options: PublishConfiguration, errorIfCannot: boolean = true): Promise<PublishConfiguration | null> {
+async function getResolvedPublishConfig(packager: PlatformPackager<any>, options: PublishConfiguration, arch: Arch | null, errorIfCannot: boolean = true): Promise<PublishConfiguration | null> {
   options = Object.assign(Object.create(null), options)
-  expandPublishConfig(options, packager)
+  expandPublishConfig(options, packager, arch)
 
   const provider = options.provider
   if (provider === "generic") {

packages/electron-builder/src/targets/ArchiveTarget.ts

@@ -40,6 +40,6 @@ export class ArchiveTarget extends Target {
       await archive(packager.config.compression, format, outFile, appOutDir)
     }
 
-    packager.dispatchArtifactCreated(outFile, this, isMac ? packager.generateName2(format, "mac", true) : packager.generateName(format, arch, true, packager.platform === Platform.WINDOWS ? "win" : null))
+    packager.dispatchArtifactCreated(outFile, this, Arch.x64, isMac ? packager.generateName2(format, "mac", true) : packager.generateName(format, arch, true, packager.platform === Platform.WINDOWS ? "win" : null))
   }
 }

packages/electron-builder/src/targets/WebInstaller.ts

@@ -21,7 +21,7 @@ export default class WebInstallerTarget extends NsisTarget {
 
     let appPackageUrl = (<NsisWebOptions>options).appPackageUrl
     if (appPackageUrl == null) {
-      const publishConfigs = await getPublishConfigsForUpdateInfo(packager, await getPublishConfigs(packager, this.options))
+      const publishConfigs = await getPublishConfigsForUpdateInfo(packager, await getPublishConfigs(packager, this.options, null), null)
       if (publishConfigs == null || publishConfigs.length === 0) {
         throw new Error("Cannot compute app package download URL")
       }

packages/electron-builder/src/targets/appImage.ts

@@ -100,6 +100,6 @@ export default class AppImageTarget extends Target {
 
     await chmod(resultFile, "0755")
 
-    packager.dispatchArtifactCreated(resultFile, this)
+    packager.dispatchArtifactCreated(resultFile, this, arch)
   }
 }

packages/electron-builder/src/targets/appx.ts

@@ -71,7 +71,7 @@ export default class AppXTarget extends Target {
     await spawn(path.join(vendorPath, "windows-10", arch === Arch.ia32 ? "ia32" : "x64", "makeappx.exe"), args)
 
     await packager.sign(destination)
-    packager.dispatchArtifactCreated(destination, this, packager.generateName("appx", arch, true))
+    packager.dispatchArtifactCreated(destination, this, arch, packager.generateName("appx", arch, true))
   }
 
   private async writeManifest(templatePath: string, preAppx: string, safeName: string, arch: Arch, publisher: string) {

packages/electron-builder/src/targets/dmg.ts

@@ -171,7 +171,7 @@ export class DmgTarget extends Target {
     await spawn("hdiutil", addVerboseIfNeed(["convert", tempDmg, "-ov", "-format", specification.format!, "-imagekey", `zlib-level=${process.env.ELECTRON_BUILDER_COMPRESSION_LEVEL || "9"}`, "-o", artifactPath]))
     await exec("hdiutil", addVerboseIfNeed(["internet-enable", "-no"]).concat(artifactPath))
 
-    this.packager.dispatchArtifactCreated(artifactPath, this, `${appInfo.name}-${appInfo.version}.dmg`)
+    this.packager.dispatchArtifactCreated(artifactPath, this, arch, `${appInfo.name}-${appInfo.version}.dmg`)
   }
 
   computeVolumeName(custom?: string | null): string {