GZ_LAUNCH_VERSION_NAMESPACE not defined?

[traversaro]

Environment

• OS Version: Not relevant
• Source or binary build? Not relevant

Description

It seems that this library uses a GZ_LAUNCH_VERSION_NAMESPACE macro, but this macro is not defined in config.hh.

• Expected behavior: The macro is defined.
• Actual behavior: The macro is not defined, so GZ_LAUNCH_VERSION_NAMESPACE is inserted instead of the versioned namespace.

Steps to reproduce

Inspect the code and see that GZ_LAUNCH_VERSION_NAMESPACE is not defined.

1. 2. 3.

Output

At this point, we can't define GZ_LAUNCH_VERSION_NAMESPACE for gz-launch 6, otherwise we would create an ABI break. However we should remember to define it for gz-launch 7.

[azeey]

Good catch! We could add this to the main branch now.