nasa / FM

The Core Flight System (cFS) File Manager (FM) application.
Apache License 2.0
42 stars 24 forks source link

Internal delete command support not in requirements, doesn't match description #41

Closed skliper closed 2 years ago

skliper commented 2 years ago

Checklist (Please check before submitting)

Is your feature request related to a problem? Please describe. FM_DELETE_INT_CC is out of family. FM is operator interface to the file system, not what other elements should use to delete files. Also strange handling in that it updates command counters but doesn't send the event message, so it would break normal/simple command counter confirmation approach for ground commands.

Describe the solution you'd like Deprecate and/or directly remove support of this command. Other apps should use OS_remove directly instead of relying on FM. Goal is to reduce interdependency.

Describe alternatives you've considered None

Additional context Replaces:

Requester Info Jacob Hageman - NASA/GSFC

skliper commented 2 years ago

Switched to bug since it also doesn't match it's description in that it claims it won't increment command counter. There's also no requirement that I could find.

skliper commented 2 years ago

Marking as Draco so we address this one... may need to mark as deprecated or similar. I'd rather not write a requirement for something that I want to remove.

skliper commented 2 years ago

Skimmed design documentation and didn't see any references to this functionality.