ngraham created this revision. ngraham added reviewers: apol, Plasma. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. ngraham requested review of this revision.
REVISION SUMMARY System Settings installs two desktop files, and uses one with "KDE" in the name when run in a non-Plasma session. Unfortunately in today's world, this causes System Settings to appear twice in app lists (Discover, GNOME Software, appstreamCLI, etc). For example: F7265164: Two System Settings in search list.png <https://phabricator.kde.org/F7265164> F7265165: Two System Settings in updates list.png <https://phabricator.kde.org/F7265165> It wasn't a bad idea, but I don't think it really works anymore. TEST PLAN Can't test AppStream changes :/ REPOSITORY R124 System Settings BRANCH remove-redundant-desktop-file (branched from master) REVISION DETAIL https://phabricator.kde.org/D23302 AFFECTED FILES app/CMakeLists.txt app/kdesystemsettings.desktop app/main.cpp To: ngraham, apol, #plasma Cc: plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
