Skip to content

Conversation

@bosilca
Copy link
Member

@bosilca bosilca commented Mar 10, 2025

The logic to install the dpc was missing a protection around the case where sphinx was installed but the modules required for OMPI were not. I slightly change the order of operations to ensure that we never try to install the docs if there were not available (aka not from a tarball) or not properly generated.

The logic to install the dpc was missing a protection around the case
where sphinx was installed but the modules required for OMPI were not. I
slightly change the order of operations to ensure that we never try to
install the docs if there were not available (aka not from a tarball) or
not properly generated.

Signed-off-by: George Bosilca <[email protected]>
@bosilca bosilca force-pushed the topic/fix_doc_generation branch from 9615285 to 9101740 Compare March 10, 2025 02:05
Copy link
Member

@jsquyres jsquyres left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch.

@jsquyres
Copy link
Member

Do you want to merge this and then update the relevant OMPI submodules (main and v5.0.x)?

@bosilca
Copy link
Member Author

bosilca commented Mar 10, 2025

How do we update the submodule consistently across all projects (ompi, prrte and openpmix) ?

@jsquyres
Copy link
Member

How do we update the submodule consistently across all projects (ompi, prrte and openpmix) ?

We don't. We just have to update the submodule pointer everywhere, which means a bunch of individual PRs. ☹️

However, this is still far better than having N copies of this m4 code all over the place. Consolidating the OAC definitely fixed that problem and really helped avoid all the m4 code drift that was really causing issues across the different projects.

@rhc54
Copy link
Contributor

rhc54 commented Mar 10, 2025

I'll take care of PRRTE and PMIx once this gets committed.

@bosilca
Copy link
Member Author

bosilca commented Mar 10, 2025

I can't merge, I guess I don't have write access.

@jsquyres jsquyres merged commit 5e954b6 into open-mpi:main Mar 11, 2025
1 check passed
@jsquyres
Copy link
Member

I can't merge, I guess I don't have write access.

Merged!

@rhc54
Copy link
Contributor

rhc54 commented Mar 12, 2025

PMIx and PRRTE have been updated!

@bosilca bosilca deleted the topic/fix_doc_generation branch October 2, 2025 04:53
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.

3 participants