Skip to content

refactor(nix/cargo-pgrx): build cargo-pgrx by overriding upstream pkg - #2666

Draft
brainrake wants to merge 4 commits into
developfrom
martonboros/mpg-1348-cargo-pgrx-override
Draft

brainrake wants to merge 4 commits into
developfrom
martonboros/mpg-1348-cargo-pgrx-override

Conversation

@brainrake

@brainrake brainrake commented Nov 13, 2026 •

Copy link
Copy Markdown
Collaborator

Replace the vendored cargo-pgrx expression with an override of the upstream nixpkgs package. This drops the manual dependency wiring and the meta block.

The crates.io workarounds (fix-cargo.nix, registryDl, auditable = false) stay on top of the override.

Built psql_17/bin on aarch64-darwin from develop and from an earlier revision of this branch, before the latest develop merge. The closure diff shows no version changes. The only differences in wrappers-0.4.4.dylib are the embedded build path, the Mach-O UUID, and the code signature hashes.

Not checked: psql_15, orioledb-17, Linux, nix flake check.

Based on earlier work by Kranzes.

MPG-1348

Kranzes and others added 4 commits May 31, 2026 00:23
Replace the vendored and outdated cargo-pgrx expression with an override
of the upstream nixpkgs package, dropping the manual dependency wiring,
custom check flags, and meta block.

Remove the cargo-pgrx overlay and call the package set
directly via callPackages in the package list. Using the overlay shadows
the upstream package, and in addition just adds another layer of
unneeded indirection.
Refs: MPG-1348

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Refs: MPG-1348

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Refs: MPG-1348

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@blacksmith-sh

blacksmith-sh Bot commented Nov 13, 2026 •

Copy link
Copy Markdown

Blacksmith runners detected OOM events on the following jobs:

Job Details
Check Docker Image Changes View Job
Package Dependency Analysis View Job

@github-actions

Copy link
Copy Markdown

Package version diff: none ()

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