nasa / ogma

Generator of runtime monitors for flight and robotics applications.
Other
335 stars 25 forks source link

`ogma-core`: Generated cFS application mentions the sample app #105

Closed ivanperez-keera closed 11 months ago

ivanperez-keera commented 1 year ago

Description

Ogma's cFS template is based on cFS' Sample Application, and the generated code includes multiple references to the sample app.

Keeping those references there is misleading, since this is not the sample app, but a monitoring app.

Type

Additional context

None.

Requester

Method to check presence of bug

Searching for mentions of the sample app in the ogma directory brings up multiple results. It should not bring any.

$ grep -niHre 'sample.\?app' *
ogma-core/src/Command/CFSApp.hs:212:      , "**   This file contains the source code for the Sample App."
ogma-core/templates/copilot-cfs/fsw/mission_inc/copilot_cfs_perfids.h:6:**  Define Sample App Performance IDs
ogma-core/templates/copilot-cfs/fsw/src/copilot_cfs.c:5:**   This file contains the source code for the Sample App.
ogma-core/templates/copilot-cfs/fsw/src/copilot_cfs_version.h:6:**  The Sample Application header file containing version number
ogma-core/templates/copilot-cfs/fsw/platform_inc/copilot_cfs_msgids.h:6:**  Define Sample App  Message IDs
ogma-core/templates/copilot-cfs/CMakeLists.txt:2:project(CFE_SAMPLE_APP C)

Expected result

The result of the grep command above should be empty, indicating that there are no mentions of the sample app.

Desired result

The result of the grep command above should be empty, indicating that there are no mentions of the sample app.

Proposed solution

Replace all mentions of the sample app in the cFS template.

Further notes

None.

ivanperez-keera commented 11 months ago

Change Manager: Confirmed that the issue exists.

ivanperez-keera commented 11 months ago

Technical Lead: Confirmed that the issue should be addressed.

ivanperez-keera commented 11 months ago

Technical Lead: Issue scheduled for fixing in Ogma 1.0.12.

Fix assigned to: @ivanperez-keera.

ivanperez-keera commented 11 months ago

Implementor: Solution implemented, review requested.

ivanperez-keera commented 11 months ago

Change Manager: Verified that:

ivanperez-keera commented 11 months ago

Change Manager: Implementation ready to be merged.