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 #34, Build fails with deprecated cFE/OSAL elements removed #29

Merged
merged 1 commit into from
Feb 25, 2020

Conversation

dmknutsen
Copy link
Contributor

Describe the contribution
Fixes #35, Build fails with deprecated cFE/OSAL elements removed

Testing performed
Nominal build process:

make distclean
make prep
make
make install
followed by building with OMIT_DEPRECATED = true
make distclean
make OMIT_DEPRECATED=true prep
make
make install

Confirmed clean build in both cases.

Expected behavior changes
Clean build when OMIT_DEPRECATED = true

System(s) tested on
Oracle VM VirtualBox
OS: ubuntu-18.04.3
Versions: cFE 6.7.5.0, OSAL 5.0.5.0, PSP 1.4.3.0

Additional context
There is a total of three pull requests to address issue_35:
nasa/to_lab
nasa/sample_app
nasa/sch_lab

All three are intended to be delivered together.

Contributor Info
Dan Knutsen
GSFC/NASA

@astrogeco astrogeco changed the title Fixes #35, Build fails with deprecated cFE/OSAL elements removed Fixes sample_app#35, Build fails with deprecated cFE/OSAL elements removed Feb 11, 2020
@astrogeco astrogeco changed the title Fixes sample_app#35, Build fails with deprecated cFE/OSAL elements removed Fix sample_app#35, Build fails with deprecated cFE/OSAL elements removed Feb 11, 2020
@astrogeco astrogeco added the CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) label Feb 11, 2020
@astrogeco astrogeco added this to the 2.4.0 milestone Feb 11, 2020
@astrogeco astrogeco self-requested a review February 12, 2020 04:35
@astrogeco astrogeco added CCB:Ignore Pull Request has work pending and removed CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) labels Feb 12, 2020
Copy link
Contributor

@astrogeco astrogeco left a comment

Choose a reason for hiding this comment

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

Please create an issue in sch_lab that mirrors nasa/sample_app#35 and link it to this PR

@astrogeco astrogeco added CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) and removed CCB:Ignore Pull Request has work pending CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) labels Feb 12, 2020
@dmknutsen dmknutsen linked an issue Feb 12, 2020 that may be closed by this pull request
@astrogeco astrogeco changed the title Fix sample_app#35, Build fails with deprecated cFE/OSAL elements removed Fix #34, Build fails with deprecated cFE/OSAL elements removed Feb 12, 2020
@astrogeco
Copy link
Contributor

CCB 20200212 - Approved

1 similar comment
@astrogeco
Copy link
Contributor

CCB 20200212 - Approved

@astrogeco astrogeco added code reviewed and removed CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) labels Feb 13, 2020
@astrogeco astrogeco changed the base branch from master to ic-20200226 February 25, 2020 19:26
@astrogeco astrogeco merged commit ca19528 into nasa:ic-20200226 Feb 25, 2020
@astrogeco astrogeco mentioned this pull request Feb 26, 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.

Build fails with deprecated cFE/OSAL elements removed #35
3 participants