445016 2021-11-05 13:50:52 +0000 Kate fails to save changes to the running session when it receives a SIGTERM 2022-03-02 18:41:24 +0000 1 1 2 Applications kate sessions Git Other Linux RESOLVED FIXED NOR normal --- 1 nate kwrite-bugs-null christoph waqar.17a https://invent.kde.org/utilities/kate/commit/a8a9eb10f4e9159504fd4c58c87bc755114c5b15 22.04 0 oldest_to_newest 2075044 0 nate 2021-11-05 13:50:52 +0000 All KDE software built from source this morning on top of Fedora 34. STEPS TO REPRODUCE 1. Open Kate with a session 2. Open a new file 3. `killall kate` (which sends the SIGTERM signal) 4. Launch Kate again OBSERVED RESULT Kate opens with the set of tabs that does not include the file opened in step 2. Changed session data is lost. EXPECTED RESULT Kate saves any changed session data (including newly-opened files) when it receives a SIGTERM signal. IMPACT Anytime Kate gets quit automatically by the user shutting down or restarting the computer, any changed session data is not saves, causing it to be lost. This is very annoying if the set of files opened in tabs differs significantly at the moment kate is SIGKILL'ed from what it was when Kate was last launched. 2075045 1 nate 2021-11-05 13:51:22 +0000 (I mean SIGTERM'ed) 2108851 2 bug-janitor 2022-03-02 06:23:34 +0000 A possibly relevant merge request was started @ https://invent.kde.org/utilities/kate/-/merge_requests/649 2108884 3 waqar.17a 2022-03-02 11:16:44 +0000 Git commit a8a9eb10f4e9159504fd4c58c87bc755114c5b15 by Waqar Ahmed. Committed on 02/03/2022 at 11:09. Pushed by waqar into branch 'master'. Handle SIGINT/SIGTERM to prevent accidental data loss (linux only) M +2 -0 kate/CMakeLists.txt M +16 -0 kate/main.cpp M +1 -0 shared/CMakeLists.txt A +101 -0 shared/signal_watcher.cpp [License: LGPL(v2.0+)] A +37 -0 shared/signal_watcher.h [License: LGPL(v2.0+)] https://invent.kde.org/utilities/kate/commit/a8a9eb10f4e9159504fd4c58c87bc755114c5b15 2108970 4 nate 2022-03-02 16:57:29 +0000 Thank you so much, Waqar! 2108987 5 christoph 2022-03-02 18:41:24 +0000 Will be extended to KWrite in https://invent.kde.org/utilities/kate/-/merge_requests/651