Skip to content

Commit

Permalink
fix(maker): add a santizied app id to the electronWinstaller config o…
Browse files Browse the repository at this point in the history
…n init
  • Loading branch information
MarshallOfSound committed Dec 4, 2016
1 parent 661b1eb commit 20ae889
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 5 deletions.
1 change: 1 addition & 0 deletions src/init/init-npm.js
Expand Up @@ -19,6 +19,7 @@ export default async (dir, lintStyle) => {
const initSpinner = ora.ora('Initializing NPM Module').start();
const packageJSON = JSON.parse(await fs.readFile(path.resolve(__dirname, '../../tmpl/package.json'), 'utf8'));
packageJSON.productName = packageJSON.name = path.basename(dir).toLowerCase();
packageJSON.config.forge.electronWinstallerConfig.name = packageJSON.name.replace(/-/g, '_');
packageJSON.author = await username();
switch (lintStyle) {
case 'standard':
Expand Down
4 changes: 1 addition & 3 deletions src/makers/win32/squirrel.js
Expand Up @@ -7,9 +7,7 @@ export default async (dir, appName, forgeConfig, packageJSON) => { // eslint-dis
const outPath = path.resolve(dir, '../make/squirrel.windows');
await ensureDirectory(outPath);

const winstallerConfig = Object.assign({
description: 'This is the default electron-forge description, you can override it in your config',
}, forgeConfig.electronWinstallerConfig, {
const winstallerConfig = Object.assign({}, forgeConfig.electronWinstallerConfig, {
appDirectory: dir,
outputDirectory: outPath,
});
Expand Down
2 changes: 1 addition & 1 deletion test/cli_spec.js
Expand Up @@ -67,7 +67,7 @@ describe(`electron-forge CLI (with installer=${installer.substr(12)})`, () => {
let dir;

before(async () => {
dir = path.resolve(os.tmpdir(), `electron-forge-test-${`${Date.now()}`.substr(7)}`);
dir = path.resolve(os.tmpdir(), `electron-forge-test-${Date.now()}/electron-forge-test`);
await pSpawn(['init', dir]);
await pSpawn(['package', dir]);
});
Expand Down
4 changes: 3 additions & 1 deletion tmpl/package.json
Expand Up @@ -18,7 +18,9 @@
"linux": ["deb", "rpm"]
},
"electronPackagerConfig": {},
"electronWinstallerConfig": {},
"electronWinstallerConfig": {
"name": ""
},
"electronInstallerDebian": {},
"electronInstallerRedhat": {}
}
Expand Down

0 comments on commit 20ae889

Please sign in to comment.