Invalid separator in SetPayload srv

[Briancbn]

I know this will potentially break a lot of things, so maybe this doesn't need to be merged at this moment, but the current field name and separator will not be able to build in ROS2 or a ROS1-ROS2 bridge.
Changes:

• License and DCO
• Change field name to fit ROS2 regex check
• fix invalid separator

[gavanderhoorn]

Thanks for the PR.

Could you please only include 3308c36 in this PR? The others seem unrelated to the problem you report.

[gavanderhoorn]

Did you keep 38d0b36 on purpose?

I understand why you did it, but that change will indeed break all sorts of software, and does not follow the naming of the field in RTDE.

[Briancbn]

I understand why you did it, but that change will indeed break all sorts of software, and does not follow the naming of the field in RTDE.

Noted, I change this PR to only fixing the separator.

[gavanderhoorn]

Thanks for the fix.

That should indeed have been just ---. Interesting we didn't see any warnings from the .msg parser.

[gavanderhoorn]

Oh. I just noticed you opened this against melodic, which is the release branch.

It should've been opened against melodic-devel.

I'll revert melodic and push the commit to melodic-devel.

[gavanderhoorn]

Fixed.

I'll create a release later.