Skip to content
This repository has been archived by the owner on Mar 23, 2023. It is now read-only.

Remove deprecated 'electron-prebuilt' #177

Merged
merged 1 commit into from
Jul 27, 2017
Merged

Remove deprecated 'electron-prebuilt' #177

merged 1 commit into from
Jul 27, 2017

Conversation

sleepdefic1t
Copy link
Contributor

Fixes macOS build issues.

macOS >= 10.12.0 would not npm run start after npm install.
Throwing error:

> ArkClient@1.3.0 start /Users/.../github/ark-desktop
> electron main.js

App threw an error during load
Error: Module version mismatch. Expected 50, got 53.
    at Error (native)
    at process.module.(anonymous function) [as dlopen] (ELECTRON_ASAR.js:173:20)
    at Object.Module._extensions..node (module.js:583:18)
    at Object.module.(anonymous function) [as .node] (ELECTRON_ASAR.js:173:20)
    at Module.load (module.js:473:32)
    at tryModuleLoad (module.js:432:12)
    at Function.Module._load (module.js:424:3)
    at Module.require (module.js:483:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/Users/.../github/ark-desktop/node_modules/node-hid/nodehid.js:9:15)

After removing deprecated dependency,
npm run start launches ark-desktop as expected.

Reference

Empy package:
https://www.npmjs.com/package/electron-prebuilt

"The electron package name on npm"
electron-userland/electron-prebuilt#160

"deprecation: electron-prebuilt has been renamed to electron"
egoist/vuepack#51

deprecation: electron-prebuilt has been renamed to electron  
egoist/vuepack#51

Fixes macOS build issues.
@fix
Copy link

fix commented Jul 27, 2017

nice catch actually
+5

@fix fix merged commit e735b31 into ArkEcosystem:master Jul 27, 2017
krau612 pushed a commit to krau612/ark-desktop that referenced this pull request Sep 16, 2017
Remove deprecated 'electron-prebuilt'
alexbarnsley pushed a commit that referenced this pull request Dec 3, 2018
* test: add PouchDB mocks and test the `DbInterface` class
* feat: implement a class `Model` to use with `DbInterface`
* refactor: rename the file of the  instance
* chore: add `pouchdb-debug` as a dependency and configure it
* feat: add the `find`, `getAll` and `query` methods to `DbInterface`
* fix: the `update` method of the `DbInterface`
* docs: explain how revisions are managed in some of the `DbInterface` methods
* test: update the tests of `DbInterface`
* fix: problems of the `Model` class for using `__` on an attribute that is stored on PoucDB
* feat: revamp the Vuex `DbModule` class to keep the reactivy on the `getters`
* test: add 1 test to the `DbModule` class
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants