SICKAG / sick_safetyscanners

ROS driver for SICK safety laser scanners
https://www.sick.com/de/en/opto-electronic-protective-devices/safety-laser-scanners/c/g187225
Apache License 2.0
61 stars 59 forks source link

ReadWriteHelper.hpp is not going to be installed #76

Closed Crcodlus closed 3 years ago

Crcodlus commented 3 years ago

Hello,

Due to FILES_MATCHING PATTERN "*.h" at CMakeLists.txt

...
# Mark cpp header files for installation
install(
  DIRECTORY include/${PROJECT_NAME}/
  DESTINATION ${CATKIN_PACKAGE_INCLUDE_DESTINATION}
  FILES_MATCHING PATTERN "*.h"
)
...

ReadWriteHelper.hpp is not going to be installed.

This can be fixed by using

...
# Mark cpp header files for installation
install(
  DIRECTORY include/${PROJECT_NAME}/
  DESTINATION ${CATKIN_PACKAGE_INCLUDE_DESTINATION}
  FILES_MATCHING
    PATTERN "*.h"
    PATTERN "*.hpp"
)
...
lenpuc commented 3 years ago

Oh, thanks for spotting this!

Fixed it in the latest commit.