ardupilot icon indicating copy to clipboard operation
ardupilot copied to clipboard

AP_DDS: Added namespace representation for global constants

Open snktshrma opened this issue 7 months ago • 2 comments

This updates global constant in sync with this PR: https://github.com/ArduPilot/Micro-XRCE-DDS-Gen/pull/5

snktshrma avatar Apr 21 '25 18:04 snktshrma

Without this PR building Ardupilot Gazebo SITL currently fails with:

[‘FS_RADIO’ was not declared in this scope; did you mean ‘FLT_RADIX’](https://discuss.ardupilot.org/t/dds-sitl-fs-radio-was-not-declared-in-this-scope-did-you-mean-flt-radix/134537)

Related discussion: https://discuss.ardupilot.org/t/dds-sitl-fs-radio-was-not-declared-in-this-scope-did-you-mean-flt-radix/134537/3

TannerGilbert avatar May 28 '25 22:05 TannerGilbert

Without this PR building Ardupilot Gazebo SITL currently fails with:

[‘FS_RADIO’ was not declared in this scope; did you mean ‘FLT_RADIX’](https://discuss.ardupilot.org/t/dds-sitl-fs-radio-was-not-declared-in-this-scope-did-you-mean-flt-radix/134537)

Related discussion: https://discuss.ardupilot.org/t/dds-sitl-fs-radio-was-not-declared-in-this-scope-did-you-mean-flt-radix/134537/3

Can confirm this PR works.

zhouzhiwen2000 avatar Jun 06 '25 07:06 zhouzhiwen2000

.... rebased it in the hopes that CI passes

peterbarker avatar Jun 10 '25 13:06 peterbarker

Dies due to the tool being old, I think?

peterbarker avatar Jun 10 '25 22:06 peterbarker

Dies due to the tool being old, I think?

Yes seems like still the CI autotests not using latest Micro-XRCE-DDS-Gen

snktshrma avatar Jun 11 '25 06:06 snktshrma

I've pushed up another patch here to install the generator from source, which should get CI to pass.

peterbarker avatar Jun 12 '25 04:06 peterbarker