[mavlink_messages] Missmatch between header macros and class used

[loicdubois]

Solved Problem

Trying to get the SYSTEM_TIME mavlink stream to work

WARN  [mavlink] stream SYSTEM_TIME not found
ERROR [mavlink] configure_streams_to_default() failed

Fixes #{Github issue ID}

Solution

Simply add the stream corresponding to the right header preprocessor directives

Changelog Entry

Bugfix: SYSTEM_TIME stream not found

Alternatives

Test coverage

• add stream to desired mode
• SITL starts without complaining

Context

[junwoo091400]

What is the use case for this? Could you give some context?

[junwoo091400]

Sorry, accidentally closed it!

[dagar]

Good catch @loicdubois, perhaps we should just generate this instead.

[dagar]

Fixing SYS_STATUS is correct, but SYSTEM_TIME doesn't need to be added.

[loicdubois]

What is the use case for this? Could you give some context?

I actually don't use it, just got a warning by playing around with streams to add and remove. I actually had it commented out at line 379 and was checking which streams were added from the preprocessor directives, not the actual stream list append.