Skip to content

pgrx: use clang from nixpkgs-oldstable for old pgrx builds - #2647

Draft
mmlb wants to merge 1 commit into
developfrom
push-kmzzzuxtppqt
Draft

mmlb wants to merge 1 commit into
developfrom
push-kmzzzuxtppqt

Conversation

@mmlb

@mmlb mmlb commented Nov 12, 2026

Copy link
Copy Markdown
Collaborator

What kind of change does this PR introduce?

Gardening

What is the current behavior?

We fetch nixpkgs-22.11 during nix eval, every eval.

What is the new behavior?

We don't do that any more, making eval go slightly faster.

Old pgrx versions (before 0.11.3) need an older clang to build on
aarch64-linux. We got it by downloading the whole nixos-22.11 tarball
during eval. That download was slow, could not come from our binary
cache, and ran on every platform.

Take clang 12 from nixpkgs-oldstable instead and only apply it on
aarch64-linux like the comment implies.

nixpkgs-oldstable's oldest clang is 12 while 22.11 defaulted to 11, but
the build succeeds where a newer clang would fail.

Assisted-by: Claude <noreply@anthropic.com>
@mmlb

mmlb commented Nov 12, 2026

Copy link
Copy Markdown
Collaborator Author

Draft to see if clang12 works or not.

@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.

1 participant