Skip to content

Adds a LINK_DIRS variable to CMakeLists.#44

Open
vik748 wants to merge 2 commits intoYDLIDAR:masterfrom
dapsterai:master
Open

Adds a LINK_DIRS variable to CMakeLists.#44
vik748 wants to merge 2 commits intoYDLIDAR:masterfrom
dapsterai:master

Conversation

@vik748
Copy link

@vik748 vik748 commented Jan 25, 2024

Fixes issue linking against this SDK library when installed locally using the --install-prefix flag. The root cause is that YDLIDAR_SDK_LIBRARY_DIRS is set to empty. LINK_DIRS (PACKAGE_LINK_DIRS) variable is used to set the YDLIDAR_SDK_LIBRARY_DIRS in the resulting ydlidar_sdkConfig.cmake file.

See discussion at colcon/colcon-core#605 for details.

Fixes issue linking against this SDK library when installed locally
using the `--install-prefix` flag. The root cause is that
YDLIDAR_SDK_LIBRARY_DIRS is set to empty. LINK_DIRS (PACKAGE_LINK_DIRS)
variable is used to set the YDLIDAR_SDK_LIBRARY_DIRS in the resulting
ydlidar_sdkConfig.cmake file.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant