Skip to content
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

Zeromq doesn't build on Electron 9 #2034

Closed
2 of 5 tasks
aminya opened this issue Dec 16, 2020 · 7 comments · Fixed by #2082
Closed
2 of 5 tasks

Zeromq doesn't build on Electron 9 #2034

aminya opened this issue Dec 16, 2020 · 7 comments · Fixed by #2082
Labels
bug 🐛 For unexpected issues

Comments

@aminya
Copy link
Member

aminya commented Dec 16, 2020

Prerequisites

Description

Hydrogen fails to build on Atom with Electron 9.

Full log is included in the log section. This is a summary:

❯ apm-dev rebuild --verbose
Rebuilding modules failed
gyp ERR! build error
gyp ERR! stack Error: `C:\Program Files (x86)\Microsoft Visual Studio\2019\Preview\MSBuild\Current\Bin\MSBuild.exe` failed with exit code: 1
gyp ERR! stack     at ChildProcess.onExit (C:\Users\aminy\Documents\GitHub\JavaScript\@atom-ide-community\atom\out\Atom Dev x64\resources\app\apm\node_modules\npm\node_modules\node-gyp\lib\build.js:194:23)
gyp ERR! stack     at ChildProcess.emit (events.js:200:13)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:272:12)
gyp ERR! System Windows_NT 10.0.19042
gyp ERR! command "C:\\Users\\aminy\\Documents\\GitHub\\JavaScript\\@atom-ide-community\\atom\\out\\Atom Dev x64\\resources\\app\\apm\\bin\\node.exe" "C:\\Users\\aminy\\Documents\\GitHub\\JavaScript\\@atom-ide-community\\atom\\out\\Atom Dev x64\\resources\\app\\apm\\node_modules\\npm\\node_modules\\node-gyp\\bin\\node-gyp.js" "rebuild"
gyp ERR! cwd C:\.atom\packages\Hydrogen\node_modules\zeromq
gyp ERR! node -v v12.4.0
gyp ERR! node-gyp -v v5.1.0
gyp ERR! not ok
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR! [email protected] install: `node scripts/prebuild-install.js || (node scripts/preinstall.js && node-gyp rebuild)`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the [email protected] install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     C:\Users\aminy\.atom\.apm\_logs\2020-12-16T23_12_45_444Z-debug.log

Steps to Reproduce

  1. Build Atom in set app.allowRendererProcessReuse to false + fix package.json + fix remote/shell deprecation atom/atom#21805
  2. Try to build Hydrogen with the included apm under ./out/Atom Dev x64/resources/app/apm/bin/apm

Versions

OS version:
[Your OS's version]

atom --version

❯ atom-dev9 --version

Atom    : 1.56.0-dev-ffaba34b3
Electron: 9.3.5
Chrome  : 83.0.4103.122
Node    : 12.14.1

apm --version

❯ apm-dev --version
apm  2.5.2
npm  6.14.8
node 12.4.0 x64
atom 1.56.0-dev-ffaba34b3
python 3.6.8
git 2.29.2.windows.3
visual studio

Hydrogen version:

Hydrogen 2.14.6

Plugins

Have you installed and activated any of the Hydrogen plugins below ?

  • hydrogen-python
  • Hydrogen Launcher
  • Data Explorer

Logs

Here is a full log:
2020-12-16T23_12_45_444Z-debug.log

Expected behavior

[A clear and concise description of what you expected to happen.]

Screenshots

[If applicable, add screenshots to help explain your problem.]

Additional context

[Add any other context about the problem here.]

@aminya

This comment has been minimized.

@aminya
Copy link
Member Author

aminya commented Apr 6, 2021

I made a fix for this in Zeromq. We need to wait until that is merged to Zeromq and then Jmp.
zeromq/zeromq.js#444

@aminya aminya changed the title Fails to build on Electron 9 Zeromq doesn't build on Electron 9 Apr 13, 2021
@nteract nteract deleted a comment from joseph5119 Apr 13, 2021
@aminya aminya added the windows label Apr 13, 2021
@dahaorendrm
Copy link

dahaorendrm commented Apr 13, 2021

Same problem here.

Problem solved with lower version atom 1.51.0

@WhyBeU
Copy link

WhyBeU commented Apr 14, 2021

Regarding #2077 is this the fix : "We need to wait until that is merged to Zeromq "
Any recommendation meanwhile?
Thanks.

@aminya
Copy link
Member Author

aminya commented Apr 14, 2021

@captainsafia is registering the a patched ZeroMQ. After that one is registered, I will quickly create a PR to fix it. I will do my best to fix the issue as soon as possible, hopefully, tonight or tomorrow at the latest.

For now, you can use Atom 1.55. Fortunately, Atom doesn't remove the previous version, and you can actually use it. Here is my Atom installation which still has 1.55.

image

@aminya
Copy link
Member Author

aminya commented Apr 15, 2021

Hydrogen 2.15.0 is released. Now you can use Hydrogen on the latest Atom 🎉

@DOOMKEKER
Copy link

same problem with Hydrogen 2.15.3 :c

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 For unexpected issues
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants