Skip to content

html-to-markdown: add build-html-to-markdown.yml for riscv64 wheels - #1060

Open
luhenry wants to merge 4 commits into
mainfrom
html-to-markdown
Open

html-to-markdown: add build-html-to-markdown.yml for riscv64 wheels#1060
luhenry wants to merge 4 commits into
mainfrom
html-to-markdown

Conversation

@luhenry

@luhenry luhenry commented Sep 6, 2026

Copy link
Copy Markdown
Member

Compiles a PyO3/maturin extension (html_to_markdown._html_to_markdown, abi3-py310) from the packages/python member of a large polyglot monorepo. Upstream publishes no riscv64 wheel.

Mirrors the python-wheels job of upstream's publish.yaml.

Differs from upstream

  • Drops the yum install openssl-devel and aarch64-gcc-symlink steps from upstream's before-build - neither dependency in this crate tree needs them (verified with a local cargo check; TLS is rustls-based, no openssl-sys anywhere).

Matrix: single cp310-abi3 build; no free-threaded wheel exists upstream either (abi3-py310 is unconditional in [tool.maturin]).

Testing

  • Runs upstream's own black-box e2e suite (e2e/python/tests), the same tests alef test --e2e --lang python exercises in CI, instead of that private tool.

License: OK - MIT, LICENSE already checked into packages/python/.

test_options_preprocessing_{aggressive,minimal} fail with
NameError: name 'PreprocessingPreset' is not defined on every architecture,
not just riscv64: the test module's top-level import list omits
PreprocessingPreset even though two tests use it and html_to_markdown's
public __init__.py exports it. Already fixed on upstream main, just not in
a tagged release yet.
Documents diagnosing a NameError in an e2e test as a broken test file
at the pinned tag (already fixed upstream, unreleased) rather than a
riscv64 or extension-export problem, and why -k over --deselect avoids
the rootdir-nodeid trap gotcha 283 already hit.
luhenry added a commit that referenced this pull request Sep 6, 2026
2 test_options.py tests failed with a pre-existing v3.12.0 NameError
unrelated to riscv64 (PreprocessingPreset unimported in the test file,
already fixed unreleased on upstream main); deselected via -k in
CIBW_TEST_COMMAND and documented as gotcha 285.
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.

1 participant