electron-edge icon indicating copy to clipboard operation
electron-edge copied to clipboard

Can't work on Electron 1.7.2!!

Open rensiu opened this issue 7 years ago • 12 comments

edge_nativeclr.node was compiled against a different Node.js version using NODE_MODULE_VERSION 53. This version of Node.js requires NODE_MODULE_VERSION 54.

rensiu avatar Jun 12 '17 09:06 rensiu

From the Readme: It is currently compatible with Electron-v1.6.2 (until node.js v7.4.0) while Electron 1.7.2 runs on node.js v7.9.0.

HZSamir avatar Jun 12 '17 11:06 HZSamir

Wish to support Electron 1.7.x!! @kexplo

rensiu avatar Jun 12 '17 13:06 rensiu

@kexplo Can you update your repository to support Electron 1.7.2?

Now, I have rebuilt edge for Electron 1.7.2, directly using edge, but also wish you update your repository to support Electron 1.7.2

rensiu avatar Jun 14 '17 04:06 rensiu

@rensiu Can you explain how you rebuilt it for Electron 1.7.2 please? In detail?

HZSamir avatar Jun 14 '17 07:06 HZSamir

Go to edge package directory Run node-gyp clean Run node-gyp rebuild --target=1.7.2 --disturl=https://atom.io/download/electron --msvs_version=2015 --build-from-source

rensiu avatar Jun 14 '17 07:06 rensiu

And that's it? No copying the built .node files from the Release folder to the folder?

HZSamir avatar Jun 14 '17 07:06 HZSamir

@rensiu Sorry, I was very busy these days. I will try update the version soon.

kexplo avatar Jun 14 '17 08:06 kexplo

Why need to copy .node files? var fs = require('fs') , path = require('path') , builtEdge = path.resolve(__dirname, '../build/Release/' + (process.env.EDGE_USE_CORECLR || !fs.existsSync(path.resolve(__dirname, '../build/Release/edge_nativeclr.node')) ? 'edge_coreclr.node' : 'edge_nativeclr.node')) , edge; in lib/edge.js file. @Unforgiven-wanda

rensiu avatar Jun 14 '17 08:06 rensiu

@kexplo I expect the community would be happy to maintain these version bumps so you don't have to! If you could explain what is needed to update the version then we can get these updates through faster into electron-edge :)

greenimpala avatar Jun 21 '17 09:06 greenimpala

any update on this ? would love to use this on electron 1.7.5 to allow be to use Brothers P-Touch SDK via this package

alandoyle95 avatar Aug 25 '17 19:08 alandoyle95

There is another module called electron-edge-js https://github.com/agracio/electron-edge-js that offers support for multiple Electron version, try it out.

agracio avatar Sep 25 '17 11:09 agracio

@agracio coool, thanks

ruleechen avatar Oct 30 '17 07:10 ruleechen