Skip to content

Require CMake 3.10 or newer#98

Merged
Ybalrid merged 2 commits intowgois:masterfrom
gruenich:master
Oct 14, 2025
Merged

Require CMake 3.10 or newer#98
Ybalrid merged 2 commits intowgois:masterfrom
gruenich:master

Conversation

@gruenich
Copy link
Contributor

@gruenich gruenich commented Jul 8, 2025

Summary of changes
Increase minimum required version of CMake to 3.10.

Affected backeds (DirectInput, X11... Fill in if applicable)
does not apply

CMake 4.0 refuses to work with minimum
require of CMake 3.4 or below. Everything
below 3.10 is deprecated.
@gruenich
Copy link
Contributor Author

gruenich commented Jul 8, 2025

Once this is merged, would be nice to get a release to make packaging easier. (#97)

@gruenich
Copy link
Contributor Author

gruenich commented Oct 8, 2025

@Ybalrid Do you mind merging this?

@Sirius902
Copy link
Contributor

It looks like this line must also be removed from the CMakeLists.txt otherwise the build will fail on Linux with newer CMake. Here's the patch for that.

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 98c67d3..006b36b 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -218,7 +218,6 @@ endif()
 if(UNIX)
 
     if (NOT APPLE)
-        add_dependencies(OIS X11)
         target_link_libraries(OIS X11)
     endif()
 

@gruenich
Copy link
Contributor Author

@Sirius902 I added a second commit with you as the author. Please check.

@Ybalrid Ybalrid self-assigned this Oct 14, 2025
@Ybalrid Ybalrid merged commit 152ca18 into wgois:master Oct 14, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants