eProsima / Fast-DDS-statistics-backend

eProsima Fast DDS Statistics Backend is a C++ library that provides collection and procession the statistics measurements reported by Fast DDS Statistics Module. Looking for commercial support? Contact info@eprosima.com
https://eprosima.com
Apache License 2.0
17 stars 9 forks source link

Build Failed with incompatible Generated Types #216

Closed deshpandeyash closed 10 months ago

deshpandeyash commented 10 months ago

I am compiling with from source with Cmake and I get this result when building:

In file included from /home/lkn/Fast-DDS/Fast-DDS-statistics-backend/src/cpp/StatisticsBackend.cpp:52: /home/lkn/Fast-DDS/Fast-DDS-statistics-backend/src/cpp/topic_types/typesPubSubTypes.h:36:2: error: #error Generated types is not compatible with current installed Fast DDS. Please, regenerate it with fastddsgen.

Followed by a bunch of Type errors. I am unsure how to proceed here as I also cannot find a tutorial on what to do.

Any help would be great.

JesusPoderoso commented 10 months ago

Hi @deshpandeyash , thanks for using Fast DDS. I would recommend you to use latest stable versions of Fast DDS and Fast DDS Statistics Backend. New versions of both of them are going to be released really soon, but if you are using master branch in both of them, make sure that you have the changes introduced in this PR, because it solves those types issues:

In any case, the way to solve those issues is by re-generating the types using Fast DDS Gen tool. ~ cd <folder where types are: src/cpp/topic_types/> ~ fastddsgen -replace types.idl

Edit: make sure that the Fast DDS Gen version is updated and compatible with your current Fast DDS and Fast DDS Statistics Backend versions

deshpandeyash commented 10 months ago

Thanks a lot. Our setup was set to a 6 month old version of FastDDS and changing it was too risky. Therefore, i pulled from v.0.11.0 and then it worked.

Just a heads up that the topic_types directory is missing in the main now. I am new to this and don't know if its needed.

JesusPoderoso commented 10 months ago

Hi @deshpandeyash, thanks for your concern. The types have been moved from src/cpp/topic_types to include/fastdds_statistics_backend/topic_types to expose them as public API. That was needed for the next release, which would be ready in a couple of days.