Skip to content

add GFX Spark ABI wheels#29

Merged
bghira merged 1 commit intomainfrom
feature/gfx-spark-builds
Apr 27, 2026
Merged

add GFX Spark ABI wheels#29
bghira merged 1 commit intomainfrom
feature/gfx-spark-builds

Conversation

@bghira
Copy link
Copy Markdown
Owner

@bghira bghira commented Apr 27, 2026

This pull request updates the CI and publish workflows for building and verifying Python wheels on Linux aarch64, with a focus on improving reliability, caching, and build environment consistency. The main changes include refactoring the aarch64 build job to use a static OpenCV build, updating environment setup steps, and ensuring the built wheels are properly tested and repaired before upload.

Build process improvements:

  • The build-wheels-aarch64 job in .github/workflows/ci.yml now builds OpenCV statically using a dedicated script, caches the build to speed up CI, and sets up all required environment variables for consistent builds.
  • The Python setup step now uses the correct matrix variable (python instead of python-version) and is executed earlier in the workflow for better dependency management.

Wheel building and verification:

  • The wheel build and repair process is now more robust: wheels are built using maturin-action, repaired with a custom script, and unrepaired wheels are pruned before installation and testing.
  • After building, the workflow installs the wheel and imports the trainingsample module to verify the build succeeded. The same verification step is enforced in the publish workflow (publish.yml) by removing the fallback to ignore import errors. [1] [2]

Environment and dependency handling:

  • The workflow now ensures all relevant environment variables for LLVM, libclang, and OpenCV are set correctly, including dynamic detection of libclang.so if needed.
  • The build dependencies list has been expanded to include additional tools required for building static libraries (e.g., curl, nasm, yasm).

@bghira bghira merged commit 9948da5 into main Apr 27, 2026
10 checks passed
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