Fixes for macos build (#38)

* set C++ standard to 17

* link to spdlog

Co-authored-by: svetad <sv.dmytr@gmail.com>
Co-authored-by: G. Dan Miron <dan.miron@psi.com>

CMakeLists.txt

@@ -53,8 +53,8 @@ if(NOT CMAKE_BUILD_TYPE)
 endif()
 
 # Set the C++ standard
-set(CMAKE_CXX_STANDARD 14)
-set(CMAKE_CXX_STANDARD_REQUIRED ON)
+set(CMAKE_CXX_STANDARD 17)
+#set(CMAKE_CXX_STANDARD_REQUIRED ON)
 
 # Set the list of compiler flags for MSVC compiler
 if(${CMAKE_CXX_COMPILER_ID} STREQUAL MSVC)

ThermoFun/CMakeLists.txt

@@ -32,6 +32,9 @@ if(ChemicalFun_FOUND)
     target_link_libraries(ThermoFun PUBLIC ChemicalFun::ChemicalFun)
 endif()
 
+target_link_libraries(ThermoFun PRIVATE spdlog::spdlog)
+
+
 if (CMAKE_CXX_COMPILER_ID MATCHES "MSVC")
     #link_directories("/usr/local/lib")
 else() 

azure-pipelines.yml

@@ -19,7 +19,7 @@ jobs:
 
 - job: macOS
   pool:
-    vmImage: 'macos-12'
+    vmImage: 'macOS-11'
 
   steps:
   - task: Bash@3

ci/pipelines/install/install-osx.sh

@@ -25,9 +25,10 @@ cmake \
     -DCMAKE_INSTALL_LIBDIR=lib \
     ..
 make install
+conda list
+python -c "import thermofun"
 if [ $? -eq 1 ]
 then
 echo "The install failed" >&2
 exit 1
 fi
-conda list

cmake/modules/ThermoFunFindDeps.cmake

@@ -19,3 +19,10 @@ if(NOT ChemicalFun_FOUND)
 else()
   message(STATUS "Found ChemicalFun v${ChemicalFun_VERSION}")
 endif()
+
+find_package(spdlog REQUIRED)
+if(NOT spdlog_FOUND)
+  message(FATAL_ERROR "spdlog not found")
+else()
+  message(STATUS "Found spdlog v${spdlog_VERSION}")
+endif()

environment.devenv.yml

@@ -3,16 +3,16 @@ name: thermofun
 {% set python_version = os.environ.get("PY_VER", "3.7") %}
 
 dependencies:
-  - pip
   - setuptools=59.7.0
-  - gxx_linux-64=7.3.0  # [linux]
+  - pip
+  - gxx_linux-64=9.4.0  # [linux]
   - cmake>=3.13
   - ninja
   - ccache  # [unix]
   - pybind11
   - nlohmann_json
-  - fmt>=8.1.1
-  - chemicalfun>=0.1.3
+  - spdlog=1.10.0
+  - chemicalfun=0.1.4
   - python={{ python_version }}
   - pytest
   - pip: