search

nasa / ECI

CFS application framework for integrating external C code into the CFS environment
https://nasa.github.io/ECI/
Other
35 stars 12 forks source link

ECI is not compatible with cFS version bootes-rc2 (cFS 6.8-RC2) #52

Open LornDMiller opened 3 years ago

LornDMiller commented 3 years ago

cFS tag bootes-rc2 (cFS version 6.8, release candidate 2?), or earlier, removes the boolean type, renames Event Type level constants, and removes CCSDS parsing utilities. The current version of the ECI needs to be updated to be compatible with these changes.

The immediate files that I have identified as requiring changes are: fsw/src/eci_app.c fsw/src/eci_app.h faultrep/app_faultrep.c faultrep/app_faultrep_priv.h

I have not evaluated the unit test files to determine if they are also affected.

oliverhamburger commented 3 years ago

Ill take this issue

nasabenz commented 3 years ago

I was happy to see this request already logged and a PR is in review. Just curious on how this ECI repository will manage compatibility among the different cFS releases? Will there be a tag for Bootes and another for Caelum?

oliverhamburger commented 3 years ago

I was happy to see this request already logged and a PR is in review. Just curious on how this ECI repository will manage compatibility among the different cFS releases? Will there be a tag for Bootes and another for Caelum?

I believe so, Steve and I discussed merging the bootes updates, tagging that, and then merging the caelum update. Steve is out of town for the month of August so not sure when the updates will get merged.

SpaceSteve121 commented 3 years ago

My plan is that the ECI will always support the current version of CFS.

To get to that point, we'll update ECI to work with Bootes, tag it, then proceed with ongoing development/maintenance (which will include updating to Caelum as well as new features/bugfixes). Going forward, I'm not intending to backport new features/bug fixes to the Bootes tag... We don't have the manpower to maintain multiple versions.