Skip to content

[codex] Add MSYS2 MinGW CI coverage#110

Draft
oconnor663 wants to merge 1 commit into
masterfrom
codex/add-msys2-mingw-ci
Draft

[codex] Add MSYS2 MinGW CI coverage#110
oconnor663 wants to merge 1 commit into
masterfrom
codex/add-msys2-mingw-ci

Conversation

@oconnor663

@oconnor663 oconnor663 commented Jun 23, 2026

Copy link
Copy Markdown
Owner

(experiment, not yet reviewed by me, Codex description below)

Summary

Add a focused MSYS2/MinGW CI job to the tests workflow so source builds are exercised under MSYS2 Python and the MinGW Rust/GCC toolchain.

The job currently runs for UCRT64 and MINGW64, installs MSYS2-packaged Python, pip, maturin, Rust, and GCC, then builds this repo from source with pip and smoke-tests the installed extension import. This is intended to catch failures like #109 where PyO3/MinGW linking breaks even though the normal Windows/MSVC path still passes.

Validation

  • Parsed .github/workflows/tests.yml with Ruby YAML
  • Ran git diff --check

The actual MSYS2 build needs to run on GitHub Actions.

python -m pip --version
maturin --version
- name: Build and install from source
run: python -m pip install --no-build-isolation --no-deps --verbose .

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

required deps should be installed with pacman (the same pacboy field used above) or inside of venv

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.

2 participants