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

Integration Candidate 2020-04-01 #64

Merged
merged 3 commits into from
Apr 9, 2020
Merged

Conversation

astrogeco
Copy link
Contributor

@astrogeco astrogeco commented Apr 6, 2020

Describe the contribution
Integration Candidate
Integrates:

  1. cFE 6.7.12 - Integration Candidate - 2020-04-01 cFE#586
  2. osal 5.0.11 - Integration Candidate 2020-04-01 osal#408
  3. psp 1.4.8 - Integration Candidate 2020-04-01 PSP#151
  4. cFS-GroundSystem 2.1.6 - Integration Candidate 2020-04-01 cFS-GroundSystem#69

Testing performed
Bundle CI - https://travis-ci.com/github/nasa/cFS/builds/159710339

Expected behavior changes
See PRs

System(s) tested on
Bundle CI - Ubuntu:Bionic

Contributor Info - All information REQUIRED for consideration of pull request
Gerardo E. Cruz-Ortiz, NASA-GSFC

@astrogeco astrogeco changed the title Integration Candidate 2020-04-01 WIP: Integration Candidate 2020-04-01 Apr 6, 2020
@astrogeco
Copy link
Contributor Author

astrogeco commented Apr 7, 2020

@skliper I think the deprecated OS_IntLock and OS_IntUnlock are causing trouble in cfe_es_perf.c 's CFE_ES_PerfLogAdd:437

@skliper
Copy link
Contributor

skliper commented Apr 7, 2020

See the comment on nasa/osal#396

This won't compile with deprecated removed until nasa/cFE#456 gets resolved

@astrogeco
Copy link
Contributor Author

@skliper I made two commits in my cFE fork that I think fix this but I worry that it might break something else:

astrogeco/cFE@8da0e47
astrogeco/cFE@bb2a401

@jphickey
Copy link
Contributor

jphickey commented Apr 9, 2020

Note that the pull request for nasa/cFE#456 is now submitted in nasa/cFE#595, and this will cause a merge conflict with the #ifdef blocks where currently it is a clean merge.

Should be an easy rebase though, so if you do put this in let me know so I can rebase the pull request.

@astrogeco
Copy link
Contributor Author

Note that the pull request for nasa/cFE#456 is now submitted in nasa/cFE#595, and this will cause a merge conflict with the #ifdef blocks where currently it is a clean merge.

Should be an easy rebase though, so if you do put this in let me know so I can rebase the pull request.

I think a better option is to fast-track nasa/cFE#595 and merge that instead @skliper thoughts?

@skliper
Copy link
Contributor

skliper commented Apr 9, 2020

I'm all for moving nasa/cFE#595 forward. I already approved.

@jphickey
Copy link
Contributor

jphickey commented Apr 9, 2020

I've got another pull request that I'm just about to submit for 295/480 which depends on 595, so if you decide to include nasa/cFE#595, then let me know as soon as the integration candidate is ready so I can rebase to it.

cFE - 6.7.12
osal - 5.0.11
psp - 1.4.8
cFS-GroundSystem - 2.1.6
@astrogeco astrogeco changed the title WIP: Integration Candidate 2020-04-01 Integration Candidate 2020-04-01 Apr 9, 2020
@astrogeco astrogeco merged commit 788b8d6 into master Apr 9, 2020
@astrogeco
Copy link
Contributor Author

I've got another pull request that I'm just about to submit for 295/480 which depends on 595, so if you decide to include nasa/cFE#595, then let me know as soon as the integration candidate is ready so I can rebase to it.

@jphickey, I just merged the IC.

@skliper skliper added this to the 6.8.0 milestone Jun 1, 2020
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.

3 participants