-
-
Notifications
You must be signed in to change notification settings - Fork 215
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
Wrong binaries in prebuild versions for electron v70/v73 #338
Comments
@lgeiger Can you take a look please? |
Indeed, these binaries are wrong will not work with Electron 6 . You need to rebuild manually:
|
@andsav can you please confirm that manual build works? I've tried it either but surprisingly got the same result, for |
@andsav Attempting to run this command produces an error for me: Command: Error: This is using Visual Studio 2017 on Windows 10. |
@qwe313 manual build works for me with the command above (not @crnbarr93 for windows, try downloading the libzmq-4.2.0-x64.lib here: https://github.com/nteract/libzmq-win/tree/master/v140 and copy it to |
@andsav that worked! Thanks! |
Any idea when the precompiled binaries will be corrected? I wasted the better part of a day looking for this solution. It'd be great to save others the grief. @andsav and @crnbarr93 you maybe interested to know that the zeromq package.json already includes a script named build:libzmq which downloads the appropriate libzmq.lib for you. My solution was to create a script in my package.json specifically for rebuilding zeromq manually. It looks like this:
Obviously the target value will have to be updated anytime I change the version of Electron, but it matches the version I've specified in the devDependencies section of my package.json and I can give this file to another developer and it'll work for them, which was my first concern. Note: my devenv is Windows 10 and I'm using currently latest Electron, 6.0.12 |
Could you check if the issue is still present in the v5.1.1 release? |
Can confirm it works fine: ZeroMQ: 5.1.1 Thanks a lot @lgeiger |
Great! This was due to a bug in |
Hi, Can this issue be reopened? The prebuilt zeromq binaries still aren't declaring compatibility with a supported Electron version, and attempting to rebuild this library on Windows has never worked for me (lots of MSVC compiler errors and I'm not sure if there's a way I can tell it to use mingw gcc to build instead like you can with Windows Python). |
It seems that binaries that are labeled by
zeromq-v5.1.0-electron-v70-X
andzeromq-v5.1.0-electron-v73-X
are actually built against NODE_MODULE_VERSION 47 and 48 respectively.The text was updated successfully, but these errors were encountered: