Skip to content

Follow-up to #228: CI for the MSVC pkg-config fallback; one shared pkg-config probe - #229

Merged
dginev merged 4 commits into
mainfrom
followup/pkgconfig-msvc
Nov 6, 2026
Merged

dginev merged 4 commits into
mainfrom
followup/pkgconfig-msvc

Conversation

@dginev

@dginev dginev commented Nov 6, 2026 •

Copy link
Copy Markdown
Member

Follow-up to #228.

  • CI: a new job, Windows MSVC pkg-config (conda-forge). It installs libxml2-devel and pkg-config from conda-forge with no vcpkg, runs cargo test, then checks the build script's stderr to confirm the fallback was taken.
    • Red: main plus this job fails with Could not find libxml2. (run)
    • Green: this branch passes (run). Its log also shows the vcpkg error, which was discarded before.
  • build.rs: a single pkg_config_find_libxml2() serves both Unix and the MSVC fallback, so the fallback also honours LIBXML2_STATIC. No CI covers a static MSVC build. vcpkg_find_libxml2 now returns vcpkg's error, and the build prints it next to pkg-config's.
  • Cargo.toml: requires pkg-config >= 0.3.14, the first release whose MSVC support works according to its CHANGELOG. The build works with 0.3.14.
  • README: explains how to point an MSVC build at a pkg-config libxml2, and that VCPKGRS_DISABLE=1 skips vcpkg.

🤖 Generated with Claude Code

dginev and others added 4 commits November 6, 2026 12:51
…onfig

No vcpkg, as in conda-forge's own builds, so build.rs takes the pkg-config
fallback; a last step checks that it did.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The MSVC fallback now honours LIBXML2_STATIC as the Unix branch does, and
vcpkg's error is printed instead of discarded, next to pkg-config's.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…t works

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@dginev
dginev changed the base branch from pr-228 to main November 6, 2026 17:51
@dginev
dginev force-pushed the followup/pkgconfig-msvc branch from e3f70f6 to e9898f6 Compare November 6, 2026 17:51
@dginev
dginev merged commit ad7ae25 into main Nov 6, 2026
33 checks passed
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