code clean up for POSIX.

Drv/CMakeLists.txt

@@ -15,12 +15,7 @@ add_fprime_subdirectory("${CMAKE_CURRENT_LIST_DIR}/LinuxSpiDriver/")
 add_fprime_subdirectory("${CMAKE_CURRENT_LIST_DIR}/LinuxI2cDriver/")
 add_fprime_subdirectory("${CMAKE_CURRENT_LIST_DIR}/StreamCrossover/")
 
-# IP Socket is only supported for Linux, Darwin, VxWorks
-if(${CMAKE_SYSTEM_NAME} STREQUAL "Linux" OR ${CMAKE_SYSTEM_NAME} STREQUAL "Darwin" OR ${CMAKE_SYSTEM_NAME} STREQUAL "VxWorks")
-	add_fprime_subdirectory("${CMAKE_CURRENT_LIST_DIR}/Ip/")
-	add_fprime_subdirectory("${CMAKE_CURRENT_LIST_DIR}/TcpClient/")
-	add_fprime_subdirectory("${CMAKE_CURRENT_LIST_DIR}/TcpServer/")
-	add_fprime_subdirectory("${CMAKE_CURRENT_LIST_DIR}/Udp/")
-else()
-    message(STATUS "Cannot use IP sockets with platform ${CMAKE_SYSTEM_NAME}. Skipping.")
-endif()
+add_fprime_subdirectory("${CMAKE_CURRENT_LIST_DIR}/Ip/")
+add_fprime_subdirectory("${CMAKE_CURRENT_LIST_DIR}/TcpClient/")
+add_fprime_subdirectory("${CMAKE_CURRENT_LIST_DIR}/TcpServer/")
+add_fprime_subdirectory("${CMAKE_CURRENT_LIST_DIR}/Udp/")

Drv/Ip/CMakeLists.txt

@@ -5,6 +5,8 @@
 # MOD_DEPS: (optional) module dependencies
 #
 ####
+restrict_platforms(Posix)
+
 set(SOURCE_FILES
 	"${CMAKE_CURRENT_LIST_DIR}/IpSocket.cpp"
 	"${CMAKE_CURRENT_LIST_DIR}/TcpClientSocket.cpp"

Drv/TcpClient/CMakeLists.txt

@@ -5,6 +5,8 @@
 # MOD_DEPS: (optional) module dependencies
 #
 ####
+restrict_platforms(Posix)
+
 set(SOURCE_FILES
 	"${CMAKE_CURRENT_LIST_DIR}/TcpClient.fpp"
 	"${CMAKE_CURRENT_LIST_DIR}/TcpClientComponentImpl.cpp"

Drv/TcpServer/CMakeLists.txt

@@ -5,6 +5,8 @@
 # MOD_DEPS: (optional) module dependencies
 #
 ####
+restrict_platforms(Posix)
+
 set(SOURCE_FILES
 	"${CMAKE_CURRENT_LIST_DIR}/TcpServer.fpp"
 	"${CMAKE_CURRENT_LIST_DIR}/TcpServerComponentImpl.cpp"

Drv/Udp/CMakeLists.txt

@@ -5,6 +5,8 @@
 # MOD_DEPS: (optional) module dependencies
 #
 ####
+restrict_platforms(Posix)
+
 set(SOURCE_FILES
 	"${CMAKE_CURRENT_LIST_DIR}/Udp.fpp"
 	"${CMAKE_CURRENT_LIST_DIR}/UdpComponentImpl.cpp"

Os/Posix/CMakeLists.txt

@@ -5,7 +5,7 @@
 # MOD_DEPS: (optional) module dependencies
 #
 ####
-restrict_platforms(Linux Darwin)
+restrict_platforms(Posix)
 add_custom_target("${FPRIME_CURRENT_MODULE}")
 set(SOURCE_FILES
     "${CMAKE_CURRENT_LIST_DIR}/File.cpp"

Svc/CMakeLists.txt

@@ -53,7 +53,5 @@ if (FPRIME_ENABLE_TEXT_LOGGERS)
 	add_fprime_subdirectory("${CMAKE_CURRENT_LIST_DIR}/ActiveTextLogger/")
 endif()
 
-if(${CMAKE_SYSTEM_NAME} STREQUAL "Linux" OR ${CMAKE_SYSTEM_NAME} STREQUAL "Darwin")
-	add_fprime_subdirectory("${CMAKE_CURRENT_LIST_DIR}/PosixTime/")
-	add_fprime_subdirectory("${CMAKE_CURRENT_LIST_DIR}/LinuxTimer/")
-endif()
+add_fprime_subdirectory("${CMAKE_CURRENT_LIST_DIR}/PosixTime/")
+add_fprime_subdirectory("${CMAKE_CURRENT_LIST_DIR}/LinuxTimer/")

Svc/LinuxTimer/CMakeLists.txt

@@ -5,6 +5,7 @@
 # MOD_DEPS: (optional) module dependencies
 #
 ####
+restrict_platforms(Linux Darwin)
 
 
 if(${CMAKE_SYSTEM_NAME} STREQUAL "Darwin")

Svc/PosixTime/CMakeLists.txt

@@ -6,6 +6,8 @@
 #
 # Note: using PROJECT_NAME as EXECUTABLE_NAME
 ####
+restrict_platforms(Posix)
+
 set(SOURCE_FILES
   "${CMAKE_CURRENT_LIST_DIR}/PosixTime.fpp"
   "${CMAKE_CURRENT_LIST_DIR}/PosixTime.cpp"

cmake/API.cmake

@@ -31,7 +31,17 @@ set(FPRIME_AUTOCODER_TARGET_LIST "" CACHE INTERNAL "FPRIME_AUTOCODER_TARGET_LIST
 #####
 macro(restrict_platforms)
     set(__CHECKER ${ARGN})
-    if (NOT FPRIME_TOOLCHAIN_NAME IN_LIST __CHECKER AND NOT FPRIME_PLATFORM IN_LIST __CHECKER)
+
+    # Each of these empty if blocks are the valid-case, that is, the platform is supported.
+    # However, the reason why this is necessary is that this function is a macro and not a function.
+    # Macros copy-paste the code into the calling context. Thus, all these valid cases want to avoid calling return.
+    # The return call  in the else block returns from the calling context (i.e. a restricted CMakeList.txt will
+    # return and not process the component setup). We do not want this return when the platform is allowed.
+
+    if (FPRIME_TOOLCHAIN_NAME IN_LIST __CHECKER)
+    elseif(FPRIME_PLATFORM IN_LIST __CHECKER)
+    elseif("Posix" IN_LIST __CHECKER AND FPRIME_USE_POSIX)
+    else()
         get_module_name("${CMAKE_CURRENT_LIST_DIR}")
         message(STATUS "Neither toolchain ${FPRIME_TOOLCHAIN_NAME} nor platform ${FPRIME_PLATFORM} supported for module ${MODULE_NAME}")
         append_list_property("${MODULE_NAME}" GLOBAL PROPERTY RESTRICTED_TARGETS)