| dfaure | |
| apol |
With flatpak applications we can have the following scenario:
appId= "org.kde.ark.kdbus-_1_769" newAppId= "org.kde.ark.kdbus" pendingAppId= "org.kde.ark"
because of https://phabricator.kde.org/D5775
This breaks matchesPendingRequest(), resulting in an annoying
"KDEInit could not launch '/usr/bin/flatpak'" dialog after closing the app.
This patch adds an explicit check for this case.
| Automatic diff as part of commit; lint not applicable. |
| Automatic diff as part of commit; unit tests not applicable. |
I just realized that ".kdbus" is appended by KDBusAddons, not by flatpak. Maybe we could also fix it there...
Nope, it's intentional (see D5775). So this fix in klauncher should be ok.
Btw kate is also affected, while for some reasons okular is not.
| src/klauncher/klauncher.cpp | ||
|---|---|---|
| 364 | QString::endsWith() has a QLatin1String overload, which would be slightly better here. | |
| 365 | So if I start kdesudoku and kdesu quickly, when kdesudoku is started, the pending entry for kdesu will think it matches? ;) | |
| src/klauncher/klauncher.cpp | ||
|---|---|---|
| 323–324 | This seems to be mixed with an unrelated change, now. | |