D10057: Require libinput and udev
Martin Flöser
noreply at phabricator.kde.org
Tue Jan 23 17:17:43 UTC 2018
graesslin created this revision.
graesslin added reviewers: KWin, Plasma.
Restricted Application added a project: KWin.
Restricted Application added subscribers: kwin, plasma-devel.
graesslin requested review of this revision.
Restricted Application edited projects, added Plasma; removed KWin.
REVISION SUMMARY
The main reason for not having it as a mandatory dependency was that BSD
doesn't support it. But as I learned recently it is available on our CI
system. So BSDs have support now.
Even more it showed that the code doesn't compile if the dependency is
missing. And there's one thing I hate: broken build configuration
options.
So let's make UDEV and libinput a required dependency and get rid of the
problems.
TEST PLAN
Compiles
REPOSITORY
R108 KWin
BRANCH
require-input-udev
REVISION DETAIL
https://phabricator.kde.org/D10057
AFFECTED FILES
CMakeLists.txt
autotests/CMakeLists.txt
config-kwin.h.cmake
debug_console.cpp
debug_console.h
input.cpp
main.cpp
main_wayland.cpp
plugins/platforms/CMakeLists.txt
plugins/platforms/virtual/egl_gbm_backend.cpp
tabletmodemanager.cpp
tests/CMakeLists.txt
workspace.cpp
To: graesslin, #kwin, #plasma
Cc: plasma-devel, kwin, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180123/b33d893a/attachment.html>
More information about the Plasma-devel
mailing list