Skip to content

Run Windows on CI#64

Merged
nokonoko1203 merged 32 commits into
mainfrom
ci/windows
Jan 30, 2026
Merged

Run Windows on CI#64
nokonoko1203 merged 32 commits into
mainfrom
ci/windows

Conversation

@yutannihilation
Copy link
Copy Markdown
Member

@yutannihilation yutannihilation commented Jan 29, 2026

Close #0

What I did

  • Enable Windows build on CI (test & release) by using vcpkg
  • Remove build cache on release build (Since the release process is invoked less frequently, the cache is likely to be outdated, which might make some problem. But, I don't have strong opinion here)
  • Some minor tweaks
    • Update actions/checkout to v6

Notes

This pull request attempts to use vcpkg to install PROJ v9.6.2. While vcpkg feels a bit too heavy to solve the problem, but it seems this is the only reliable way.

For reference, these are the possible alternative path, but both requires windows-gnu target instead of windows-msvc. Now that windows-gnu is less maintained, I think we should use windows-msvc for now (I hope we can migrate to windows-gnullvm someday).

  • Use MSYS2 and install mingw-w64-ucrt-x86_64-proj
  • Use proj-sys's bundled_proj feature and install SQLite via Chocolaty

Comment thread .github/workflows/build.yml Outdated
run: cargo build --verbose --package app

- name: Clippy (lint)
if: runner.os == 'Linux'
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

It seems proj-sys's bundled feature doesn't work on Windows, so we need to skip here.

  -- EMBED_RESOURCE_FILES=ON
  -- Configuring incomplete, errors occurred!

  --- stderr
  feature flags specified source build
  building libproj from source
  disabling tiff support
  CMake Error at CMakeLists.txt:211 (message):
    sqlite3 binary not found!


  CMake Warning at CMakeLists.txt:248 (message):
    TIFF support is not enabled and will result in the inability to read some
    grids



  thread 'main' (2764) panicked at C:\Users\runneradmin\.cargo\registry\src\index.crates.io-1949cf8c6b5b557f\cmake-0.1.54\src\lib.rs:1119:5:

  command did not execute successfully, got: exit code: 1

  build script failed, must exit now
  note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
warning: build failed, waiting for other jobs to finish...
Error: Process completed with exit code 1.

@yutannihilation yutannihilation marked this pull request as ready for review January 30, 2026 01:18
Copy link
Copy Markdown
Collaborator

@nokonoko1203 nokonoko1203 left a comment

Choose a reason for hiding this comment

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

great work!!!
lgtm!!!

@nokonoko1203 nokonoko1203 merged commit f540fd3 into main Jan 30, 2026
5 checks passed
@nokonoko1203 nokonoko1203 deleted the ci/windows branch January 30, 2026 04:34
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