-
Notifications
You must be signed in to change notification settings - Fork 655
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Electron 4.0.4 complains about NODE_MODULE_VERSION #762
Comments
This is impossible with the current tools we have. Patch releases aren't supposed to break semantic versioning like this. |
True, I suppose the upgraded node module version was supposed to be there from the beginning and they just fixed it up in the patch release. Probably better to break compatibility with 4.0.0-4.0.3 and make it working with >=4.0.4 than what is there now. |
This might be what we end up doing, yes. We'll discuss internally first. |
Any update on this? |
We've decided to wait for the Electron team to fix it on their end by publishing new packages. Doing anything on our side makes little sense as we'd need to republish every single version of grpc that ever supported electron 4, whereas all they need to do is publish fixed packages. |
So the next grpc release will have Electron packages for 4.1? I get this error:
|
Right. In this case, we might even build 4.1 binaries for 1.19 right away. We'll update this later today. |
We've been struggling with some compiler update which broke our build. We'll try posting 4.1 binaries when this is cleared up. |
Hello, When will 4.1 binaries for 1.19 released? |
I am facing the same issue, is there any solution ? I am trying to use electron 4.0.8 and grpc 1.19 |
The solution is to wait until we managed to get our compilation issues sorted. |
@nicolasnoble is there any status update on this? I'm not sure if it's related, but I believe it is; I'm suddenly seeing my Electron build size increase by 40% (a little over 30mb) all attributable to the |
This is related; your machine builds its own binaries. |
We've finally sorted out the various issues plaguing our build, and we'll be releasing grpc 1.20 soon with Electron 3.1 and 4.1 prebuild binaries. |
Glad to here that. Thanks ! |
We just published grpc-1.20.0-pre1 with hopefully electron 3.1 and 4.1 binaries. We're no longer going to produce 4.0 binaries due to the ABI instability issue on this branch. |
thanks, works for me |
Problem description
The grpc_node module seems to be compiled for NODE_MODULE_VERSION=64, but Electron 4 since 4.0.4 switched (electron/electron#16687) to specifying NODE_MODULE_VERSION=69 and there is a mismatch at runtime trying to load the module.
Reproduction steps
Run project with Electron 4.0.4 (or latest 4.0.7) and grpc npm dependency on 1.19.
The following error message is shown
Environment
Additional context
Hopefully somewhere it's possible to specify the range of NODE_MODULE_VERSION accepted by the module... since it would be nice for the same module to work both with Electron 4.0.3 (which specifes NODE_MODULE_VERSION=64) and with newest Electron versions >=4.0.4
The text was updated successfully, but these errors were encountered: