Skip to content

[8.0] Align Windows CI setup with 10.0 - #3952

Merged
roji merged 1 commit into
hotfix/8.0.12from
roji-windows-ci-8-0
Oct 14, 2026
Merged

roji merged 1 commit into
hotfix/8.0.12from
roji-windows-ci-8-0

Conversation

@roji

@roji roji commented Oct 14, 2026 •

Copy link
Copy Markdown
Member

Backport the relevant Windows CI fixes from #3649, #3650, and #3931, without changing this branch's PostgreSQL, SDK, or action versions.

  • Remove the Windows PostGIS overlay that breaks pgcrypto.dll loading in #3948.
  • Keep one Windows PostgreSQL 16/Release leg; retain every existing Linux matrix entry.
  • Remove unused SSL certificate setup and print logfile if PostgreSQL startup fails.
  • Require PostGIS on Linux via NPGSQL_TEST_POSTGIS, while allowing PostGIS tests to skip on Windows. Preserve the 8.0 test-utility namespace.

Verified on commit 108f6454f7a1bf4b3a2c147daed8ec8d0dcaaa6d: Windows PostgreSQL 16/Release passed. Functional tests: 25,286 passed, 530 skipped; unit tests: 345 passed, 7 skipped; NodaTime tests: 284 passed. Zero failures. All five Linux legs and dependency submission passed; publish/release jobs were skipped as expected for a PR.

Local validation: functional-test project builds cleanly; eight focused extension/PostGIS tests pass; 18 PostGIS condition cases pass; existing spatial tests all use conditional attributes. An isolated server without PostGIS confirmed seven optional tests skip and four required spatial/database-creator tests fail as expected. Workflow YAML, Bash syntax, and matrix preservation checked.

Stop overlaying PostGIS DLLs onto PostgreSQL on Windows and run only the PostgreSQL 16 Windows leg. Preserve Linux coverage and require PostGIS there. Remove unused SSL setup and print the server log on startup failure.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 36992289-a0b8-4975-aa51-1ec19a38004a
@roji
roji merged commit 809a4c9 into hotfix/8.0.12 Oct 14, 2026
9 checks passed
@roji
roji deleted the roji-windows-ci-8-0 branch October 14, 2026 18:35
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