Skip to content

ompi nightly tarball builder - use python venv#97

Open
hppritcha wants to merge 1 commit into
open-mpi:masterfrom
hppritcha:fix_nightly_tarball_builder
Open

ompi nightly tarball builder - use python venv#97
hppritcha wants to merge 1 commit into
open-mpi:masterfrom
hppritcha:fix_nightly_tarball_builder

Conversation

@hppritcha

Copy link
Copy Markdown
Member

since we now rely on sphinx python modules that are not available via linux package managers.

This patch reenables nightly tarball builds which had not been happening since mid January 2026.

since we now rely on sphinx python modules that are not available
via linux package managers.

This patch reenables nightly tarball builds which had not been
happening since mid January 2026.

Signed-off-by: Howard Pritchard <howardp@lanl.gov>
@hppritcha hppritcha requested a review from bwbarrett June 20, 2026 22:33

@jsquyres jsquyres left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@hppritcha Is this fixing a problem? I thought that @bwbarrett made the AWS images such that the sphinx modules were globally installed somewhere (but I could be wrong).

@hppritcha

Copy link
Copy Markdown
Member Author

nope they are not. the sphinxobj thingy isn't part of a distro and @bwbarrett suggested the venv approach.
Without this change there are no nightly tarballs being generated.

# not all of which are available via linux distros,
# so set up a virtual env to run the command in
#
REQ_FILE="${PWD}/docs/requirements.txt"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can you make this check 2 things:

  1. If ${PWD}/requirements.txt exists, use that.
  2. Otherwise, if ${PWD}/requirements.txt exists, use that.

With the llms.txt stuff that went in, we now have a top-level requirements.txt that contains the union of all the pip's we need across the OMPI source tree.

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