Differential D18570 Diff 50516 plugins/platforms/drm/CMakeLists.txt

Changeset View

Standalone View

View Options

plugins/platforms/drm/CMakeLists.txt

1set(DRM_SOURCES 1​set(DRM_SOURCES
2 drm_backend.cpp 2​ drm_backend.cpp
3 drm_object.cpp 3​ drm_object.cpp
4 drm_object_connector.cpp 4​ drm_object_connector.cpp
5 drm_object_crtc.cpp 5​ drm_object_crtc.cpp
6 drm_object_plane.cpp 6​ drm_object_plane.cpp
7 drm_output.cpp 7​ drm_output.cpp
8 drm_buffer.cpp 8​ drm_buffer.cpp
9 drm_inputeventfilter.cpp 9​ drm_inputeventfilter.cpp
10 logging.cpp 10​ logging.cpp
11 scene_qpainter_drm_backend.cpp 11​ scene_qpainter_drm_backend.cpp
12 screens_drm.cpp 12​ screens_drm.cpp
13​ egl_stream_backend.cpp
13) 14​)
14 15​
15if(HAVE_GBM) 16​if(HAVE_GBM)
16 set(DRM_SOURCES ${DRM_SOURCES} 17​ set(DRM_SOURCES ${DRM_SOURCES}
17 egl_gbm_backend.cpp 18​ egl_gbm_backend.cpp
18 drm_buffer_gbm.cpp 19​ drm_buffer_gbm.cpp
19 gbm_surface.cpp 20​ gbm_surface.cpp
20 remoteaccess_manager.cpp 21​ remoteaccess_manager.cpp
21 ) 22​ )
22endif() 23​endif()
23 24​
24include_directories(${CMAKE_SOURCE_DIR}/platformsupport/scenes/opengl) 25​include_directories(${CMAKE_SOURCE_DIR}/platformsupport/scenes/opengl)
25 26​
26add_library(KWinWaylandDrmBackend MODULE ${DRM_SOURCES}) 27​add_library(KWinWaylandDrmBackend MODULE ${DRM_SOURCES})
27set_target_properties(KWinWaylandDrmBackend PROPERTIES LIBRARY_OUTPUT_DIRECTORY "${CMAKE_BINARY_DIR}/bin/org.kde.kwin.waylandbackends/") 28​set_target_properties(KWinWaylandDrmBackend PROPERTIES LIBRARY_OUTPUT_DIRECTORY "${CMAKE_BINARY_DIR}/bin/org.kde.kwin.waylandbackends/")
28target_link_libraries(KWinWaylandDrmBackend kwin Libdrm::Libdrm SceneQPainterBackend SceneOpenGLBackend) 29​target_link_libraries(KWinWaylandDrmBackend kwin Libdrm::Libdrm SceneQPainterBackend SceneOpenGLBackend dl wayland-server)
graesslinUnsubmitted
Not Done

Why are these two libraries needed?

graesslin: Why are these two libraries needed?
ekurzingerAuthorUnsubmitted
Not Done

wayland-server is needed in order to register the wl_eglstream_controller interface. The NVIDIA driver uses this when an application creates a native EGL window to pass the corresponding EGLStream to the compositor

dl is needed because we dynamically load libnvidia-egl-wayland.so since that library would only be present on systems with the NVIDIA driver installed, so we would only want to require it if using the EGLStream backend.

ekurzinger: wayland-server is needed in order to register the wl_eglstream_controller interface. The NVIDIA…
fredrikUnsubmitted
Not Done

You can probably avoid linking to dl by using QLibrary:

https://doc.qt.io/qt-5/qlibrary.html

fredrik: You can probably avoid linking to dl by using QLibrary: https://doc.qt.io/qt-5/qlibrary.html
29 30​
30if(HAVE_GBM) 31​if(HAVE_GBM)
31 target_link_libraries(KWinWaylandDrmBackend gbm::gbm) 32​ target_link_libraries(KWinWaylandDrmBackend gbm::gbm)
32endif() 33​endif()
33 34​
34install( 35​install(
35 TARGETS 36​ TARGETS
36 KWinWaylandDrmBackend 37​ KWinWaylandDrmBackend
37 DESTINATION 38​ DESTINATION
38 ${PLUGIN_INSTALL_DIR}/org.kde.kwin.waylandbackends/ 39​ ${PLUGIN_INSTALL_DIR}/org.kde.kwin.waylandbackends/
39) 40​)