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