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.

Sign up for GitHub

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

Jump to bottom

Remove OSAL deprecated code for next release #561

Closed
jphickey opened this issue Aug 18, 2020 · 0 comments · Fixed by #562 or #582
Closed

Remove OSAL deprecated code for next release #561

jphickey opened this issue Aug 18, 2020 · 0 comments · Fixed by #562 or #582
Assignees
@jphickey
Labels
enhancement
Milestone
Caelum

Comments

@jphickey
Copy link
Contributor

Is your feature request related to a problem? Please describe.
Now that the "5.1.x" release branch has been created, the next release should remove all code/functions that are currently marked as deprecated.

Describe the solution you'd like
Removal of all code currently contained within an OMIT_DEPRECATED conditional compile switch.

Additional context
The removal only applies to "main" branch - not the 5.1.x release branch, where it will continue to exist for that release series. This assumes the next release (main) will become version 6.0.

There are other changes currently in development which do break some deprecated items - hence why I'd like to remove this sooner rather than later, as it saves the work of updating/fixing this old code if the intent is to remove it anyway.

Requester Info
Joseph Hickey, Vantage Systems, Inc.
jphickey added a commit to jphickey/osal that referenced this issue Aug 18, 2020
@jphickey
Fix nasa#561, remove all deprecated code
1f0903d 
In the next major OSAL release, this code will be no longer
supported at all.  It should be removed early in the cycle
to avoid needing to maintain this compatibility code.
@jphickey jphickey mentioned this issue Aug 18, 2020
Merged
yammajamma added a commit that referenced this issue Aug 27, 2020
@yammajamma
Merge pull request #562 from jphickey/fix-561-remove-deprecated
881aa63 
Fix #561, remove all deprecated code
@yammajamma yammajamma mentioned this issue Aug 27, 2020
Merged
@skliper skliper added this to the 6.0.0 milestone Sep 1, 2020
@jphickey jphickey changed the title Remove OSAL deprected code for next release Remove OSAL deprecated code for next release Sep 1, 2020
@skliper skliper mentioned this issue Sep 21, 2020
Closed
jphickey pushed a commit to jphickey/osal that referenced this issue Aug 10, 2022
@skliper
Fix nasa#561, Add branch coverage reporting
c6308dc
jphickey pushed a commit to jphickey/osal that referenced this issue Aug 10, 2022
@astrogeco
Merge pull request nasa#562 from skliper/fix561-branch-cov-reporting
13213fd 
Fix nasa#561, Add branch coverage reporting
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jphickey jphickey

Labels
enhancement
Projects
None yet
Milestone
Caelum
Development

Successfully merging a pull request may close this issue.

Fix #561, remove all deprecated code jphickey/osal
Integration Candidate: 2020-08-26 nasa/osal
2 participants
@jphickey @skliper