nasa / EdsLib

CCSDS SOIS Electronic Data Sheet Tool and Library
Apache License 2.0
31 stars 12 forks source link

eds2cfetbl tool needs to handle "static" table definitions for backward compatibility #26

Open jphickey opened 1 year ago

jphickey commented 1 year ago

For some reason, many CFS apps instantiated their tables (in C structs) qualified as static. This serves no useful purpose but it breaks the edscfetbl tool which expects to be able to find the symbol by name.

While a workaround is simple (remove static) to get the table to build, this feature of supporting C struct table definitions was intended for backward compatibility, so it would be better if it worked without needing to modify the original table definitions. Having to modify the original table def somewhat defeats the purpose. It seems most, if not all CFS apps have defined the table using static.

Need There may be a way to work around this via the CFE_TBL_FILEDEF macro and possibly tweaking the build rule.

Filing this bug against eds2cfetbl for now, but the fix might involve CFS CMake table build rule changes as well. Need to investigate so this seems like the best place to start.