Created attachment 129057 [details] gdb backtrace SUMMARY Tellico compiled with kde-craft.Websearch with musicbrainz and discogs and save collection works as expected. Tellico crash when I use 'file->close' or close button. STEPS TO REPRODUCE 1. start tellico 2. search/add stuff/save 3. quit tellico OBSERVED RESULT Tellico crash and maybe it could probably corrupt collection EXPECTED RESULT No crash SOFTWARE/OS VERSIONS Windows: 10 1909 KDE Frameworks 5.69.0 Qt 5.14.1 (construit sur 5.14.1) Le système de fenêtres windows ADDITIONAL INFORMATION
I've build tellico with CMAKE_DISABLE_FIND_PACKAGE_KF5NewStuff=TRUE but it still crash. Then I commented this line "NewStuff::Manager::self();" in src/mainwindow.cpp and no more crash after quit. I do not know impact of this change. According to kde-craft dev, there is patch in qt for dbus on Windows and it could crash. Is this possible to add an option to disable dbus/knewstuff in tellico ?
(In reply to gitdevmod from comment #1) > I've build tellico with CMAKE_DISABLE_FIND_PACKAGE_KF5NewStuff=TRUE but it > still crash. > Then I commented this line "NewStuff::Manager::self();" in > src/mainwindow.cpp and no more crash after quit. I do not know impact of > this change. It just removes the DBUS interface for installing new themes, which you would probably not even miss. I'm not sure why it would crash though > According to kde-craft dev, there is patch in qt for dbus on Windows and it > could crash. > Is this possible to add an option to disable dbus/knewstuff in tellico ? That might be the brute-force approach. I'll work it. Thanks for working that out.
Git commit abd4e2aa357fdbf0818f875a836a0f409fc075c1 by Robby Stephenson. Committed on 03/03/2021 at 12:16. Pushed by rstephenson into branch 'master'. Check dbus interface pointer before unregistered Try to avoid crash on windows M +5 -1 src/newstuff/manager.cpp https://invent.kde.org/office/tellico/commit/abd4e2aa357fdbf0818f875a836a0f409fc075c1
From my basic test, I think the recent commit fixed the crashing bug.