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

upgrade electron to 32.2.5 #5957

Open
wants to merge 1 commit into
base: prerelease/2025.01
Choose a base branch
from

Conversation

sharon-wang
Copy link
Member

Summary

Release Notes

New Features

  • N/A

Bug Fixes

Explanation

Running positron on Linux in a terminal via positron --verbose and then clicking the File option in the menubar results in Positron crashing. The terminal output shows the following error message:

[12657:0110/105954.066563:FATAL:check.cc(376)] Check failed: false. NOTREACHED log messages are omitted in official builds. Sorry!

This looks to be the same issue as microsoft/vscode#230604, which is ultimately due to a change in Chromium on Linux.

From microsoft/vscode#230604 (comment):

Update on this crash, it is triggered by the menu item File > New File... (command: welcome.showNewFileEntries) which has accelerator

vscode/src/vs/workbench/contrib/welcomeViews/common/newFile.contribution.ts

Line 34 in e62dc67

primary: KeyMod.Alt + KeyMod.CtrlCmd + KeyMod.WinCtrl + KeyCode.KeyN,
. The runtime on Linux cannot handle Win or Super keys for accelerator shortcut text and they always end up reaching this block https://source.chromium.org/chromium/chromium/src/+/main:ui/base/accelerators/accelerator.cc;l=413 which has not been a fatal call in chromium up until a recent runtime update (despite its name it only logs in release builds and in debug builds it will crash). Chromium has changed its implication in release builds starting with Electron 32 to crash as intended.
For long term, we should handle the shortcut text in Electron to avoid such crashes. Upstream issue electron/electron#44196

Electron v32.2.5 contains the fix for this issue.

QA Notes

Clicking on the File option in the native menubar on Linux should no longer crash Positron.

Since Electron has been updated by 4 patch versions, so it would be good to check that desktop builds across OS's are still working well!

This change fixes the issue for me in a local DEB release build on Ubuntu and also worked for a dev build on Linux for Wes 🎉

Copy link

E2E Tests 🚀  ?
This PR will run tests tagged with: @critical

@sharon-wang
Copy link
Member Author

@petetronic
Copy link
Collaborator

Full test suite run: https://github.com/posit-dev/positron/actions/runs/12718317587

Could we ask QA about the win failure in the full run suite?

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 this pull request may close these issues.

3 participants