pal-robotics / backward_ros

MIT License
188 stars 25 forks source link

cmake warnings #10

Open tylerjw opened 2 years ago

tylerjw commented 2 years ago

I am using the Rolling release and get these warnings when following the instructions to use backward_ros:

--- stderr: moveit_ros_move_group                                           
CMake Warning (dev) at /usr/local/share/cmake-3.20/Modules/FindPackageHandleStandardArgs.cmake:438 (message):
  The package name passed to `find_package_handle_standard_args` (libdw) does
  not match the name of the calling package (backward_ros).  This can lead to
  problems in calling code that expects `find_package` result variables
  (e.g., `_FOUND`) to follow a certain pattern.
Call Stack (most recent call first):
  /opt/ros/rolling/share/backward_ros/cmake/BackwardConfig.cmake:55 (find_package_handle_standard_args)
  /opt/ros/rolling/share/backward_ros/cmake/backward_rosConfig.cmake:41 (include)
  /home/tyler/code/rolling/ws/install/moveit_common/share/moveit_common/cmake/moveit_package.cmake:36 (find_package)
  CMakeLists.txt:6 (moveit_package)
This warning is for project developers.  Use -Wno-dev to suppress it.

CMake Warning (dev) at /usr/local/share/cmake-3.20/Modules/FindPackageHandleStandardArgs.cmake:438 (message):
  The package name passed to `find_package_handle_standard_args` (libbfd)
  does not match the name of the calling package (backward_ros).  This can
  lead to problems in calling code that expects `find_package` result
  variables (e.g., `_FOUND`) to follow a certain pattern.
Call Stack (most recent call first):
  /opt/ros/rolling/share/backward_ros/cmake/BackwardConfig.cmake:66 (find_package_handle_standard_args)
  /opt/ros/rolling/share/backward_ros/cmake/backward_rosConfig.cmake:41 (include)
  /home/tyler/code/rolling/ws/install/moveit_common/share/moveit_common/cmake/moveit_package.cmake:36 (find_package)
  CMakeLists.txt:6 (moveit_package)
This warning is for project developers.  Use -Wno-dev to suppress it.

CMake Warning (dev) at /usr/local/share/cmake-3.20/Modules/FindPackageHandleStandardArgs.cmake:438 (message):
  The package name passed to `find_package_handle_standard_args` (Backward)
  does not match the name of the calling package (backward_ros).  This can
  lead to problems in calling code that expects `find_package` result
  variables (e.g., `_FOUND`) to follow a certain pattern.
Call Stack (most recent call first):
  /opt/ros/rolling/share/backward_ros/cmake/BackwardConfig.cmake:128 (find_package_handle_standard_args)
  /opt/ros/rolling/share/backward_ros/cmake/backward_rosConfig.cmake:41 (include)
  /home/tyler/code/rolling/ws/install/moveit_common/share/moveit_common/cmake/moveit_package.cmake:36 (find_package)
  CMakeLists.txt:6 (moveit_package)
This warning is for project developers.  Use -Wno-dev to suppress it.
v4hn commented 2 years ago

It seems the usual way to include sub-dependencies in a config (and avoid this warning) is find_dependency. But I did not dig deep enough to propose a patch.