Build System "Run CMAKE" fails with CMAKE

[jbrzensk]

Possibly due to new CMAKE 4.0, which removes compatibility with CMAKE <3.5.

CMake Error at CMakeLists.txt:8 (cmake_minimum_required):
-- Configuring incomplete, errors occurred!
  Compatibility with CMake < 3.5 has been removed from CMake.

@cpaolini @JananiPSrinivasan @mdumett
This affects pull requests currently failing:
#100 , #61 , #63

[cpaolini]

@jbrzensk @JananiPSrinivasan @mdumett

The CMakeLists.txt file that comes with the superlu-5.3.0.tar.gz download contains

cmake_minimum_required(VERSION 2.8.12)

We can use sed -i to change this to

cmake_minimum_required(VERSION 3.10)

prior to running cmake to configure superlu.

This will remove the "Compatibility with CMake < 3.5 has been removed from CMake" error.

[JananiPSrinivasan]

Issue fixed

The cmake minimum version has been set to 3.10 in the superLU CMakeLists.txt by adding the sed -i in the global CMakeLists.txt

Output on Ubuntu 22.04

-- The CXX compiler identification is GNU 11.4.0
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Detected CXX Compiler ID: GNU
-- Using non-Clang compiler flags.
-- SuperLU CMakeLists.txt after patch: cmake_minimum_required(VERSION 3.10)

Output on MacOS (intel Architecture)

janani@Christophers-MacBook-Pro build % cmake ..
-- The CXX compiler identification is AppleClang 12.0.0.12000032
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /Library/Developer/CommandLineTools/usr/bin/c++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Detected CXX Compiler ID: AppleClang
-- Using AppleClang-specific flags.
-- Found Eigen3: /usr/local/include/eigen3 (Required is at least version "3.3.7")
-- SuperLU CMakeLists.txt after patch: cmake_minimum_required(VERSION 3.10)

@cpaolini @jbrzensk

[cpaolini]

Excellent! Yes, we tested this together in the lab on notos with cmake 4.0.