Link to X11 targets instead of CMake variables (AcademySoftwareFounda…

…tion#1606) This prevents absolute paths from creeping into the output MaterialXConfig.cmake, which makes it not portable. Also, in general in modern CMake, targets are preferred over baked paths.
KhronosGroup · Nov 29, 2023 · b8cc4de · b8cc4de
1 parent b50dea8
commit b8cc4de
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/source/MaterialXRenderGlsl/CMakeLists.txt b/source/MaterialXRenderGlsl/CMakeLists.txt
@@ -78,8 +78,8 @@ elseif(UNIX)
     target_link_libraries(
         ${MATERIALX_MODULE_NAME}
         OpenGL::GL
-        ${X11_LIBRARIES}
-        ${X11_Xt_LIB})
+        X11::X11
+        X11::Xt)
 endif()
 
 set_target_properties(

diff --git a/source/MaterialXRenderHw/CMakeLists.txt b/source/MaterialXRenderHw/CMakeLists.txt
@@ -57,8 +57,8 @@ elseif(UNIX)
         ${MATERIALX_MODULE_NAME}
         MaterialXRender
         ${CMAKE_DL_LIBS}
-        ${X11_LIBRARIES}
-        ${X11_Xt_LIB})
+        X11::X11
+        X11::Xt)
 endif()
 
 set_target_properties(

diff --git a/source/MaterialXRenderMsl/CMakeLists.txt b/source/MaterialXRenderMsl/CMakeLists.txt
@@ -67,8 +67,8 @@ elseif(UNIX)
     target_link_libraries(
         ${MATERIALX_MODULE_NAME}
         OpenGL::GL
-        ${X11_LIBRARIES}
-        ${X11_Xt_LIB})
+        X11::X11
+        X11::Xt)
 endif()
 
 set_target_properties(