Skip to content

memory-allocator: add build-memory-allocator.yml for riscv64 wheels - #1378

Merged
luhenry merged 1 commit into
mainfrom
memory-allocator
Sep 7, 2026
Merged

memory-allocator: add build-memory-allocator.yml for riscv64 wheels#1378
luhenry merged 1 commit into
mainfrom
memory-allocator

Conversation

@luhenry

@luhenry luhenry commented Sep 7, 2026

Copy link
Copy Markdown
Member

Compiles a tiny Cython extension class (MemoryAllocator) that SageMath and related projects use for manual memory allocation. Upstream publishes no riscv64 wheel.

Mirrors upstream's dist.yml build_wheels job.

Differs from upstream

  • Pins the release commit instead of the v0.2.0 tag - that tag predates upstream's pyproject.toml version bump.

Testing

  • same as upstream

License: GPL-3.0-or-later; upstream ships no LICENSE text in its wheels because meson-python has no default LICENSE glob, so the patch adds one.

Patches

  • 0001-ship-the-GPL-3.0-or-later-LICENSE-in-the-wheel-s-dis.patch - To upstream (no fork to push from). Every wheel upstream publishes today lacks a LICENSE file; reproduces off riscv64.

Built and tested locally on cp314 (macOS, patched pyproject.toml); 3 doctests passed, 0 skipped.

Small Cython extension (used by SageMath) built with meson-python;
cibuildwheel builds directly from a checkout of the upstream repo.

The v0.2.0 git tag predates the version bump in pyproject.toml (a
history rewrite around the meson migration left it behind), so the
workflow pins the actual release commit instead, whose tree is
byte-identical to the released PyPI sdist.

Also carries a patch adding `license-files = ["LICENSE"]` to
pyproject.toml: meson-python has no default LICENSE glob the way some
other backends do, so every wheel upstream publishes today ships with
none.
@luhenry
luhenry merged commit 8ce4d70 into main Sep 7, 2026
13 checks passed
@luhenry
luhenry deleted the memory-allocator branch September 7, 2026 16:06
@luhenry luhenry linked an issue Sep 7, 2026 that may be closed by this pull request
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.

memory-allocator riscv64 support

1 participant