fix(maker): wix only works on win32 currently

electron · Jan 30, 2018 · 707a1e3 · 707a1e3
1 parent 2d4179f
commit 707a1e3
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/makers/win32/wix.js b/src/makers/win32/wix.js
@@ -3,9 +3,9 @@ import path from 'path';
 import { ensureDirectory } from '../../util/ensure-output';
 import getNameFromAuthor from '../../util/author-name';
 import configFn from '../../util/config-fn';
-import isInstalled from '../../util/is-installed';
 
-export const isSupportedOnCurrentPlatform = async () => isInstalled('electron-wix-msi');
+// electron-wix-msi doesn't set its 'os' field even though it only runs on win32
+export const isSupportedOnCurrentPlatform = async () => process.platform === 'win32';
 
 export default async ({ dir, appName, targetArch, forgeConfig, packageJSON }) => {
   const { MSICreator } = require('electron-wix-msi');

diff --git a/test/fast/makers_spec.js b/test/fast/makers_spec.js
@@ -10,6 +10,7 @@ describe('makers', () => {
       'linux/rpm': ['darwin', 'linux'],
       'win32/appx': ['win32'],
       'win32/squirrel': ['darwin', 'linux', 'win32'],
+      'win32/wix': ['win32'],
     };
 
     Object.keys(expected).forEach(async (maker) => {