compile options target only:

Author: scivision

CMakeLists.txt

@@ -1,4 +1,4 @@
-cmake_minimum_required(VERSION 3.21...4.3)
+cmake_minimum_required(VERSION 3.22...4.3)
 
 if(CMAKE_SOURCE_DIR STREQUAL CMAKE_BINARY_DIR)
   message(FATAL_ERROR "In-source builds are not allowed. Do like:
@@ -17,6 +17,8 @@ VERSION 1.7.2
 
 enable_testing()
 
+include(GNUInstallDirs)
+
 list(APPEND CMAKE_MODULE_PATH ${CMAKE_CURRENT_SOURCE_DIR}/cmake)
 
 include(options.cmake)
@@ -48,12 +50,13 @@ endif()
 
 add_library(nc4fortran)
 target_include_directories(nc4fortran PUBLIC
-$<BUILD_INTERFACE:${CMAKE_CURRENT_BINARY_DIR}/include>
-$<INSTALL_INTERFACE:include>
+$<BUILD_INTERFACE:${CMAKE_CURRENT_BINARY_DIR}/${CMAKE_INSTALL_INCLUDEDIR}>
+$<INSTALL_INTERFACE:${CMAKE_INSTALL_INCLUDEDIR}>
 )
+target_compile_options(nc4fortran PRIVATE ${nc4fortran_fflags})
 target_link_libraries(nc4fortran PUBLIC NetCDF::NetCDF_Fortran)
 set_target_properties(nc4fortran PROPERTIES
-Fortran_MODULE_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}/include
+Fortran_MODULE_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}/${CMAKE_INSTALL_INCLUDEDIR}
 LABELS core
 VERSION ${PROJECT_VERSION}
 )
@@ -70,11 +73,9 @@ endif()
 
 add_subdirectory(src)
 
-# GLOBAL needed for FetchContent use
-add_library(nc4fortran::nc4fortran INTERFACE IMPORTED GLOBAL)
-target_link_libraries(nc4fortran::nc4fortran INTERFACE nc4fortran)
+add_library(nc4fortran::nc4fortran ALIAS nc4fortran)
 
-install(FILES ${CMAKE_CURRENT_BINARY_DIR}/include/nc4fortran.mod TYPE INCLUDE)
+install(FILES ${CMAKE_CURRENT_BINARY_DIR}/${CMAKE_INSTALL_INCLUDEDIR}/nc4fortran.mod TYPE INCLUDE)
 
 install(TARGETS nc4fortran EXPORT nc4fortran-targets)
 

cmake/compilers.cmake

@@ -1,40 +1,19 @@
-# --- C compile flags
-if(CMAKE_C_COMPILER_ID MATCHES "Clang|GNU|^Intel")
-  add_compile_options(
-  "$<$<AND:$<COMPILE_LANGUAGE:C>,$<CONFIG:Debug>>:-Wextra>"
-  "$<$<COMPILE_LANGUAGE:C>:-Wall>"
-  "$<$<COMPILE_LANGUAGE:C>:-Werror=implicit-function-declaration>"
-  )
-elseif(CMAKE_C_COMPILER_ID MATCHES "MSVC")
-  add_compile_options("$<$<COMPILE_LANGUAGE:C>:/W3>")
-endif()
+# --- Fortran compile flags
 
-if(WIN32)
-  if(CMAKE_C_COMPILER_ID MATCHES "^Intel|MSVC")
-    add_compile_options($<$<AND:$<COMPILE_LANGUAGE:C>,$<CONFIG:Debug>>:/Od>)
-  endif()
-elseif(CMAKE_C_COMPILER_ID MATCHES "^Intel")
-  add_compile_options($<$<AND:$<COMPILE_LANGUAGE:C>,$<CONFIG:Debug>>:-O0>)
-endif()
+set(nc4fortran_fflags)
 
-# --- Fortran compile flags
 if(CMAKE_Fortran_COMPILER_ID MATCHES "^Intel")
 
-add_compile_options(
-"$<$<COMPILE_LANGUAGE:Fortran>:-warn>"
+list(APPEND nc4fortran_fflags
+"$<$<AND:$<COMPILE_LANGUAGE:Fortran>,$<CONFIG:Debug,RelWithDebInfo>>:-warn>"
 "$<$<AND:$<COMPILE_LANGUAGE:Fortran>,$<CONFIG:Debug>>:-traceback;-check;-debug>"
 )
 
-if(WIN32)
-  add_compile_options($<$<AND:$<COMPILE_LANGUAGE:Fortran>,$<CONFIG:Debug>>:/Od>)
-else()
-  add_compile_options($<$<AND:$<COMPILE_LANGUAGE:Fortran>,$<CONFIG:Debug>>:-O0>)
-endif()
-
 elseif(CMAKE_Fortran_COMPILER_ID STREQUAL "GNU")
 
-add_compile_options(
-"$<$<COMPILE_LANGUAGE:Fortran>:-Wall;-fimplicit-none;-Wno-maybe-uninitialized>"
+list(APPEND nc4fortran_fflags
+"$<$<COMPILE_LANGUAGE:Fortran>:-fimplicit-none>"
+"$<$<AND:$<COMPILE_LANGUAGE:Fortran>,$<CONFIG:Debug,RelWithDebInfo>>:-Wall>"
 "$<$<AND:$<COMPILE_LANGUAGE:Fortran>,$<CONFIG:Debug>>:-Wextra;-fcheck=all;-Werror=array-bounds>"
 "$<$<AND:$<COMPILE_LANGUAGE:Fortran>,$<CONFIG:Release>>:-fno-backtrace>"
 )