Skip to content

Bump pinned MEOS to include the wasm pg_config fix#142

Draft
estebanzimanyi wants to merge 1 commit into
fix/stage-icu-macos-platformfrom
fix/bump-meos-pin-wasm-pgconfig
Draft

Bump pinned MEOS to include the wasm pg_config fix#142
estebanzimanyi wants to merge 1 commit into
fix/stage-icu-macos-platformfrom
fix/bump-meos-pin-wasm-pgconfig

Conversation

@estebanzimanyi
Copy link
Copy Markdown
Member

MobilityDuck builds MEOS from a pinned MobilityDB commit; the current pin predates the fix that emits SIZEOF_LONG_LONG, so the DuckDB-Wasm jobs fail in the vendored postgres pg_bitutils.h on wasm32. This advances the pin to a MobilityDB commit that carries that fix (MobilityDB PR #1036). It is provisionally pointed at estebanzimanyi/MobilityDB@40145502c because #1036 is not yet on MobilityDB/MobilityDB master, kept as a Draft for that reason; the landable form points REPO back to MobilityDB/MobilityDB at the post-merge master commit with SHA512 recomputed from that archive. Stacked on fix/stage-icu-macos-platform (#141); its commits show in this diff until it merges.

MobilityDuck builds MEOS from a pinned MobilityDB commit; the current
pin predates the fix that emits SIZEOF_LONG_LONG, so the DuckDB-Wasm
jobs fail in the vendored postgres pg_bitutils.h on wasm32. This
advances the pin to a MobilityDB commit that carries that fix
(MobilityDB PR #1036). It is provisionally pointed at
estebanzimanyi/MobilityDB@40145502c because #1036 is not yet on
MobilityDB/MobilityDB master; the landable form points REPO back to
MobilityDB/MobilityDB at the post-merge master commit with the SHA512
recomputed from that archive.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@estebanzimanyi
Copy link
Copy Markdown
Member Author

Reviewer's quickstart — ~2 minutes

What this PR does: Bump pinned MEOS to include the wasm pg_config fix.

Files to read: vcpkg_ports/meos/portfile.cmake (plus minimal surrounding context).

Risk: narrow scope; the diff is small and self-contained. Stacked on its base PR — once the stack ahead of it lands, this rebases trivially.

Cross-link: Linux arm64 CI needs #161 for the MeosType build error (orthogonal to this PR's content).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant