Skip to content

Fix Docker mounts for manylinux builds#37

Merged
madhavajay merged 3 commits intomainfrom
madhava/release-issue
Jan 8, 2026
Merged

Fix Docker mounts for manylinux builds#37
madhavajay merged 3 commits intomainfrom
madhava/release-issue

Conversation

@madhavajay
Copy link
Copy Markdown
Contributor

Summary

  • Mount sibling deps at exact paths Cargo expects
  • Previous approach: /parent mount + symlinks (failed - Cargo resolves relative to actual workspace path)
  • New approach: mount at ${{ github.workspace }}/../syft-crypto-core so ../syft-crypto-core resolves correctly

Test plan

  • CI builds pass on this PR (verifies Linux wheel builds work)
  • After merge, trigger release and verify PyPI upload succeeds

PyPI rejected linux_aarch64 wheels because manylinux: auto on native
ARM runners skips Docker, producing non-manylinux platform tags.

Using explicit manylinux_2_28 forces Docker container usage, producing
PyPI-compatible manylinux_2_28_* wheel tags.
Previous approach mounted parent at /parent and created symlinks,
but Cargo resolves paths relative to actual workspace location.

Now mounts sibling deps at their exact expected paths so Cargo's
`../syft-crypto-core` resolves correctly inside Docker container.
@madhavajay madhavajay merged commit 2c364a3 into main Jan 8, 2026
14 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