Skip to content

Commit

Permalink
Prevent cmake warnings and errors depending on the order of find_pack…
Browse files Browse the repository at this point in the history
…age() for ROS
  • Loading branch information
jlblancoc committed Jun 20, 2024
1 parent 4885c63 commit 5f1578d
Showing 1 changed file with 7 additions and 2 deletions.
9 changes: 7 additions & 2 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -254,6 +254,13 @@ if(NOT MSVC AND NOT XCODE_VERSION)
endif()
endif()

# Leave these includes here before ROS to prevent the warning:
# cmakemodules/ECMFindModuleHelpers.cmake:112 (message):
# Your project should require at least CMake 3.16.0 to use FindEGL.cmake
#
include(cmakemodules/script_gl_glut.cmake REQUIRED) # Check for the GL,GLUT libraries
include(cmakemodules/script_jsoncpp.cmake REQUIRED) # Check for jsoncpp

# ----------------------------------------------------------------------------
# ROS
# ----------------------------------------------------------------------------
Expand All @@ -275,14 +282,12 @@ include(cmakemodules/script_ffmpeg.cmake REQUIRED) # Check for ffmpeg C lib
include(cmakemodules/script_flycapture2.cmake REQUIRED) # Check for PointGreyResearch (PGR) FlyCapture2 library
include(cmakemodules/script_ftdi.cmake REQUIRED) # Check for the FTDI headers (Linux only, in win32 we use built-in header & dynamic DLL load):
include(cmakemodules/script_gcc_clang_id.cmake REQUIRED) # Helper variables
include(cmakemodules/script_gl_glut.cmake REQUIRED) # Check for the GL,GLUT libraries
include(cmakemodules/script_gridmap_options.cmake REQUIRED) # Gridmap options
include(cmakemodules/script_gtest.cmake REQUIRED) # Unit testing lib
include(cmakemodules/script_inotify.cmake REQUIRED) # Check for the sys/inotify headers (Linux only, in win32 we use the equivalent API for file system monitoring):
include(cmakemodules/script_isense.cmake REQUIRED) # Support for INTERSENSE Sensors
include(cmakemodules/script_iwyu.cmake REQUIRED) # Include-what-you-use
include(cmakemodules/script_jpeg.cmake REQUIRED) # Check for jpeg
include(cmakemodules/script_jsoncpp.cmake REQUIRED) # Check for jsoncpp
include(cmakemodules/script_kinect.cmake REQUIRED) # Kinect support in a set of different ways
include(cmakemodules/script_libdc1394.cmake REQUIRED) # Check for libdc1394-2
include(cmakemodules/script_libfyaml.cmake REQUIRED) # Defines embedded version of libfyaml
Expand Down

0 comments on commit 5f1578d

Please sign in to comment.