-
Notifications
You must be signed in to change notification settings - Fork 230
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 + Angular + node-pty missing file when compile #337
Comments
#326 is this error related to this issue? |
It's related to that yeah, you're probably compiling it to the wrong arch so it's not loading the Release version. Can you comment out to try/catch where it's loading it (inside node_modules/node-pty) and check the error? node-pty/src/windowsPtyAgent.ts Lines 60 to 68 in 8c89c19
|
Yes, I just did it and the errors disappeared. Thank you very much. By the way, is electron suppose to compile node-pty even with binary? Am I doing anything wrong here?... As I know, some people encountered this (different package) and they can fix it but with webpack. Is it a must to have webpack? |
Electron must compile the binary, otherwise it won't be able to load. The error you're seeing is related to configuring webpack which I don't know too much about, I think you need to make webpack ignore it or declare it as an external or something. |
Environment details
Issue description
I'm using Electron + Angular (clone from https://github.com/maximegris/angular-electron) and I want to implement a simple terminal within the application. Currently I have successfully installed node-pty (after 2 days of struggling) without error (it does show warnings though).
npm install node-pty@0.9.0-beta19 --save
However when I use hot reload, Angular seems can't compile the package as there are some missing packages ??
Checking node-pty in node_modules, I don't see any debug folder in build.
I did try to rebuild electron (electron-rebuild), re-installing package multiple times (did delete package-lock.json).
Can anyone please help me with this?
The text was updated successfully, but these errors were encountered: