Skip to content

edlib: add build-edlib.yml for riscv64 wheels - #1244

Merged
luhenry merged 2 commits into
mainfrom
edlib
Sep 7, 2026
Merged

edlib: add build-edlib.yml for riscv64 wheels#1244
luhenry merged 2 commits into
mainfrom
edlib

Conversation

@luhenry

@luhenry luhenry commented Sep 7, 2026

Copy link
Copy Markdown
Member

Cython binding compiling edlib's C++ edit-distance (Levenshtein) core into a per-interpreter extension. Upstream publishes no riscv64 wheel.

Mirrors upstream's python_edlib job (ci.yaml) and bindings/python/Makefile's wheels target.

Differs from upstream

  • Cythonizes edlib.pyx in CIBW_BEFORE_BUILD instead of via make sdist - skips the cog-generated README.rst, which needs a built wheel to generate.

Testing

  • Same as upstream: python3 {project}/test.py.

License: Wheel now ships the root LICENSE under dist-info/licenses/; upstream's own PyPI wheels carry none at all.

Patches

  • 0001-ship-the-MIT-LICENSE-in-the-wheel-s-dist-info.patch - To upstream [no fork to push from]. Without it the wheel has no LICENSE file; reproduces off riscv64 too.

Built off the checkout on cp312 (macOS/arm64, local, off-riscv64); test.py passed.

Cython/C++ edit-distance library. Wraps edlib's C++ core, compiled
per-interpreter from a Cython-generated .cpp (no pyproject.toml, and
the pre-generated source isn't checked in upstream either).

Ships the root LICENSE into dist-info/licenses/, which upstream's own
wheels omit entirely (patches/edlib/1.3.9.post1).
luhenry added a commit that referenced this pull request Sep 7, 2026
before-build's cwd is {project} (the invocation dir), not {package}
(the package-dir subdir) -- cython was looking for edlib.pyx at the
checkout root and failing on every job. Prefix with {package}.
luhenry added a commit that referenced this pull request Sep 7, 2026
@luhenry
luhenry merged commit 5619a50 into main Sep 7, 2026
13 checks passed
@luhenry
luhenry deleted the edlib branch September 7, 2026 08:27
@luhenry luhenry mentioned this pull request Sep 7, 2026
@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.

edlib riscv64 support

1 participant