Skip to content

install pcms and PyOmega_h python modules into spack's platlib - #22

Merged
jacobmerson merged 1 commit into
SCOREC:mainfrom
jacobmerson:python-install-location
Oct 22, 2026
Merged

jacobmerson merged 1 commit into
SCOREC:mainfrom
jacobmerson:python-install-location

Conversation

@jacobmerson

Copy link
Copy Markdown
Collaborator

Both packages installed their python module under
${CMAKE_INSTALL_LIBDIR}/pythonX.Y/site-packages, which is lib64/ on RHEL, while spack only puts lib/pythonX.Y/site-packages on PYTHONPATH. So import pcms and import PyOmega_h failed after spack load.

Patch each project's CMake so the install dir is an overridable cache variable (default lib/pythonX.Y/site-packages) and the module's rpath is computed from wherever it is installed, then pass spack's python_platlib from the recipes. Drop the patches once merged upstream.

@jacobmerson
jacobmerson force-pushed the python-install-location branch from 2592380 to 713dc3d Compare October 21, 2026 04:59
…atlib

- Point omega-h 11.4.0-scorec at omega_h master (07ec34b), which includes
  SCOREC/omega_h#198 (r3d::Few strict-aliasing fix), and drop the local
  r3d-few-strict-aliasing.patch. Move pumi-pic from 11.2.0-scorec to
  11.4.0-scorec so it gets the fix too.
- PyOmega_h installed into ${CMAKE_INSTALL_LIBDIR}/pythonX.Y/site-packages
  (lib64/ on RHEL), which spack doesn't put on PYTHONPATH. Patch omega_h so
  the install dir is an overridable cache variable with an rpath computed
  from it, and pass spack's python_platlib. Drop once merged upstream.
- Pass python_platlib to pcms as PCMS_PYTHON_INSTALL_LOCATION, which is
  overridable since SCOREC/pcms#394.
@jacobmerson
jacobmerson force-pushed the python-install-location branch from 713dc3d to 06eb106 Compare October 22, 2026 06:10
@jacobmerson
jacobmerson merged commit 568c3dc into SCOREC:main Oct 22, 2026
4 checks passed
@jacobmerson
jacobmerson deleted the python-install-location branch October 22, 2026 14: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.

1 participant