Fix RessourceManager constructor

[christophfroehlich]

I'm currently working on a CI job with downstream builds for ros2_control, and this package here fails.

The API of the RessourceManager has changed. To be able to compile this repository on jazzy+rolling, one has to pass some more arguments to it. I think it is time to branch-off humble and fix that for the newer distros.

[christophfroehlich]

Well, that is also a possibility. but there is a simpler way: as we use ament_generate_version_header you simply can use

#include<hardware_interface/version.h>
#if HARDWARE_INTERFACE_VERSION_GTE(major, minor, patch) 
#endif

[christophfroehlich]

@JafarAbdi shall I change it? And the CI job has now a wrong (humble) in the title.

[JafarAbdi]

Well, that is also a possibility. but there is a simpler way: as we use ament_generate_version_header you simply can use

#include<hardware_interface/version.h>
#if HARDWARE_INTERFACE_VERSION_GTE(major, minor, patch) 
#endif

I thought about using it, but it was introduced in 4.8.0 😿

[JafarAbdi]

@JafarAbdi shall I change it? And the CI job has now a wrong (humble) in the title.

It should be fixed now

[christophfroehlich]

I thought about using it, but it was introduced in 4.8.0 😿

Oh, we should backport that 🙈

[christophfroehlich]

We have it now in the humble branch as well, maybe you'd consider updating this after the next release of ros2_control.

[JafarAbdi]

We have it now in the humble branch as well, maybe you'd consider updating this after the next release of ros2_control.

Thanks! I'll open a PR to use it