cmake: move OS compatibility checks into subdirectories

The top-level CMakeLists.txt shouldn't need to know these prerequisites for each implementation.
energymon · Aug 9, 2022 · d022fee · d022fee
1 parent 1a3d4b6
commit d022fee
Show file tree

Hide file tree

Showing 16 changed files with 74 additions and 31 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -158,31  158,22 @@ endfunction()
 
 # Libraries
 
-add_subdirectory(dummy)
-
-if(UNIX)
-  add_subdirectory(shmem)
-endif()
-
-if(${CMAKE_SYSTEM_NAME} MATCHES "Linux|Android")
-  add_subdirectory(cray-pm)
-  add_subdirectory(ibmpowernv)
-  add_subdirectory(jetson)
-  add_subdirectory(msr)
-  add_subdirectory(odroid)
-  add_subdirectory(rapl)
-  add_subdirectory(raplcap-msr)
-  add_subdirectory(zcu102)
-endif()
-
-if(UNIX OR WIN32)
-  add_subdirectory(osp)
-  add_subdirectory(wattsup)
-endif()
 # Must be added before libraries that build shmem providers
 add_subdirectory(shmem)
 
-if(APPLE)
-  add_subdirectory(ipg)
-endif()
 # Please keep these in alphabetical order
 add_subdirectory(cray-pm)
 add_subdirectory(dummy)
 add_subdirectory(ibmpowernv)
 add_subdirectory(ipg)
 add_subdirectory(jetson)
 add_subdirectory(msr)
 add_subdirectory(odroid)
 add_subdirectory(osp)
 add_subdirectory(rapl)
 add_subdirectory(raplcap-msr)
 add_subdirectory(wattsup)
 add_subdirectory(zcu102)
 
 if(NOT TARGET energymon-default AND NOT ${ENERGYMON_BUILD_DEFAULT} MATCHES "NONE")
   message(FATAL_ERROR

diff --git a/cray-pm/CMakeLists.txt b/cray-pm/CMakeLists.txt
@@ -1,3  1,7 @@
 if(NOT ${CMAKE_SYSTEM_NAME} MATCHES "Linux|Android")
   return()
 endif()
 
 set(SNAME cray-pm)
 set(LNAME energymon-cray-pm)
 set(SOURCES ${LNAME}.c;

diff --git a/ibmpowernv/CMakeLists.txt b/ibmpowernv/CMakeLists.txt
@@ -1,3  1,7 @@
 if(NOT ${CMAKE_SYSTEM_NAME} MATCHES "Linux|Android")
   return()
 endif()
 
 set(SNAME ibmpowernv)
 set(SNAME_POWER ibmpowernv-power)
 set(LNAME energymon-ibmpowernv)

diff --git a/ipg/CMakeLists.txt b/ipg/CMakeLists.txt
@@ -1,3  1,7 @@
 if(NOT APPLE)
   return()
 endif()
 
 set(SNAME ipg)
 set(LNAME energymon-ipg)
 set(SOURCES ${LNAME}.c;${ENERGYMON_UTIL})

diff --git a/jetson/CMakeLists.txt b/jetson/CMakeLists.txt
@@ -1,3  1,7 @@
 if(NOT ${CMAKE_SYSTEM_NAME} MATCHES "Linux|Android")
   return()
 endif()
 
 set(SNAME jetson)
 set(LNAME energymon-jetson)
 set(SOURCES ${LNAME}.c;${ENERGYMON_UTIL};${ENERGYMON_TIME_UTIL};util.c;ina3221.c;ina3221x.c)

diff --git a/msr/CMakeLists.txt b/msr/CMakeLists.txt
@@ -1,3  1,7 @@
 if(NOT ${CMAKE_SYSTEM_NAME} MATCHES "Linux|Android")
   return()
 endif()
 
 set(SNAME msr)
 set(LNAME energymon-msr)
 set(SOURCES ${LNAME}.c;${ENERGYMON_UTIL})

diff --git a/odroid/CMakeLists.txt b/odroid/CMakeLists.txt
@@ -1,3  1,7 @@
 if(NOT ${CMAKE_SYSTEM_NAME} MATCHES "Linux|Android")
   return()
 endif()
 
 set(SNAME odroid)
 set(LNAME energymon-odroid)
 set(SNAME_IOCTL odroid-ioctl)

diff --git a/osp/CMakeLists.txt b/osp/CMakeLists.txt
@@ -1,3  1,7 @@
 if(NOT (UNIX OR WIN32))
   return()
 endif()
 
 set(SNAME osp)
 set(LNAME energymon-osp)
 set(SNAME_POLLING osp-polling)

diff --git a/rapl/CMakeLists.txt b/rapl/CMakeLists.txt
@@ -1,3  1,7 @@
 if(NOT ${CMAKE_SYSTEM_NAME} MATCHES "Linux|Android")
   return()
 endif()
 
 set(SNAME rapl)
 set(LNAME energymon-rapl)
 set(SOURCES ${LNAME}.c;${ENERGYMON_UTIL})

diff --git a/raplcap-msr/CMakeLists.txt b/raplcap-msr/CMakeLists.txt
@@ -1,3  1,7 @@
 if(NOT ${CMAKE_SYSTEM_NAME} MATCHES "Linux|Android")
   return()
 endif()
 
 set(SNAME raplcap-msr)
 set(LNAME energymon-raplcap-msr)
 set(SOURCES ${LNAME}.c;${ENERGYMON_UTIL})

diff --git a/shmem/CMakeLists.txt b/shmem/CMakeLists.txt
@@ -1,3  1,7 @@
 if(NOT UNIX)
   return()
 endif()
 
 set(SNAME shmem)
 set(LNAME energymon-shmem)
 set(EXAMPLE energymon-shmem-example)

diff --git a/wattsup/CMakeLists.txt b/wattsup/CMakeLists.txt
@@ -11,10  11,6 @@ endif()
 
 # Libraries
 
-if(UNIX)
-  add_subdirectory(dev)
-endif()
-if(UNIX OR WIN32)
-  add_subdirectory(libftdi)
-  add_subdirectory(libusb)
-endif()
 add_subdirectory(dev)
 add_subdirectory(libftdi)
 add_subdirectory(libusb)
diff --git a/wattsup/dev/CMakeLists.txt b/wattsup/dev/CMakeLists.txt
@@ -1,3  1,7 @@
 if(NOT UNIX)
   return()
 endif()
 
 set(SNAME wattsup)
 set(LNAME energymon-wattsup)
 set(SOURCES ../energymon-wattsup.c;wattsup-driver-dev.c;${ENERGYMON_UTIL};${ENERGYMON_TIME_UTIL})

diff --git a/wattsup/libftdi/CMakeLists.txt b/wattsup/libftdi/CMakeLists.txt
@@ -1,3  1,7 @@
 if(NOT (UNIX OR WIN32))
   return()
 endif()
 
 set(SNAME wattsup-libftdi)
 set(LNAME energymon-wattsup-libftdi)
 set(SOURCES ../energymon-wattsup.c;wattsup-driver-libftdi.c;${ENERGYMON_UTIL};${ENERGYMON_TIME_UTIL})

diff --git a/wattsup/libusb/CMakeLists.txt b/wattsup/libusb/CMakeLists.txt
@@ -1,3  1,7 @@
 if(NOT (UNIX OR WIN32))
   return()
 endif()
 
 set(SNAME wattsup-libusb)
 set(LNAME energymon-wattsup-libusb)
 set(SOURCES ../energymon-wattsup.c;wattsup-driver-libusb.c;${ENERGYMON_UTIL};${ENERGYMON_TIME_UTIL})

diff --git a/zcu102/CMakeLists.txt b/zcu102/CMakeLists.txt
@@ -1,3  1,7 @@
 if(NOT ${CMAKE_SYSTEM_NAME} MATCHES "Linux|Android")
   return()
 endif()
 
 set(SNAME zcu102)
 set(LNAME energymon-zcu102)
 set(SOURCES ${LNAME}.c;${ENERGYMON_UTIL};${ENERGYMON_TIME_UTIL})