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 PSP stubs from MD #77

Open
3 tasks done
jphickey opened this issue Aug 7, 2024 · 0 comments · May be fixed by #78
Open
3 tasks done

Remove PSP stubs from MD #77

jphickey opened this issue Aug 7, 2024 · 0 comments · May be fixed by #78
Assignees
@jphickey

Comments

@jphickey
Copy link
Contributor

jphickey commented Aug 7, 2024

Checklist (Please check before submitting)

  • I reviewed the Contributing Guide.
  • I reviewed the README file to see if the feature is in the major future work.
  • I performed a cursory search to see if the feature request is relevant, not redundant, nor in conflict with other tickets.

Is your feature request related to a problem? Please describe.
The MD coverage test contains stubs for some PSP functions. These stubs will be added to the official PSP in nasa/PSP#435,

Describe the solution you'd like
MD should use the stubs provided by the PSP - the local copy will conflict with them, and should be removed.

Additional context
May need to put this in an OMIT_DEPRECATED block to avoid breaking builds.

Requester Info
Joseph Hickey, Vantage Systems, Inc.
@jphickey jphickey self-assigned this Aug 7, 2024
jphickey added a commit to jphickey/MD that referenced this issue Aug 7, 2024
@jphickey
Fix nasa#77, deprecate PSP stubs from MD coverage test
c41a991 
The PSP now provides the complete set of stubs.
@jphickey jphickey linked a pull request Aug 7, 2024 that will close this issue
Open
2 tasks
jphickey added a commit to jphickey/MD that referenced this issue Aug 15, 2024
@jphickey
Fix nasa#77, deprecate PSP stubs from MD coverage test
f50c45f 
The PSP now provides the complete set of stubs.

IMPORTANT: This needs to be used in conjunction with nasa/PSP#436 to
get the complete set of stubs, otherwise linking will fail.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jphickey jphickey

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix #77, deprecate PSP stubs from MD coverage test jphickey/MD
1 participant
@jphickey