Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix #126, simplify build to use wrappers and interface libs #127

Merged
merged 1 commit into from
Mar 5, 2021

Conversation

jphickey
Copy link
Contributor

@jphickey jphickey commented Mar 4, 2021

Describe the contribution
Use the wrapper functions now provided by CFE to simplify the build recipe and work with interface libraries

Fixes #126

Testing performed
Build and run CFE unit tests

Expected behavior changes
Unit test targets are named a little differently when using the wrapper functions, but transparent if running via ctest. The filename is different if running manually though.

System(s) tested on
Ubuntu 20.04

Additional context
Requires nasa/cFE#1207 and must be merged before nasa/cFE#1203 to avoid breaking UT

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

Use the wrapper functions now provided by CFE to simplify the
build recipe and work with interface libraries
@jphickey jphickey added CCB:FastTrack CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) labels Mar 4, 2021
@skliper skliper added this to the 1.3.0 milestone Mar 5, 2021
@skliper skliper self-assigned this Mar 5, 2021
@skliper skliper merged commit b548714 into nasa:main Mar 5, 2021
@jphickey jphickey deleted the fix-126-use-intf-libs branch March 29, 2021 19:20
@astrogeco astrogeco removed the CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) label May 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update build to work with interface libraries
3 participants