Skip to content

Commit

Permalink
Warn instead of fail when dds.communication.dynamic_interfaces cann…
Browse files Browse the repository at this point in the history
…ot be built (#5310)

* Refs #21826. Fix

Signed-off-by: Ricardo González Moreno <ricardo@richiware.dev>

* Refs #21826. Apply suggestions

Signed-off-by: Ricardo González Moreno <ricardo@richiware.dev>

---------

Signed-off-by: Ricardo González Moreno <ricardo@richiware.dev>
  • Loading branch information
richiware authored Oct 9, 2024
1 parent ce4ce74 commit 9b680b5
Showing 1 changed file with 8 additions and 5 deletions.
13 changes: 8 additions & 5 deletions test/dds/communication/dyn_network/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,8 @@ message(STATUS "Configuring dynamic network interfaces tests")
# Find docker
find_program(DOCKER_EXECUTABLE docker)
if(NOT DOCKER_EXECUTABLE)
message(FATAL_ERROR "Docker not found")
message(WARNING "Docker executable not found. dds.communication.dynamic_interfaces test won't be compiled.")
return()
endif()

set(SHELL_EXECUTABLE "")
Expand All @@ -28,19 +29,21 @@ if(UNIX AND NOT(APPLE) AND NOT(QNXNTO) AND NOT(ANDROID))
# Find bash
find_program(BASH_EXECUTABLE bash)
if(NOT BASH_EXECUTABLE)
message(FATAL_ERROR "bash not found")
message(WARNING "bash executable not found. dds.communication.dynamic_interfaces test won't be compiled.")
return()
endif()

set(SHELL_EXECUTABLE ${BASH_EXECUTABLE})

# Windows configurations
elseif(WIN32)
# We don't know which docker image to use for Windows yet
message(FATAL_ERROR "Windows not supported yet")

message(WARNING "Windows not supported yet. dds.communication.dynamic_interfaces test won't be compiled.")
return()
# Unsupported platform
else()
message(FATAL_ERROR "Unsupported platform")
message(WARNING "Unsupported platform. dds.communication.dynamic_interfaces test won't be compiled.")
return()
endif()

# Configure TinyXML2 library path if installed in user library path
Expand Down

0 comments on commit 9b680b5

Please sign in to comment.