Skip to content

Commit

Permalink
Merge pull request #227 from KoenLav/electron-5
Browse files Browse the repository at this point in the history
electron-5: Added support for Electron 5
  • Loading branch information
wojtkowiak authored Nov 28, 2019
2 parents 20dd58e + b84fa88 commit 5aea4cc
Show file tree
Hide file tree
Showing 22 changed files with 11,121 additions and 8,873 deletions.
2 changes: 1 addition & 1 deletion .babelrc
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
"presets": [
["@babel/env", {
"targets": {
"node": "4"
"node": "12"
}
}]
],
Expand Down
6 changes: 3 additions & 3 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ version: 2
jobs:
test:
docker:
- image: circleci/node:8.11.3-jessie-browsers
- image: circleci/node:10.16.0-jessie-browsers
steps:
- checkout
- run:
Expand All @@ -24,7 +24,7 @@ jobs:
command: npm run test
test-integration:
docker:
- image: circleci/node:8.11.3-jessie-browsers
- image: circleci/node:10.16.0-jessie-browsers
steps:
- checkout
- run:
Expand Down Expand Up @@ -58,7 +58,7 @@ jobs:
name: install Meteor
command: |
# only install meteor if bin isn't found
command -v meteor >/dev/null 2>&1 || curl https://install.meteor.com/?release=1.5.4.1 | /bin/sh
command -v meteor >/dev/null 2>&1 || curl https://install.meteor.com/?release=1.8.1 | /bin/sh
- run:
name: copy meteor bin to build cache
command: |
Expand Down
5 changes: 5 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -11,3 +11,8 @@ coverage
*.lcov
version
port.cfg

.reify-cache
tests/.__tmp/autoupdate/Downloading
tests/.__tmp/autoupdate/autoupdate.json
tests/.__tmp/meteorProject
6 changes: 3 additions & 3 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,9 @@ os:

env:
- TEST=normal NODE_VERSION=8
- TEST=normal NODE_VERSION=9
- TEST=normal NODE_VERSION=10
- TEST=integration NODE_VERSION=8
- TEST=integration NODE_VERSION=9
- TEST=integration NODE_VERSION=10

before_install:
- nvm install $NODE_VERSION
Expand All @@ -26,7 +26,7 @@ before_install:
then
meteor --version;
else
curl https://install.meteor.com/?release=1.5.4.1 | /bin/sh;
curl https://install.meteor.com/?release=1.8.1 | /bin/sh;
fi;
fi

Expand Down
96 changes: 52 additions & 44 deletions CHANGELOG.md

Large diffs are not rendered by default.

246 changes: 123 additions & 123 deletions README.md

Large diffs are not rendered by default.

10 changes: 5 additions & 5 deletions appveyor.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,16 +6,16 @@ skip_tags: true

environment:
matrix:
- nodejs_version: "8"
- nodejs_version: "10"
test: "normal"
platform: x64
- nodejs_version: "8"
- nodejs_version: "10"
test: "integration"
platform: x64
- nodejs_version: "9"
- nodejs_version: "10"
test: "integration"
platform: x64
- nodejs_version: "9"
- nodejs_version: "10"
test: "normal"
platform: x64

Expand All @@ -36,7 +36,7 @@ install:
- ps: >-
if ($env:test -eq "integration") {
if ((Get-Command "meteor" -ErrorAction SilentlyContinue) -eq $null) {
choco install meteor --params "'/RELEASE:1.5.4.1'"
choco install meteor --params "'/RELEASE:1.8.1'"
}
}
- ps: refreshenv
Expand Down
2 changes: 1 addition & 1 deletion devEnvSetup.js
Original file line number Diff line number Diff line change
Expand Up @@ -116,7 +116,7 @@ question('Do you want to use another path (yes/no)? ')
});
}
console.log('\nCreating meteor-desktop-test-app');
exec('meteor create meteor-desktop-test-app --release=METEOR@1.4.2.7');
exec('meteor create meteor-desktop-test-app --release=METEOR@1.6');

console.log('Installing deps in meteor-desktop...\n');
return spawn(npm, ['install'], path.join(resolvedPath, 'meteor-desktop'));
Expand Down
7 changes: 3 additions & 4 deletions lib/defaultDependencies.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
module.exports = {
electron: '2.0.10',
'electron-builder': '20.28.4',
'electron-packager': '12.1.2'
electron: '6.0.1',
'electron-builder': '21.2.0',
'electron-packager': '14.0.4'
};

2 changes: 1 addition & 1 deletion lib/electronApp.js
Original file line number Diff line number Diff line change
Expand Up @@ -842,7 +842,7 @@ export default class ElectronApp {

const uglifyingEnabled = 'uglify' in settings && !!settings.uglify;

const preset = presetEnv(undefined, { targets: { node: '8' } });
const preset = presetEnv({ assertVersion: () => { } }, { targets: { node: '12' } });

const { data: files } = await this.$.utils.readDir(this.$.env.paths.desktopTmp.root);

Expand Down
15 changes: 14 additions & 1 deletion lib/electronBuilder.js
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,7 @@ export default class InstallerBuilder {
this.lastRebuild = {};
this.currentContext = null;
this.installerDir = path.join(this.$.env.options.output, this.$.env.paths.installerDir);
this.platforms = [];
}

async init() {
Expand All @@ -66,7 +67,8 @@ export default class InstallerBuilder {
* @returns {Object}
*/
prepareLastRebuildObject(arch, platform = process.platform) {
const productionDeps = this.packageDependencies.createLazyProductionDeps(this.$.env.paths.electronApp.root);
const productionDeps = this.packageDependencies
.createLazyProductionDeps(this.$.env.paths.electronApp.root);
this.lastRebuild = {
frameworkInfo: { version: this.$.getElectronVersion(), useCustomDist: true },
platform,
Expand Down Expand Up @@ -142,6 +144,11 @@ export default class InstallerBuilder {
* @returns {Promise}
*/
afterPack(context) {
this.platforms = this.platforms
.filter(platform => platform !== context.electronPlatformName);
if (this.platforms.length !== 0) {
return Promise.resolve();
}
return new Promise((resolve, reject) => {
shell.config.fatal = true;

Expand Down Expand Up @@ -329,6 +336,12 @@ export default class InstallerBuilder {
output: path.join(this.$.env.options.output, this.$.env.paths.installerDir)
};

if ('mac' in builderOptions && 'target' in builderOptions.mac) {
if (builderOptions.mac.target.includes('mas')) {
this.platforms = ['darwin', 'mas'];
}
}

try {
this.log.debug('calling build from electron-builder');
await this.builder.dependency.build(Object.assign({
Expand Down
2 changes: 1 addition & 1 deletion lib/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -151,7 +151,7 @@ class MeteorDesktop {
const { dependencies = {}, devDependencies = {} } = JSON.parse(
fs.readFileSync(path.join(this.env.paths.meteorApp.root, 'package.json'), 'UTF-8')
);
if (!( name in dependencies ) && !( name in devDependencies )) {
if (!(name in dependencies) && !(name in devDependencies)) {
await this.meteorApp.runNpm(['i', '-D', '-E', '--only=dev', `${name}@${version}`], 'inherit');
}
} catch (e) {
Expand Down
Loading

0 comments on commit 5aea4cc

Please sign in to comment.