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

v5.0 PMIX_MAX_NSLEN and MAX_KEYLEN #448

Closed
2 tasks done
abouteiller opened this issue Apr 14, 2023 · 0 comments · Fixed by #447
Closed
2 tasks done

v5.0 PMIX_MAX_NSLEN and MAX_KEYLEN #448

abouteiller opened this issue Apr 14, 2023 · 0 comments · Fixed by #447
Assignees
@abouteiller
Labels
Correction Editorial
Milestone
PMIx v5 Standard

Comments

@abouteiller
Copy link
Contributor

abouteiller commented Apr 14, 2023 •
edited
Loading

Overview

These two constants are set to 255 and 511, but an advise later say they can be set to 63. This is problematic for the ABI.

pmix-standard/Chap_API_Struct.tex

Line 44 in d165b57

\declareconstitemvalue{PMIX_MAX_NSLEN}{255}

Actions

  • Verify that no other constants are impacted the same
  • Remove advise that say that 63 is a valid value: for the current ABI it is as defined in the normative part of the document.

Discussion Items

ASC Chairs recommend editorial change procedure.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@abouteiller abouteiller

Labels
Correction Editorial
Projects
None yet
Milestone
PMIx v5 Standard
Development

Successfully merging a pull request may close this issue.

Editorial changes from proofread of v5.0 final document abouteiller/pmix-standard
1 participant
@abouteiller