Skip to content

Fix issues with building using CMake#7

Open
ndam-hexagon wants to merge 13 commits intoNVIDIA-ISAAC-ROS:mainfrom
ndam-hexagon:hexagon/PAR-6886/patches-for-upstream
Open

Fix issues with building using CMake#7
ndam-hexagon wants to merge 13 commits intoNVIDIA-ISAAC-ROS:mainfrom
ndam-hexagon:hexagon/PAR-6886/patches-for-upstream

Conversation

@ndam-hexagon
Copy link

@ndam-hexagon 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)
  • Build Pybind11 extensions without the SOABI in the filename like what Bazel seems to do
  • Add compatibility with more recent versions of CMake (set CMake polity CMP0167 = OLD; FindBoost removed)
  • Add missing headers to installation
  • Properly install core GXF extensions like core, std, cuda, multimedia

@ndam-hexagon 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant