Skip to content

quickjs-ng: add build-quickjs-ng.yml for riscv64 wheels - #1783

Merged
luhenry merged 2 commits into
mainfrom
quickjs-ng
Sep 12, 2026
Merged

luhenry merged 2 commits into
mainfrom
quickjs-ng

Conversation

@luhenry

@luhenry luhenry commented Sep 11, 2026

Copy link
Copy Markdown
Member

Compiles the vendored QuickJS-NG C engine plus a thin CPython extension (_quickjs) into one wheel. Upstream publishes no riscv64 wheel.

Mirrors upstream's build.yml build-wheels job.

Differs from upstream

  • Builds from the git checkout, not the PyPI sdist - the sdist omits quickjs-c-atomics.h, breaking compilation
  • Stages upstream-quickjs/LICENSE at the wheel root - setup.py's default licence glob misses it otherwise

Matrix: cp310-abi3 only, manylinux + musllinux - upstream's own abi3 floor (py_limited_api = "cp310"); no free-threaded wheel since upstream skips cp*t-*.

Testing

  • same as upstream

License: Wheel bundles the vendored quickjs-ng engine (MIT); upstream's sdist ships no licence text for it, so the build adds it.

Built on cp310; 188 passed, 0 skipped.

luhenry added a commit that referenced this pull request Sep 11, 2026
…nse check

Its own entry ends in a trailing slash, so rsplit('/', 1)[1] on it yields
an empty string, not a filename -- the set never matched {LICENSE, LICENSE.upstream-quickjs}.
@luhenry
luhenry merged commit 0306d0a into main Sep 12, 2026
7 checks passed
@luhenry
luhenry deleted the quickjs-ng branch September 12, 2026 19:28
@luhenry luhenry linked an issue Sep 12, 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.

quickjs-ng riscv64 support

1 participant