Skip to content

Commit

Permalink
Don't make requiredExternalBinaries an abstract property
Browse files Browse the repository at this point in the history
  • Loading branch information
malept committed Nov 30, 2019
1 parent eccbb47 commit cb65690
Show file tree
Hide file tree
Showing 11 changed files with 5 additions and 21 deletions.
2 changes: 0 additions & 2 deletions packages/api/core/src/api/make.ts
Original file line number Diff line number Diff line change
Expand Up @@ -24,8 +24,6 @@ class MakerImpl extends MakerBase<any> {
name = 'impl';

defaultPlatforms = [];

requiredExternalBinaries = [];
}

export interface MakeOptions {
Expand Down
2 changes: 0 additions & 2 deletions packages/maker/appx/src/MakerAppX.ts
Original file line number Diff line number Diff line change
Expand Up @@ -81,8 +81,6 @@ export default class MakerAppX extends MakerBase<MakerAppXConfig> {

defaultPlatforms: ForgePlatform[] = ['win32'];

requiredExternalBinaries: string[] = [];

isSupportedOnCurrentPlatform() {
return process.platform === 'win32';
}
Expand Down
2 changes: 1 addition & 1 deletion packages/maker/base/src/Maker.ts
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ export default abstract class Maker<C> implements IForgeMaker {

public abstract defaultPlatforms: ForgePlatform[];

public abstract requiredExternalBinaries: string[] = [];
public requiredExternalBinaries: string[] = [];

__isElectronForgeMaker!: true;

Expand Down
2 changes: 0 additions & 2 deletions packages/maker/base/test/ensure-output_spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,6 @@ class MakerImpl extends MakerBase<{}> {
name = 'test';

defaultPlatforms = [];

requiredExternalBinaries = [];
}

describe('ensure-output', () => {
Expand Down
2 changes: 0 additions & 2 deletions packages/maker/dmg/src/MakerDMG.ts
Original file line number Diff line number Diff line change
Expand Up @@ -11,8 +11,6 @@ export default class MakerDMG extends MakerBase<MakerDMGConfig> {

defaultPlatforms: ForgePlatform[] = ['darwin', 'mas'];

requiredExternalBinaries: string[] = [];

isSupportedOnCurrentPlatform() {
return process.platform === 'darwin';
}
Expand Down
4 changes: 2 additions & 2 deletions packages/maker/flatpak/test/MakerFlatpak_spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -11,9 +11,9 @@ import { flatpakArch } from '../src/MakerFlatpak';
import { MakerFlatpakConfig } from '../src/Config';

class MakerImpl extends MakerBase<MakerFlatpakConfig> {
name = 'test';
name = 'test';

defaultPlatforms = [];
defaultPlatforms = [];
}

describe('MakerFlatpak', () => {
Expand Down
2 changes: 0 additions & 2 deletions packages/maker/pkg/src/MakerPKG.ts
Original file line number Diff line number Diff line change
Expand Up @@ -11,8 +11,6 @@ export default class MakerDMG extends MakerBase<MakerPKGConfig> {

defaultPlatforms: ForgePlatform[] = ['darwin', 'mas'];

requiredExternalBinaries: string[] = [];

isSupportedOnCurrentPlatform() {
return process.platform === 'darwin';
}
Expand Down
4 changes: 2 additions & 2 deletions packages/maker/rpm/test/MakerRpm_spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -10,9 +10,9 @@ import { MakerRpmConfig } from '../src/Config';
import { rpmArch } from '../src/MakerRpm';

class MakerImpl extends MakerBase<MakerRpmConfig> {
name = 'test';
name = 'test';

defaultPlatforms = [];
defaultPlatforms = [];
}

describe('MakerRpm', () => {
Expand Down
2 changes: 0 additions & 2 deletions packages/maker/squirrel/src/MakerSquirrel.ts
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,6 @@ export default class MakerSquirrel extends MakerBase<MakerSquirrelConfig> {

defaultPlatforms: ForgePlatform[] = ['win32'];

requiredExternalBinaries: string[] = [];

isSupportedOnCurrentPlatform() {
return this.isInstalled('electron-winstaller') && !process.env.DISABLE_SQUIRREL_TEST;
}
Expand Down
2 changes: 0 additions & 2 deletions packages/maker/wix/src/MakerWix.ts
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,6 @@ export default class MakerWix extends MakerBase<MakerWixConfig> {

defaultPlatforms: ForgePlatform[] = ['win32'];

requiredExternalBinaries: string[] = [];

isSupportedOnCurrentPlatform() {
return process.platform === 'win32';
}
Expand Down
2 changes: 0 additions & 2 deletions packages/maker/zip/src/MakerZIP.ts
Original file line number Diff line number Diff line change
Expand Up @@ -11,8 +11,6 @@ export default class MakerZIP extends MakerBase<MakerZIPConfig> {

defaultPlatforms: ForgePlatform[] = ['darwin', 'mas', 'win32', 'linux'];

requiredExternalBinaries: string[] = [];

isSupportedOnCurrentPlatform() {
return true;
}
Expand Down

0 comments on commit cb65690

Please sign in to comment.