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

prte_setup_pmix.m4: check for specific PMIx capability flags #1854

Merged
merged 2 commits into from
Dec 13, 2023

Conversation

jsquyres
Copy link
Contributor

Paired with openpmix/openpmix#3198

Demo showing how to check for PMIx capability flags. This commit:

  • Will abort configure if:

    • PMIX_CAPABILITIES does not exist (i.e., this PMIx is so old that it does not have any capability flags)
    • PMIX_CAP_202311CLI capability flag goes not exist
    • PMIX_CAP_202311CLI is not set in PMIX_CAPABILITIES
  • Will warn if these capability flags are not set in PMIX_CAPABILITIES (to include PMIX_CAPABILITIES not being defined at all, or either of these flags not being defined at all):

    • PMIX_CAP_SOMETHING_ELSE
    • PMIX_CAP_BOGUS (which doesn't exist -- this simulates a new PRRTE with an old PMIx)

@rhc54 rhc54 force-pushed the pr/check-for-pmix-capability-flags branch from 18904ff to bc7f1b1 Compare December 2, 2023 15:12
@rhc54 rhc54 marked this pull request as ready for review December 13, 2023 15:51
jsquyres and others added 2 commits December 13, 2023 08:52
Demo showing how to check for PMIx capability flags.  This commit:

* Will abort configure if:
  * PMIX_CAPABILITIES does not exist (i.e., this PMIx is so old that
    it does not have any capability flags)
  * PMIX_CAP_202311CLI capability flag goes not exist
  * PMIX_CAP_202311CLI is not set in PMIX_CAPABILITIES

* Will warn if these capability flags are not set in PMIX_CAPABILITIES
  (to include PMIX_CAPABILITIES not being defined at all, or either of
  these flags not being defined at all):
  * PMIX_CAP_SOMETHING_ELSE
  * PMIX_CAP_BOGUS (which doesn't exist -- this simulates a new PRRTE
    with an old PMIx)

Signed-off-by: Jeff Squyres <[email protected]>
Capability definition moved to pmix_version.h.
Remove example usage and replace it with a simple
check to exercise the code.

Signed-off-by: Ralph Castain <[email protected]>
@rhc54 rhc54 force-pushed the pr/check-for-pmix-capability-flags branch from 5f4572a to 8d5060a Compare December 13, 2023 15:52
@rhc54 rhc54 merged commit 23bbb82 into openpmix:master Dec 13, 2023
12 checks passed
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.

2 participants