Find qdbus command correctly
ClosedPublic

Authored by ngraham on Dec 11 2019, 1:30 PM.

Details

Summary

The wallpaper 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

Diff Detail

Repository
R120 Plasma Workspace
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
ngraham created this revision.Dec 11 2019, 1:30 PM
Restricted Application added a project: Plasma. · View Herald TranscriptDec 11 2019, 1:30 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
ngraham requested review of this revision.Dec 11 2019, 1:30 PM
apol accepted this revision.Dec 11 2019, 4:34 PM
This revision is now accepted and ready to land.Dec 11 2019, 4:34 PM
ngraham edited the summary of this revision. (Show Details)Dec 11 2019, 4:35 PM
This revision was automatically updated to reflect the committed changes.