Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

CFE_ES_RestartApp() WriteToSysLog Typo #309

Closed
skliper opened this issue Sep 30, 2019 · 2 comments · Fixed by #699 or #712
Closed

CFE_ES_RestartApp() WriteToSysLog Typo #309

skliper opened this issue Sep 30, 2019 · 2 comments · Fixed by #699 or #712
Labels
bug good first issue Good for newcomers
Milestone
Bootes

Comments

@skliper
Copy link
Contributor

skliper commented Sep 30, 2019

The first WriteToSysLog() occurrence indicates the message is from CFE_ES_DeleteApp instead of CFE_ES_RestartApp:

CFE_ES_WriteToSysLog ("CFE_ES_DeleteApp: Cannot Restart a CORE Application: %s.\n",
@skliper
Copy link
Contributor Author

skliper commented Sep 30, 2019

Imported from trac issue 278. Created by dcmccom3 on 2019-04-15T07:13:22, last modified: 2019-05-23T16:43:54

@skliper skliper self-assigned this Sep 30, 2019
@skliper skliper added the bug label Sep 30, 2019
@skliper
Copy link
Contributor Author

skliper commented Sep 30, 2019

Trac comment by dcmccom3 on 2019-04-15 07:16:00:

Found the same typo in CFE_ES_ReloadApp()

The file is cfe_es_api.c

@skliper skliper removed their assignment Sep 30, 2019
@skliper skliper added the good first issue Good for newcomers label Nov 4, 2019
skliper added a commit to skliper/cFE that referenced this issue Feb 11, 2020
@skliper
Fix nasa#309, ES Requirements Scrub
3aa6bf9
@skliper skliper added this to the 6.8.0 milestone May 12, 2020
skliper added a commit to skliper/cFE that referenced this issue May 12, 2020
@skliper
Fix nasa#309, Typo in CFE_ES_RestartApp WriteToSysLog
3149211
skliper added a commit to skliper/cFE that referenced this issue May 12, 2020
@skliper
Fix nasa#309, Typo in CFE_ES_RestartApp WriteToSysLog
8042159
@skliper skliper mentioned this issue May 12, 2020
Merged
astrogeco added a commit that referenced this issue May 19, 2020
@astrogeco
Merge pull request #699 from skliper/fix309-syslog-typo
28faa4c 
Fix #309, Typo in CFE_ES_RestartApp WriteToSysLog
@astrogeco astrogeco mentioned this issue May 19, 2020
Merged
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
bug good first issue Good for newcomers
Projects
None yet
Milestone
Bootes
1 participant
@skliper