ngraham created this revision. ngraham added reviewers: davidedmundson, fvogt, Plasma. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. ngraham requested review of this revision.
REVISION SUMMARY The wallpeper knsrc file file assumes that `qdbus` is in $PATH. However this is not guaranteed; the command lives in the Qt binaries dir which is not typically in $PATH, and distros typically create symlinks with different names in `/usr/bin`, but not all create a second compatibility symlink at `/usr/bin/qdbus` that points to the version for the current Qt version. Therefore, we should find the command in the Qt binaries dir itself rather than relying on these compatibility symlinks. TEST PLAN See that the knsrc file now has the absolute path to `qdbus` in it REPOSITORY R120 Plasma Workspace BRANCH find-qdbus-correctly (branched from master) REVISION DETAIL https://phabricator.kde.org/D25884 AFFECTED FILES CMakeLists.txt wallpapers/image/CMakeLists.txt wallpapers/image/wallpaper.knsrc wallpapers/image/wallpaper.knsrc.cmake To: ngraham, davidedmundson, fvogt, #plasma Cc: plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
