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

This module is not context aware and requires allowRendererProcessReuse= false #580

Open
saghul opened this issue May 20, 2020 · 7 comments · May be fixed by #696
Open

This module is not context aware and requires allowRendererProcessReuse= false #580

saghul opened this issue May 20, 2020 · 7 comments · May be fixed by #696

Comments

@saghul
Copy link

saghul commented May 20, 2020

Expected Behavior

There sshould be no need to set app.allowRendererProcessReuse=false. Note that as of Electron 9 the default flipped to true.

Current Behavior

One has to manually set allowRendererProcessReuse to false, and you get this warning:

[1] (node:82611) Electron: Loading non-context-aware native module in renderer: '/Users/saghul/work/jitsi/jitsi-meet-electron/node_modules/robotjs/build/Release/robotjs.node'. This is deprecated, see https://github.com/electron/electron/issues/18397.

Possible Solution

Looks like electron/electron#18397 may have the answers.

Steps to Reproduce (for bugs)

Context

Your Environment

  • RobotJS version: 0.6.0
  • Node.js version:
  • npm version:
  • Operating System:
@DanielMcAssey
Copy link

Any update to this? As the option has been deprecated.

@ivanjeremic
Copy link

Man why nobody is answering this?

@ianwieds
Copy link

Also interested in having this fixed. What can we do to help?

@ghost
Copy link

ghost commented Sep 2, 2020

I'm also having this issue now. Fixed it for now with app.allowRendererProcessReuse=false but in Electron 11 that won't be possible.

@aabuhijleh
Copy link

A possible workaround I'm using is to load this module in the main process and if I need to use it from the renderer process, I just use Electron's IPC calls.

@srjlulla
Copy link

srjlulla commented Jan 3, 2021

Any update on this issue?

@romain130492
Copy link

?....

@csett86 csett86 linked a pull request Nov 3, 2021 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants