fix ament_target_dependencies() deprecation

[mosfet80]

Description

fix ament_target_dependencies deprecation

Checklist

• Required by CI: Code is auto formatted using clang-format
• While waiting for someone to review your request, please consider reviewing another open pull request to support the maintainers

[AndyZe]

Please make these small changes then I'll merge it. :D

[AndyZe]

So I see the build fails because PickNik's reset_fault_controller package isn't available on Rolling. That's not related to this PR. What do you think, @mosfet80 ? Merge now regardless? Maybe find whatever tutorial depends on reset_fault_controller and skip building it, for now?

https://index.ros.org/?search_packages=true#rolling

[mosfet80]

So I see the build fails because PickNik's reset_fault_controller package isn't available on Rolling. That's not related to this PR. What do you think, @mosfet80 ? Merge now regardless? Maybe find whatever tutorial depends on reset_fault_controller and skip building it, for now?

https://index.ros.org/?search_packages=true#rolling

Confirm, the build failure issue is external to the repository.
Some more modern systems fail to build due to the deprecation of ament_target_dependencies. My opinion is to merge now.

[AndyZe]

I can't merge it. Don't have the permissions. I think if you revert #1042 it may start working again. picknik_controllers is required by ros2_kortex and robotiq, I think. (Just mentioning this because it took me awhile to track down.)

https://github.com/Kinovarobotics/ros2_kortex/blob/main/ros2_kortex-not-released.rolling.repos