nasa / LC

The Core Flight System (cFS) Limit Checker (LC) application.
Apache License 2.0
30 stars 21 forks source link

Fix #51, externalize symbols in lc_test_utils.h #52

Closed jphickey closed 1 year ago

jphickey commented 1 year ago

Checklist (Please check before submitting)

Describe the contribution Make the table objects in this header "extern" and instantiate them in the C file instead. This solves the duplicate symbol linker errors.

Fixes #51

Testing performed Build and run all tests

Expected behavior changes Tests will build, link, and execute successfully

System(s) tested on Ubuntu 22.04

Additional context Will submit a separate issue to consider adding -fno-common to the CFS CI workflows, as it masks this issue in Ubuntu 20.04.

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