Skip to content

hogql-parser: add build-hogql-parser.yml for riscv64 wheels - #1876

Merged
luhenry merged 2 commits into
mainfrom
hogql-parser
Sep 14, 2026
Merged

luhenry merged 2 commits into
mainfrom
hogql-parser

Conversation

@luhenry

@luhenry luhenry commented Sep 13, 2026

Copy link
Copy Markdown
Member

Builds PostHog's ANTLR4-based C++ HogQL parser, a setuptools Extension module living at common/hogql_parser inside the posthog monorepo. Upstream publishes no riscv64 wheel.

Mirrors upstream's build-hogql-parser.yml and its common/hogql_parser/pyproject.toml [tool.cibuildwheel] recipe.

Differs from upstream

  • Sparse-checkout of common/hogql_parser only, not the full monorepo.
  • No git tag exists; pinned to the commit that bumped setup.py to 1.3.84 (same commit hogql-parser-rs hogql-parser-rs: add build-hogql-parser-rs.yml for riscv64 wheels #1804 is pinned to).
  • before-all drops upstream's epel-release/cmake3 (EPEL doesn't exist on riscv64) and builds antlr4-cpp-runtime with Rocky 10's own cmake instead.
  • build/archs overridden 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

  • None - upstream's own workflow doesn't test the wheel either. import hogql_parser eagerly imports posthog.hogql.ast (the posthog Django app, not on PyPI) at module load, confirmed by reproducing the same ModuleNotFoundError against the real PyPI wheel outside the monorepo.

License: OK - MIT (posthog monorepo), antlr4-cpp-runtime built at CI time is BSD-3-Clause.

…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.
@luhenry
luhenry merged commit 65afd25 into main Sep 14, 2026
17 of 22 checks passed
@luhenry
luhenry deleted the hogql-parser branch September 14, 2026 09:28
@luhenry luhenry linked an issue Sep 14, 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.

hogql-parser riscv64 support

1 participant