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

Add scripts for laying ROCM artifacts out for Python packaging. #120

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

stellaraccident
Copy link
Contributor

@stellaraccident stellaraccident commented Feb 27, 2025

Functional changes:

  • Adds a script linux_python_dist_split.py which materializes build artifacts into a package structure that is conducive for distribute as pip sdists and wheels (additional work will be done to actually overlay the Python packaging artifacts with this for a combined whole).
  • Substantial documentation on the packaging design is in that script.
  • Adds _therock_utils.artifacts which can be used for introspecting a directory full of build artifacts.
  • Adds _therock_utils.exe_stub_gen which can dynamically generate Linux executables which operate as if they were a symlink (since we can't reliably have executable symlinks in Python packages).
  • Adds TARGET_NEUTRAL to the SuiteSparse build.

Organizational changes:

  • Extracts some Python utilities into an actual package (_therock_utils).
  • Moves scripts that were in packaging/portable-linux to the root build_tools and names things consistently (i.e. prefix with linux_).
  • Moves portable linux output to /output-linux-portable and adds it to .gitignore.

@stellaraccident stellaraccident changed the title Draft of python packaging. Add scripts for laying ROCM artifacts out for Python packaging. Mar 1, 2025
@stellaraccident stellaraccident marked this pull request as ready for review March 1, 2025 03:31
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