fix: Properly fix GLFW Linux backend detection

WerWolv · Jul 30, 2024 · 0038e69 · 0038e69
1 parent 2732f00
commit 0038e69
Show file tree

Hide file tree

Showing 2 changed files with 30 additions and 4 deletions.
diff --git a/lib/libimhex/CMakeLists.txt b/lib/libimhex/CMakeLists.txt
@@ -80,6 +80,32 @@ else()
     target_compile_definitions(libimhex PRIVATE IMHEX_PROJECT_NAME="${PROJECT_NAME}")
 endif()
 
+if (UNIX AND NOT APPLE)
+    # For Linux, we need to figure out whether our GLFW is built for X11, Wayland, or both
+    INCLUDE(CheckCXXSourceCompiles)
+    set(CMAKE_REQUIRED_LIBRARIES ${GLFW_LIBRARIES})
+    check_cxx_source_compiles("
+        #define GLFW_EXPOSE_NATIVE_X11
+        #include <GLFW/glfw3.h>
+        #include <GLFW/glfw3native.h>
+        Window test(GLFWwindow* w) { return glfwGetX11Window(w); }
+        int main() {}
+    " GLFW_HAS_X11)
+    check_cxx_source_compiles("
+        #define GLFW_EXPOSE_NATIVE_WAYLAND
+        #include <GLFW/glfw3.h>
+        #include <GLFW/glfw3native.h>
+        wl_surface* test(GLFWwindow* w) { return glfwGetWaylandWindow(w); }
+        int main() {}
+    " GLFW_HAS_WAYLAND)
+    if (GLFW_HAS_X11)
+        target_compile_definitions(libimhex PRIVATE GLFW_BACKEND_X11)
+    endif()
+    if (GLFW_HAS_WAYLAND)
+        target_compile_definitions(libimhex PRIVATE GLFW_BACKEND_WAYLAND)
+    endif()
+endif()
+
 
 if (DEFINED IMHEX_COMMIT_HASH_LONG AND DEFINED IMHEX_COMMIT_BRANCH)
     set(GIT_COMMIT_HASH_LONG "${IMHEX_COMMIT_HASH_LONG}")

diff --git a/lib/libimhex/source/helpers/fs.cpp b/lib/libimhex/source/helpers/fs.cpp
@@ -35,17 +35,17 @@
         typedef void NSWindow;
         #define GLFW_EXPOSE_NATIVE_COCOA
     #endif
-    #if defined(OS_LINUX)
+    #if defined(OS_LINUX) && defined(GLFW_BACKEND_X11)
         #define GLFW_EXPOSE_NATIVE_X11
     #endif
-    #if defined(OS_LINUX) && defined(GLFW_WAYLAND_APP_ID)
+    #if defined(OS_LINUX) && defined(GLFW_BACKEND_WAYLAND)
         #define GLFW_EXPOSE_NATIVE_WAYLAND
     #endif
     #include <nfd_glfw3.h>
-    #if defined(OS_LINUX) && defined(GLFW_WAYLAND_APP_ID)
+    #if defined(OS_LINUX) && defined(GLFW_BACKEND_WAYLAND)
         #undef GLFW_EXPOSE_NATIVE_WAYLAND
     #endif
-    #if defined(OS_LINUX)
+    #if defined(OS_LINUX) && defined(GLFW_BACKEND_X11)
         #undef GLFW_EXPOSE_NATIVE_X11
     #endif
     #if defined(OS_MACOS)