fix: centralize bindgen hook override for pgrx extensions

Move the bindgen hook fix for ARM NEON vector ABI issues from
pg_graphql to mkPgrxExtension, making it available to all pgrx-based
extensions. This fix applies to pgrx versions 0.6.1 through 0.11.2
when building on aarch64-linux.

Author: jfroche

nix/cargo-pgrx/buildPgrxExtension.nix

@@ -34,6 +34,7 @@
   stdenv,
   darwin,
   writeShellScriptBin,
+  defaultBindgenHook,
 }:
 
 # The idea behind: Use it mostly like rustPlatform.buildRustPackage and so
@@ -56,7 +57,7 @@
   # enable override to generate bindings using bindgenHook.
   # Some older versions of cargo-pgrx use a bindgenHook that is not compatible with the
   # current clang version present in stdenv
-  bindgenHook ? rustPlatform.bindgenHook,
+  bindgenHook ? defaultBindgenHook,
   # cargo-pgrx calls rustfmt on generated bindings, this is not strictly necessary, so we avoid the
   # dependency here. Set to false and provide rustfmt in nativeBuildInputs, if you need it, e.g.
   # if you include the generated code in the output via postInstall.

nix/cargo-pgrx/mkPgrxExtension.nix

@@ -4,6 +4,7 @@
   pgrxVersion,
   makeRustPlatform,
   rust-bin,
+  system,
 }:
 let
   inherit ((callPackage ./default.nix { inherit rustVersion; })) mkCargoPgrx;
@@ -31,8 +32,28 @@ let
       inherit (mapping) hash cargoHash;
       version = pgrxVersion;
     };
+
+  bindgenHook =
+    # Fix bindgen error on aarch64-linux for versions using pgrx with bindgen 0.68.1
+    # This affects pgrx 0.6.1 through 0.11.2 which have issues with ARM NEON vector ABI
+    if (builtins.compareVersions "0.11.3" pgrxVersion > 0) then
+      let
+        nixos2211 = (
+          import (builtins.fetchTarball {
+            url = "https://channels.nixos.org/nixos-22.11/nixexprs.tar.xz";
+            sha256 = "1j7h75a9hwkkm97jicky5rhvzkdwxsv5v46473rl6agvq2sj97y1";
+          }) { inherit system; }
+        );
+      in
+      rustPlatform.bindgenHook.overrideAttrs {
+        libclang = nixos2211.clang.cc.lib;
+        clang = nixos2211.clang;
+      }
+    else
+      rustPlatform.bindgenHook;
 in
 callPackage ./buildPgrxExtension.nix {
   inherit rustPlatform;
   inherit cargo-pgrx;
+  defaultBindgenHook = bindgenHook;
 }

nix/ext/pg_graphql/default.nix

@@ -104,30 +104,6 @@ let
           inherit (postgresql.meta) platforms;
         };
       }
-      //
-        lib.optionalAttrs
-          (
-            # Fix bindgen error on aarch64-linux for versions using pgrx with bindgen 0.68.1
-            # This affects pgrx 0.6.1 through 0.11.2 which have issues with ARM NEON vector ABI
-            # We apply the fix to all versions up to 1.5.1 (last version before 1.5.4 which uses 0.11.2)
-            builtins.compareVersions "1.5.4" version > 0
-          )
-          {
-            # Fix bindgen error on aarch64-linux by using an older version of clang
-            bindgenHook =
-              let
-                nixos2211 = (
-                  import (builtins.fetchTarball {
-                    url = "https://channels.nixos.org/nixos-22.11/nixexprs.tar.xz";
-                    sha256 = "1j7h75a9hwkkm97jicky5rhvzkdwxsv5v46473rl6agvq2sj97y1";
-                  }) { inherit system; }
-                );
-              in
-              rustPlatform.bindgenHook.overrideAttrs {
-                libclang = nixos2211.clang.cc.lib;
-                clang = nixos2211.clang;
-              };
-          }
       // lib.optionalAttrs (builtins.compareVersions "1.2.0" version >= 0) {
         # Add missing Cargo.lock
         patches = [ ./0001-Add-missing-Cargo.lock-${version}.patch ];

nix/ext/pg_jsonschema/default.nix

@@ -6,6 +6,8 @@
   fetchFromGitHub,
   postgresql,
   rust-bin,
+  system,
+  rustPlatform,
 }:
 let
   pname = "pg_jsonschema";