The sdo_serializer.hpp library on topic-#57-sdo-lib-c++ is defined as an INTERFACE target inside rise_motion, but the package neither installs the include/ directory nor exports the target via ament_export_targets / ament_export_include_directories. As a result, external packages that depend on rise_motion (e.g. control doing one-off sdo_read calls to fetch typed values such as Position Actual Value at object 0x6064) cannot include rise_motion/sdo_serializer.hpp.
Could you add the install and export bits to the CMakeLists once the library is merged to main? That way downstream packages can reuse the serializer instead of carrying their own little-endian byte conversions. Happy to open a small PR for it from our side if that helps.
The
sdo_serializer.hpplibrary ontopic-#57-sdo-lib-c++is defined as anINTERFACEtarget insiderise_motion, but the package neither installs theinclude/directory nor exports the target viaament_export_targets/ament_export_include_directories. As a result, external packages that depend onrise_motion(e.g.controldoing one-offsdo_readcalls to fetch typed values such as Position Actual Value at object 0x6064) cannot includerise_motion/sdo_serializer.hpp.Could you add the install and export bits to the CMakeLists once the library is merged to
main? That way downstream packages can reuse the serializer instead of carrying their own little-endian byte conversions. Happy to open a small PR for it from our side if that helps.