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 #161, remove OS_VolumeTable #163

Merged
merged 2 commits into from
May 27, 2020

Conversation

jphickey
Copy link
Contributor

@jphickey jphickey commented May 15, 2020

Describe the contribution
Removes all references to the OS_VolumeTable in all PSPs.

Replace with call to OS_FileSysAddFixedMap() for the FS_BASED entries. Other types are already handled at runtime anyway.

Fix #161

Testing performed
Build and sanity check CFE on all supported platforms (vxworks, linux, rtems)

Expected behavior changes
No impact to behavior.

System(s) tested on
Ubuntu 20.04 (native)
RTEMS 4.11 on pc686 / QEMU
VxWorks 6.9 on MCP750

Additional context
Makes code compatible wtih nasa/osal#367 which deprecates the OS_VolumeTable

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

astrogeco and others added 2 commits May 13, 2020 11:13
Integration Candidate COMBINED 2020-04-29 and 2020-05-06
Replace with call to OS_FileSysAddFixedMap() for the FS_BASED
entries.  Other types are already handled at runtime anyway.
@astrogeco astrogeco added the CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) label May 19, 2020
@skliper skliper added CCB-20200520 enhancement New feature or request and removed CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) labels May 20, 2020
@astrogeco
Copy link
Contributor

CCB 2020-05-20 APPROVED

@astrogeco astrogeco added CCB:Approved Indicates Approval by CCB IC-20200520 labels May 26, 2020
@astrogeco astrogeco changed the base branch from master to integration-candidate May 26, 2020 14:01
@jphickey jphickey linked an issue May 27, 2020 that may be closed by this pull request
@astrogeco astrogeco merged commit bb9c39c into nasa:integration-candidate May 27, 2020
@skliper skliper added this to the 1.5.0 milestone Jun 1, 2020
@jphickey jphickey deleted the fix-161-remove-voltabs branch June 16, 2020 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CCB:Approved Indicates Approval by CCB enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Deprecate definition/use of OS_VolumeTable
3 participants