Skip to content

Fix #976, short background task name#977

Merged
astrogeco merged 1 commit intonasa:integration-candidatefrom
jphickey:fix-976-bgtask-name
Oct 30, 2020
Merged

Fix #976, short background task name#977
astrogeco merged 1 commit intonasa:integration-candidatefrom
jphickey:fix-976-bgtask-name

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