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 #976, short background task name #977

Merged
merged 1 commit into from
Oct 30, 2020

Conversation

jphickey
Copy link
Contributor

Describe the contribution
Keep names under 16 chars to make more debugger friendly, regardless
of the OSAL limit.

Fixes #976

Testing performed
Run CFE in debugger

Expected behavior changes
Confirm task name shows up as "ES_BG_TASK"
No other impact to behavior

System(s) tested on
Ubuntu 20.04

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

Keep names under 16 chars to make more debugger friendly, regardless
of the OSAL limit.
@jphickey jphickey added the CCB:Ready Ready for discussion at the Configuration Control Board (CCB) label Oct 28, 2020
@jphickey
Copy link
Contributor Author

Low priority item, marked as CCB ready but no big deal if we don't have time to review this cycle.

@astrogeco
Copy link
Contributor

Low priority item, marked as CCB ready but no big deal if we don't have time to review this cycle.

let's discuss this next week since we have a pretty hefty agenda for today

@jphickey jphickey self-assigned this Oct 28, 2020
@astrogeco
Copy link
Contributor

CCB 2020-10-28 APPROVED

@astrogeco astrogeco added CCB-20201028 and removed CCB:Ready Ready for discussion at the Configuration Control Board (CCB) labels Oct 28, 2020
@astrogeco astrogeco changed the base branch from main to integration-candidate October 30, 2020 15:49
@astrogeco astrogeco merged commit 2e8b9b6 into nasa:integration-candidate Oct 30, 2020
astrogeco added a commit to nasa/cFS that referenced this pull request Oct 30, 2020
@jphickey jphickey deleted the fix-976-bgtask-name branch December 3, 2020 17:53
@skliper skliper added this to the 7.0.0 milestone Sep 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Shorter name for ES_BackgroundTask
3 participants