Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Document ccd dependency #12

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Ryanf55
Copy link
Contributor

@Ryanf55 Ryanf55 commented Sep 6, 2024

ccd was not installed when I built ompl,but it's required for the app.

-- Checking for module 'ccd>=2.0'
--   No package 'ccd' found
CMake Error at /usr/share/cmake-3.30/Modules/FindPackageHandleStandardArgs.cmake:233 (message):
  Could NOT find ccd (missing: CCD_LIBRARIES CCD_INCLUDE_DIRS)
Call Stack (most recent call first):
  /usr/share/cmake-3.30/Modules/FindPackageHandleStandardArgs.cmake:603 (_FPHSA_FAILURE_MESSAGE)
  CMakeModules/Findccd.cmake:9 (find_package_handle_standard_args)
  CMakeLists.txt:131 (find_package)

Maybe we only need version 2.0?

Signed-off-by: Ryan Friedman <[email protected]>
@mamoll
Copy link
Member

mamoll commented Sep 6, 2024

libccd is a dependency for FCL. We don't use libccd directly in omplapp. The top-level CMakeLists.txt should probably be updated to check for FCL before checking for libccd. FCL didn't used to have cmake config files, so we needed the libccd check to collect the right compile/link flags. We can actually probably remove the libccd references in the CMakeLists.txt altogether.

@mamoll
Copy link
Member

mamoll commented Sep 8, 2024

PR my my alternative approach is here: #13

This should also fix the issue you are seeing. Please confirm.

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.

2 participants