Skip to content

Commit

Permalink
fix: Creating an MSI on Windows fails
Browse files Browse the repository at this point in the history
Close #485
  • Loading branch information
develar committed Jun 23, 2016
1 parent b192fc1 commit d1ee7de
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 2 deletions.
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@
"deep-assign": "^2.0.0",
"electron-osx-sign-tf": "0.6.0",
"electron-packager-tf": "~7.4.0",
"electron-winstaller-fixed": "~2.10.2",
"electron-winstaller-fixed": "~2.11.3",
"fs-extra-p": "^1.0.3",
"hosted-git-info": "^2.1.5",
"image-size": "^0.5.0",
Expand Down
1 change: 1 addition & 0 deletions src/targets/squirrelWindows.ts
Original file line number Diff line number Diff line change
Expand Up @@ -67,6 +67,7 @@ export default class SquirrelWindowsTarget extends Target {
productName: appInfo.productName,
exe: `${appInfo.productName}.exe`,
setupExe: setupExeName,
msiExe: setupExeName.replace(".exe", ".msi"),
title: appInfo.productName,
appDirectory: appOutDir,
outputDirectory: installerOutDir,
Expand Down
3 changes: 2 additions & 1 deletion test/src/winPackagerTest.ts
Original file line number Diff line number Diff line change
Expand Up @@ -54,12 +54,13 @@ test.ifNotCiOsx("nsis boring", () => assertPack("test-app-one", _signed({
// ))

// very slow
test.ifWinCi("delta", () => assertPack("test-app-one", {
test.ifWinCi("delta and msi", () => assertPack("test-app-one", {
targets: Platform.WINDOWS.createTarget(null, Arch.ia32),
devMetadata: {
build: {
win: {
remoteReleases: "https://github.com/develar/__test-app-releases",
msi: true,
}
}
},
Expand Down

0 comments on commit d1ee7de

Please sign in to comment.