Skip to content

fix: bump pg_raw_parse for macOS link error, add macOS to CI - #1336

Merged
jkaczman merged 2 commits into
pgdogdev:mainfrom
jkaczman:macos-ci
Aug 11, 2026
Merged

fix: bump pg_raw_parse for macOS link error, add macOS to CI#1336
jkaczman merged 2 commits into
pgdogdev:mainfrom
jkaczman:macos-ci

Conversation

@jkaczman

@jkaczman jkaczman commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Following up on pgdogdev/pg_raw_parse#49, bumping the pg_raw_parse rev here to fix the macOS link error.

Also added a simple CI job that runs cargo build to prevent regressions. It uses macos-26 instead of macos-14 (used by release CI) since that image will be fully unsupported soon. Follow up PR: #1337

@CLAassistant

CLAassistant commented Aug 11, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@jkaczman jkaczman changed the title Bump pg_raw_parse for macOS link error, add macOS to CI fix: bump pg_raw_parse for macOS link error, add macOS to CI Aug 11, 2026
@codecov

codecov Bot commented Aug 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@levkk levkk left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Nice!

@jkaczman
jkaczman merged commit 55d55cd into pgdogdev:main Aug 11, 2026
28 checks passed
jkaczman added a commit that referenced this pull request Aug 11, 2026
…1337)

macOS 14 runners, which we use in our CI release workflow, [will be
fully unsupported in a few
months](actions/runner-images#13518). Bumped
to use macOS 26 runners.

I also got rid of an unnecessary macOS install dependency step: we don't
build openssl-sys, runner images come with pkgconf, and the build uses
Xcode's libclang (not llvm).

Verified macOS 26 build success in the release workflow in my fork
(running on top of #1336 fix):
https://github.com/jkaczman/pgdog/actions/runs/31486516587/job/93763057440
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.

3 participants