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.

Sign up for GitHub

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

Jump to bottom

Fix issues with building using CMake #7

Open

ndam-hexagon wants to merge 3 commits into NVIDIA-ISAAC-ROS:main from ndam-hexagon:hexagon/PAR-6886/patches-for-upstream

ndam-hexagon commented Jan 27, 2025

This PR addresses some issues we encountered while compiling the repository using CMake:

Fix potential double definition of the yaml-cpp target with more recent versions of the library
Add missing definitions for C++ Python bindings
Fix GTest name clashing (due to how CMake looks up test names for later execution with CTest)

ndam-hexagon added 3 commits

January 27, 2025 17:15


          Fix CMake configure issues

f9aeab1


          Fix test name clashing issues

57d8e29


          Add pybind11 modules to CMake

4f0b63a

ndam-hexagon force-pushed the hexagon/PAR-6886/patches-for-upstream branch from a447086 to 4f0b63a Compare

January 27, 2025 16:15

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet