Skip to content

dnaio: add build-dnaio.yml for riscv64 wheels - #1375

Merged
luhenry merged 1 commit into
mainfrom
dnaio
Sep 7, 2026
Merged

dnaio: add build-dnaio.yml for riscv64 wheels#1375
luhenry merged 1 commit into
mainfrom
dnaio

Conversation

@luhenry

@luhenry luhenry commented Sep 7, 2026

Copy link
Copy Markdown
Member

Compiles a single Cython extension (dnaio._core) for reading/writing FASTA/FASTQ/BAM sequencing data, from the cutadapt ecosystem. Upstream publishes no riscv64 wheel.

Mirrors the wheels job of upstream's ci.yml.

Differs from upstream

  • Shallow checkout has no tag history, so SETUPTOOLS_SCM_PRETEND_VERSION_FOR_DNAIO stands in for setuptools_scm's git-based versioning.

Matrix: cp310-cp314t, matching upstream's own published wheel tags - the only runtime dependency (xopen) is pure Python, so nothing caps the floor.

Testing

  • same as upstream

License: OK - MIT, nothing vendored; the only compiled source (ascii_check.h/bam.h) gates its sole SIMD path behind __x86_64__ and falls back to portable C elsewhere.

Built on cp312 (native, not riscv64); 645 passed, 0 skipped.

Small single-extension Cython package (FASTA/FASTQ reading/writing, part
of the cutadapt ecosystem). The only runtime dependency, xopen, is pure
Python (py3-none-any), and the sole C source (ascii_check.h/bam.h) gates
its only SIMD path behind __x86_64__, so no patching is needed. Matrix
matches upstream's own published wheel tags (cp310-cp314, cp314t).
luhenry added a commit that referenced this pull request Sep 7, 2026
luhenry added a commit that referenced this pull request Sep 7, 2026
@luhenry
luhenry merged commit e0cc487 into main Sep 7, 2026
11 checks passed
@luhenry
luhenry deleted the dnaio branch September 7, 2026 16:05
@luhenry luhenry linked an issue Sep 7, 2026 that may be closed by this pull request
@luhenry luhenry mentioned this pull request Sep 7, 2026
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.

dnaio riscv64 support

1 participant