Skip to content

Encode help-*.txt file contents into C source code #3577

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

Merged
merged 1 commit into from
Apr 12, 2025

Conversation

rhc54
Copy link
Contributor

@rhc54 rhc54 commented Mar 30, 2025

Encode the show_help text files into memory so that we don't have to find and open text files on-the-fly. This helps with relocation, although not strictly
necessary. Biggest advantage is that we can supply show_help output from the very beginning, even if
we haven't identified the text file directory.

Enable access to the new system by PRRTE via a new API by which PRRTE can register its own in-memory
array of help messages. Default to the internal
PMIx array if show_help has yet to be initialized.

Add a new "INMEMHELP" capability flag so that PRRTE can verify this PMIx version's support. Note that
PRRTE will be updated so that the master branch and all release branches beginning with v4.0 can require it to build.

Based on open-mpi/ompi#13144.

NOTE: THIS PR WILL FAIL ALL PRRTE TESTS UNTIL PRRTE CAN BE UPDATED - CHICKEN/EGG PROBLEM

Encode the show_help text files into memory so that
we don't have to find and open text files on-the-fly.
This helps with relocation, although not strictly
necessary. Biggest advantage is that we can supply
show_help output from the very beginning, even if
we haven't identified the text file directory.

Enable access to the new system by PRRTE via a new
API by which PRRTE can register its own in-memory
array of help messages. Default to the internal
PMIx array if show_help has yet to be initialized.

Add a new "INMEMHELP" capability flag so that PRRTE
can verify this PMIx version's support. Note that
PRRTE will be updated so that the master branch and
all release branches beginning with v4.0 can require
it to build.

Based on open-mpi/ompi#13144.

Signed-off-by: Ralph Castain <[email protected]>
@rhc54 rhc54 merged commit bbee0e8 into openpmix:master Apr 12, 2025
15 of 18 checks passed
@rhc54 rhc54 deleted the topic/hdrs branch April 12, 2025 21:17
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.

1 participant