Skip to content

protovalidate: add build-protovalidate.yml for riscv64 wheels - #1670

Merged
luhenry merged 1 commit into
mainfrom
protovalidate
Sep 10, 2026
Merged

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

Conversation

@luhenry

@luhenry luhenry commented Sep 10, 2026

Copy link
Copy Markdown
Member

Compiles a PyO3 extension that statically links six vendored C++ projects (abseil-cpp, antlr4, cel-cpp, protobuf, re2, protovalidate-cc) into a CEL-based protobuf validator. Upstream publishes no riscv64 wheel.

Mirrors upstream's release.yaml, using cibuildwheel instead of maturin-action directly.

Differs from upstream

  • musllinux dropped - rustup ships no riscv64 musl toolchain.

Testing

  • test pyproject.toml staged to keep the root protovalidate/ source tree from shadowing the installed wheel's compiled extension
  • pytest/pytest-benchmark/pydantic pinned to upstream's dev-required versions; protobuf (Google's package, legacy conformance coverage) pulled from our registry

License: wheel statically links abseil-cpp/cel-cpp/protovalidate-cc (Apache-2.0) and antlr4/protobuf/re2/utf8_range (BSD-3-Clause/MIT); upstream's license-files names only its own LICENSE, so a patch adds the six vendored ones.

Patches

  • 0001-list-vendored-c-licence-files-in-project-license-fi.patch - To upstream, not submitted (session policy). Without it every published wheel ships none of the six vendored licence texts. Reproduces on any platform, not riscv64-only.

CI running; test counts to follow once green.

luhenry added a commit that referenced this pull request Sep 10, 2026
@luhenry
luhenry merged commit 6efc01d into main Sep 10, 2026
7 checks passed
@luhenry
luhenry deleted the protovalidate branch September 10, 2026 10:43
@luhenry luhenry linked an issue Sep 10, 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.

protovalidate riscv64 support

1 participant