Move all arguments out of macro file

[fmauch]

This way it can be included more easily.

@p0rys this should fix the problems mentioned in #482.

[gavanderhoorn]

Would be even better if there would be a way to keep the ros2_control block out of the default macro completely.

It's really rather awkward having that present in all cases. The changes in this PR seem to also highlight that.

[codecov]

Codecov Report

❗ No coverage uploaded for pull request base (foxy@e448b15). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files

@@          Coverage Diff           @@
##             foxy    #495   +/-   ##
======================================
  Coverage        ?   3.73%           
======================================
  Files           ?      16           
  Lines           ?    1071           
  Branches        ?     212           
======================================
  Hits            ?      40           
  Misses          ?     942           
  Partials        ?      89           

Flag	Coverage Δ
unittests	3.73% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[fmauch]

Would be even better if there would be a way to keep the ros2_control block out of the default macro completely.

It's really rather awkward having that present in all cases. The changes in this PR seem to also highlight that.

I do agree on the awkward part, but so far I couldn't come up with a satisfying solution. I've started #496 to share my concerns. Maybe we can discuss things there a bit to migrate to a proper solution?

[fmauch]

@RobertWilbrandt a review on this would be appreciated before we put Foxy to its EOL sleep.

[fmauch]

Closing this, since Foxy is no longer updated.