hogql-parser: add build-hogql-parser.yml for riscv64 wheels - #1876
Merged
Merged
Conversation
luhenry
added a commit
that referenced
this pull request
Sep 13, 2026
…d wheel hogql_parser eagerly imports posthog.hogql.ast (the posthog Django app, not on PyPI) at module load, so it fails ModuleNotFoundError even on the real published PyPI wheel outside the monorepo it ships with.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
hogql-parser1.3.84Builds PostHog's ANTLR4-based C++ HogQL parser, a setuptools Extension module living at
common/hogql_parserinside the posthog monorepo. Upstream publishes no riscv64 wheel.Mirrors upstream's
build-hogql-parser.ymland itscommon/hogql_parser/pyproject.toml[tool.cibuildwheel]recipe.Differs from upstream
common/hogql_parseronly, not the full monorepo.before-alldrops upstream'sepel-release/cmake3(EPEL doesn't exist on riscv64) and builds antlr4-cpp-runtime with Rocky 10's owncmakeinstead.build/archsoverridden to add the riscv64 identifier - upstream's own list only names macOS/x86_64/aarch64.Matrix: cp310-cp313 - upstream's own coverage at this pinned commit; cp314 support landed later.
Testing
import hogql_parsereagerly importsposthog.hogql.ast(the posthog Django app, not on PyPI) at module load, confirmed by reproducing the sameModuleNotFoundErroragainst the real PyPI wheel outside the monorepo.License: OK - MIT (posthog monorepo), antlr4-cpp-runtime built at CI time is BSD-3-Clause.