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.

Sign up for GitHub

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

Jump to bottom

Remove deprecated symbol reference #54

Closed
jphickey opened this issue Aug 18, 2020 · 0 comments · Fixed by #55 or #58
Closed

Remove deprecated symbol reference #54

jphickey opened this issue Aug 18, 2020 · 0 comments · Fixed by #55 or #58
Assignees
@jphickey
Milestone
Caelum

Comments

@jphickey
Copy link
Contributor

jphickey commented Aug 18, 2020 •
edited
Loading

Is your feature request related to a problem? Please describe.
The TO_LAB app references the CFE_ES_SHELL_TLM which is deprecated and scheduled to be removed in the next release

Describe the solution you'd like
Remove this code

Additional context
This is currently within an OMIT_DEPRECATED conditional block - but in general modules shouldn't refer to deprecated items in other modules at all.

This is a prerequisite to nasa/cFE#815

Requester Info
Joseph Hickey, Vantage Systems, Inc.
@jphickey jphickey self-assigned this Aug 18, 2020
jphickey added a commit to jphickey/to_lab that referenced this issue Aug 18, 2020
@jphickey
Fix nasa#54, remove reference to CFE_ES_SHELL_TLM
801e8e5 
This message has been deprecated in cFE.
@jphickey jphickey mentioned this issue Aug 18, 2020
Merged
yammajamma added a commit that referenced this issue Aug 19, 2020
@yammajamma
Merge pull request #55 from jphickey/fix-54-remove-deprecated
439d167 
Fix #54, remove reference to CFE_ES_SHELL_TLM
@yammajamma yammajamma mentioned this issue Aug 24, 2020
Merged
@astrogeco astrogeco added this to the 2.5.0 milestone Oct 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jphickey jphickey

Labels
None yet
Projects
None yet
Milestone
Caelum
2 participants
@jphickey @astrogeco