-
-
Notifications
You must be signed in to change notification settings - Fork 613
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 Zettlr 3.10 #5135
Comments
Works after update... |
I just made a reboot and problem returned |
Same issue here Linux fedora 6.8.7-300.fc40.x86_64
|
That looks suspiciously like an error in Electron. Maybe this one |
Please try out the last beta, it uses basically the same code but with a slightly older Electron version, that one should work while we wait for Electron to fix this 🍵 |
in Fedora it won't let me install zettlr-3.1.0.beta.3-1.x86_64 as it conflicts with Slack |
Ah jesus, how I hate these stupid package conflicts … there's not much I can do then except wait :/ |
I am not sure that it is exactly this error because I use sway and when I open zettlr through terminal it opens it in half screen because sway is tiling manager |
It works in my case |
The issue talks at some point about issues with tiling managers, so there is a chance this is related. |
I run Gnome on Wayland and I can confirm that both Gnome on Wayland and Hyprland have it working properly. I can reproduce the segfault with Gnome on Xorg on my Arch Linux system with an official build from the package manager. The bug doesn't seem to appear with the AppImage version. @dustdfg and @m1ga can you see if the AppImage version works as a workaround? |
it seems to reliably trigger if I try to run any version with the disable hardware acceleration flag |
The same result with appimage |
I just tried --clear-cache and second, removed .config/Zettlr completely. When I started without the Zettlr confir dir the output changed to 3.0.5 did work immediately before the update. |
This sounds fishy, why would that produce a segfault? 🤔 |
zettlr_out.txt I also have tried the other startup parameters. Ok, now, it does not crash anymore, when I start with -m and display the window. But it crashes, when I close it: The text looks like
I have tried to close everything and reopen it (-m): The first time it crashed directly when trying to show. Then I was back to be open it and crash on exit. So: See the attached file: RangeError: Maximum call stack size exceeded Please remember, this is not the segfault, this is opening zettlr using "-m", display zettlr, and when I close it, this is printed into the console. |
I noticed that myself, but thought it might be due to some weirdness in webpack, I didn't think it reoccurred in the final builds, that may be a good observation! All in all, I believe something in Electron 30 is really bad at handling windows. I've seen segfault bugs on the Electron repo being reported as an interaction with window opening and closing, tiling managers, and maximizing windows. Let's hope they fix this asap; I may consider releasing a 3.1.1 while reverting back to the previous Electron version if a fix won't appear soon. |
I just tried it out on my work computer, and Zettlr works flawlessly. Things to note:
Update: It indeed segfaults on 30.0.2, specifically at the window maximization step (so the window draws fine). |
I have now downgraded Electron to 29.3.2 which, at least on my system, launches fine. I'll immediately schedule a manual nightly build, and request that all of you who have this issue try this one out -- just to see if it still segfaults. If it doesn't, I'll release a hotfix that downgrades Electron to 29.3.2 so that we can all use the app again. |
I run Zettlr on KDE (plasma6, x11). It segfaults when maximizing the window, manually stretching it to the screen size works fine. It also completely segfaults on start, if I choose to use the native window decorations. I tried the RPM and the AppImage, both show the same behaviour. |
No crash on startup or resizing the window.
|
I confirm: When I start Zettlr using the "-m" trick, restore it to not be in fullscreen mode and exit, I can start zettlr in normal mode. RangeError: Maximum call stack size exceeded btw: The --trace-uncaught... option did not work, changed anything. Closing the non maximized window still issues the error message about the stack error. It happens when I close the non maximized error. This time adding the nodejs version information. And also the crash output, when I maximize the window: I am going to try the nightly build now |
nightly: |
Nightly works fine |
Perfect, I'll be releasing 3.1.1 soon that ships with this fix in a "regular" release so that others who run Linux but require a package manager will soon also be able to use the 3.1.x release line. |
Hey @nathanlesage, is there an estimate for the Arch package 3.1.1 release? |
Arch packages are community maintained, you need to ask the package maintainer: Or you just wait, he's quite active, and the upgrade should arrive soon. |
I was reluctant to bump the Arch Linux package that involved downgrading ELectron for a bug that I couldn't reproduce, but if the crash is Xorg specific and I'm on Wayland that does explain that. I'll have a look ... but yes the thing to do for this would have been file a bug report on our issue tracker. |
Electron 29 is still in the supporting frame, and tbh I think that it is kind of shitty of the Electron team to push this tight of a timeline onto volunteering FOSS developers anyhow. After this experience I will go back to my usual schedule of keeping one full version distance to Electron main. This is by far not the first instance where the most recent Electron version had serious bugs for people. It's similar to macOS updates: the initial ones are kind of bad, but they get better over time. What I will do, though, is publish new releases more frequently, even if that only means to bump Electron to keep up with supported versions. |
* chore: Add Sponsors link * fix: Autocorrect entries Fixes Zettlr#4961 * fix: i18n process * No more file name/message duplication * Retain relative Vue-path for better file identification in PO files * Improved extraction regex to properly extract script tags * Add a run of msguniq over each file to remove potential duplicates Fixes Zettlr#5122 * fix: Re-run i18n extraction script to fix loads of duplications and other issues * chore: Update changelog * chore: Fix German (de-DE) translation * chore: Bump version to 3.1.0-beta.3 * chore: Fix spelling in Changelog * Update CITATION.cff * fix: Improve logging in event merger * fix: Weird bug in Nextcloud * Update CITATION.cff * fix: Prevent applying illogical updates on file trees * refactor: Improve types * chore: Bump Codemirror dependencies to newest versions * chore: Bump Electron to v30.0.1 * chore: Upgrade ESLint and fix new linter errors * chore: Bump eslint dependencies * chore: Upgrade a last set of dependencies * chore: Bump remark deps * chore: Clean up CHANGELOG, Electron to 30.0.2 * chore: Bump Electron 30.0.1 -> 3.0.2 * chore: Bump version to 3.1.0; update changelog * Update CITATION.cff * chore: Add missing paragraph to changelog * chore: Bump version to 3.1.0; update changelog * chore: Add missing paragraph to changelog * chore: Downgrade Electron to 29.3.2 See issue Zettlr#5135 * chore: Fix changelog * chore: Bump version to 3.1.1 * chore(citeproc): Update CSL files * Update CITATION.cff --------- Co-authored-by: Hendrik Erz <hendrik@zettlr.com> Co-authored-by: GitHub Action <action@github.com>
There seems to be a patch incoming that should allow us to swiftly move to Electron 30/31. I'll be launching a nightly soon to confirm, please stay ready to test it so we can release a stable fix. |
Description
I run it, it shows me a window for split second and crashes
Reproducing
Just run it
Zettlr Version
Stable (most recent version)
Specify version
3.1.0
Installation Method
From the Website or GitHub
Your Platform
Architecture
Operating System Version
Debain testing, sway (xwayland)
Additional Information
If I run through terminal I see:
The text was updated successfully, but these errors were encountered: