Build platform plugins in correct sub directory of build dir
ClosedPublic

Authored by graesslin on Dec 6 2018, 5:40 PM.

Details

Summary

This brings KWin a step closer to be run from build dir without having
to install at all. The integration tests are adjusted so that the
virtual platform is still found which makes the code be closer to what
is used in normal kwin_wayland.

Test Plan

ctest passes, manually verified correct plugin is loaded

Diff Detail

Repository
R108 KWin
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
graesslin created this revision.Dec 6 2018, 5:40 PM
Restricted Application added a project: KWin. · View Herald TranscriptDec 6 2018, 5:40 PM
Restricted Application added a subscriber: kwin. · View Herald Transcript
graesslin requested review of this revision.Dec 6 2018, 5:40 PM

cmake changes ++

Autotest change is probably fine, but there's either dead code or some workaround that needs a code comment.

autotests/integration/kwin_wayland_test.cpp
64–66

Why are we removing and adding the same path?

graesslin added inline comments.Dec 11 2018, 4:21 PM
autotests/integration/kwin_wayland_test.cpp
64–66

It's moving our own path to the front, so that a plugin in our build order is preferred over the plugin in /usr

davidedmundson accepted this revision.Dec 12 2018, 2:29 PM
This revision is now accepted and ready to land.Dec 12 2018, 2:29 PM
This revision was automatically updated to reflect the committed changes.