Skip to content

issue #307, Remove unsupported BSPs#311

Merged
skliper merged 1 commit intonasa:masterfrom
avan989:cppcheck_issue_307_remove_BSP
Dec 30, 2019
Merged

issue #307, Remove unsupported BSPs#311
skliper merged 1 commit intonasa:masterfrom
avan989:cppcheck_issue_307_remove_BSP

Conversation

@avan989
Copy link
Contributor

@avan989 avan989 commented Dec 12, 2019

Describe the contribution
Remove unsupported BSP and change genppc-vxworks6.4→ mcp750-vxworks

System(s) tested on:

  • Hardware
  • Ubuntu 18.04

Contributor Info
Anh Van, NASA Goddard

@skliper skliper added the CCB:Approved Indicates code review and approval by community CCB label Dec 18, 2019
@skliper skliper added this to the 5.1.0 milestone Dec 18, 2019
@skliper
Copy link
Contributor

skliper commented Dec 18, 2019

CCB 20191218 - Reviewed and approved for IC

@jphickey
Copy link
Contributor

NOTE: the old name of the BSP is referenced in the CFE example toolchains here:

https://github.com/nasa/cFE/blob/2b27dfc8f2a4e0627e6c4b646765269213584cda/cmake/sample_defs/toolchain-ppc-vxworks6.9.cmake#L49

This external reference also needs to be updated accordingly.

Copy link
Contributor

@skliper skliper left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Address Joe's comment (update example toolchain file) as part of this update.

@avan989
Copy link
Contributor Author

avan989 commented Dec 23, 2019

updated. Reference nasa/cFE#444

@skliper skliper merged commit 1ce7086 into nasa:master Dec 30, 2019
@skliper
Copy link
Contributor

skliper commented Dec 30, 2019

Merged to master prematurely, reverted and merged to ic branch.

skliper pushed a commit that referenced this pull request Dec 30, 2019
Fix #295, #298, #305, #307, #308,
    #313, #314, #316, #321, #323
Reviewed and approved at 2019-12-18 CCB
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CCB:Approved Indicates code review and approval by community CCB

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants