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

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
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants