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

Crash error using electron 5.0.0 on snap build #122

Closed
julian-alarcon opened this issue May 2, 2019 · 3 comments
Closed

Crash error using electron 5.0.0 on snap build #122

julian-alarcon opened this issue May 2, 2019 · 3 comments
Assignees
Labels
bug Something isn't working snap Related to snap package

Comments

@julian-alarcon
Copy link
Collaborator

Describe the bug
I'm trying the new electron version, 5.0. Has new Chromium, Node 12 and V8 engine.
https://electronjs.org/blog/electron-5-0

To Reproduce
Steps to reproduce the behaviour:

  1. Remove yarn.lock file to generate a new one
  2. Change electron version on package.json to 5.0.0
  3. Build the snap package and install it
  4. Run it on from a terminal, it will crash and show this message:
julian@mypc:~/Code/teams-for-linux$ teams-for-linux 
[31893:0502/075604.540153:FATAL:setuid_sandbox_host.cc(157)] The SUID sandbox helper binary was found, but is not configured correctly. Rather than run without sandboxing I'm aborting now. You need to make sure that /snap/teams-for-linux/x1/chrome-sandbox is owned by root and has mode 4755.
Trace/breakpoint trap (core dumped)

Expected behavior
It should run with no issues

Desktop (please complete the following information):

  • OS: Ubuntu 18.04/1904
  • Installation package snap
  • Version 0.2.1

Additional context
Build on .deb works with no issues

@julian-alarcon julian-alarcon added bug Something isn't working snap Related to snap package labels May 2, 2019
@julian-alarcon julian-alarcon self-assigned this May 2, 2019
@IsmaelMartinez
Copy link
Owner

we should not be in electron 5.0.0. Drop back to 4.x.x.

There are many changes in node12 that affect the spellchecker and other libraries, so worth waiting until is more stable.

@julian-alarcon
Copy link
Collaborator Author

electron/electron#17972

@julian-alarcon
Copy link
Collaborator Author

travis-ci/travis-ci#8836

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working snap Related to snap package
Projects
None yet
Development

No branches or pull requests

2 participants