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

Update to use OSAL socket API #23

Closed
skliper opened this issue Nov 4, 2019 · 2 comments · Fixed by #35 or #38
Closed

Update to use OSAL socket API #23

skliper opened this issue Nov 4, 2019 · 2 comments · Fixed by #35 or #38
Assignees
@jphickey
Labels
enhancement New feature or request
Milestone
Bootes

Comments

@skliper
Copy link
Contributor

skliper commented Nov 4, 2019

Is your feature request related to a problem? Please describe.
Currently uses cFE supplied network_includes.h and not the abstracted OSAL calls

Describe the solution you'd like
Update to use OSAL

Describe alternatives you've considered
None

Additional context
See osapi-os-net.h and/or OSAL API.

Requester Info
Jacob Hageman - NASA/GSFC
@skliper skliper added the enhancement New feature or request label Nov 4, 2019
@skliper skliper mentioned this issue Nov 4, 2019
Closed
@skliper skliper changed the title Update to use OS socket API Update to use OSAL socket API Nov 4, 2019
@jphickey
Copy link
Contributor

jphickey commented Nov 4, 2019

Note: This is already implemented in the EDS branch (techdev-sois-eds on babelfish). In here the CI_LAB and TO_LAB use the socket abstraction via OSAL.

It probably wouldn't be too difficult to separate the changes, and make a non-EDS CI_LAB/TO_LAB, if necessary.

@jphickey jphickey self-assigned this Feb 7, 2020
@jphickey
Copy link
Contributor

jphickey commented Feb 7, 2020

Backporting this change from the EDS branch

jphickey added a commit to jphickey/ci_lab that referenced this issue Feb 7, 2020
@jphickey
Fix nasa#23: Use OSAL Socket API for bind/receive
0115798 
This removes the need to use BSD socket calls directly within CI_LAB
jphickey added a commit to jphickey/ci_lab that referenced this issue Feb 7, 2020
@jphickey
Fix nasa#23: Use OSAL Socket API for bind/receive
3ff5cc1 
This removes the need to use BSD socket calls directly within CI_LAB
@jphickey jphickey mentioned this issue Feb 7, 2020
Merged
jphickey added a commit to jphickey/ci_lab that referenced this issue Feb 8, 2020
@jphickey
Fix nasa#23: Use OSAL Socket API for bind/receive
1f1c178 
This removes the need to use BSD socket calls directly within CI_LAB
@astrogeco astrogeco mentioned this issue Mar 10, 2020
Merged
@astrogeco astrogeco added this to the 2.4.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
enhancement New feature or request
Projects
None yet
Milestone
Bootes
Development

Successfully merging a pull request may close this issue.

Multiple cleanup items for CI LAB jphickey/ci_lab
IC 20200304 nasa/ci_lab
3 participants
@jphickey @skliper @astrogeco