-
Notifications
You must be signed in to change notification settings - Fork 511
Impossible to compile under Linux LTS distributions #519
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
What is the exact error you are getting from CMake? The required CMake version is 3.2, not 3.11, or at least it should be according to our |
Think, @AlexanderP could provide more error details |
Hi.
|
Ok, the CMake behavior around diff --git a/CMakeLists.txt b/CMakeLists.txt
index 9bfd04d0..9bede12c 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -7,8 +7,10 @@ if(CMAKE_SOURCE_DIR STREQUAL CMAKE_BINARY_DIR)
" mkdir build && cd build && cmake ..")
endif()
-cmake_minimum_required(VERSION 3.2.0 FATAL_ERROR)
-cmake_policy(VERSION 3.11)
+cmake_minimum_required(VERSION 3.10.2 FATAL_ERROR)
+if(NOT CMAKE_VERSION VERSION_LESS 3.11.0)
+ cmake_policy(VERSION 3.11.0)
+endif()
set(CMAKE_MODULE_PATH ${CMAKE_MODULE_PATH}
"${CMAKE_SOURCE_DIR}/cmake/")
set(CMAKE_CXX_STANDARD 11) |
FTR, Debian 9.x stretch has only CMake |
Ubuntu Bionic. Build was successful |
What about Debian stretch? |
|
Fixed in 0b786e6. |
System information
3.0~0472b9e
Expected behavior
SolveSpace should keep support to build (at least) on Linux LTS distros.
Actual behavior
SolveSpace could not be compiled under Linux LTS distros, as
master
actually require CMake3.11
dependency.FTR, As non-trivial solution, user should try to compile CMake
3.11
manually before build SolveSpace3.x
.Additional information
For bugs, please attach a savefile that shows the problematic behavior.
You can attach
.slvs
files by archiving them into a.zip
first.The text was updated successfully, but these errors were encountered: