cross: graphicsmagick: push patch upstream

nixpatches/list.nix

@@ -38,6 +38,13 @@ in [
     hash = "sha256-cC9GBF5tCeJ2GDSjMjlG4hYStIJPEoRBAK9/KhJiiIo=";
   })
 
+  (fetchpatch' {
+    # this forces an expensive rebuild, mostly just to reduce moby's closure by 40 MiB (if even)
+    prUrl = "https://github.com/NixOS/nixpkgs/pull/291942";
+    title = "imagemagick, graphicsmagick: remove build coreutils from cross-compiled output";
+    hash = "sha256-jUtMmT4d+/6hZgrpXhfBcToAfdecl7xtEj1d/ofxaIM=";
+  })
+
   # (fetchpatch' {
   #   title = "nixos/slskd: allow omitting username from yaml config";
   #   saneCommit = "541c37e8689b6422ea07be1395f1a63357bb0c63";

overlays/cross.nix

@@ -1021,13 +1021,15 @@ in with final; {
   });
 
   # 2024/02/27: upstreaming is unblocked
-  graphicsmagick = prev.graphicsmagick.overrideAttrs (upstream: {
-    # by default the build holds onto a reference to build `mv`
-    # N.B.: `imagemagick` package has this identical issue (but requires a different fix?)
-    configureFlags = upstream.configureFlags ++ [
-      "MVDelegate=${coreutils}/bin/mv"
-    ];
-  });
+  # this patch isn't strictly necessary: it mostly just removes build tools from the runtime closure
+  # out for PR: <https://github.com/NixOS/nixpkgs/pull/291942>
+  # graphicsmagick = prev.graphicsmagick.overrideAttrs (upstream: {
+  #   # by default the build holds onto a reference to build `mv`
+  #   # N.B.: `imagemagick` package has this identical issue (but requires a different fix?)
+  #   configureFlags = upstream.configureFlags ++ [
+  #     "MVDelegate=${coreutils}/bin/mv"
+  #   ];
+  # });
 
   # fixes: "make: gcc: No such file or directory"
   # java-service-wrapper = useEmulatedStdenv prev.java-service-wrapper;