nasa / FM

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

Fix #73, compression selectable at source file level #81

Closed jphickey closed 1 year ago

jphickey commented 1 year ago

Checklist (Please check before submitting)

Describe the contribution Adds a cmake option for compression sub-component implementation. This replaces the FM_INCLUDE_DECOMPRESS option in fm_platform_cfg.h.

Currently available options are:

This lays the groundwork to support compression in addition to decompression, using an external 3rd part library if desired.

Fixes #73

Testing performed Build and run all tests

Expected behavior changes Compression option no longer in header file - now in a CMake option with source selection. Basically equivalent though.

System(s) tested on Debian

Additional context All conditional compilation (#ifdef) around compression has been removed. This comprised a fair bit of code in FM that was not being tested in any workflow because compression is usually disabled.

Contributor Info - All information REQUIRED for consideration of pull request Joseph Hickey, Vantage Systems, Inc.