Just pushed some more changes to plugins.nix. I replaced references to nonexistent registry.gimp.org with it's archive and made sure, that nixpkgs#glimpse-with-plugins builds.

@jtojnar The more I think about it the lesser I am convinced that maintaining a separate glimpsePlugins set is reasonable atm. The duplication of efforts is already observable. Also I'm not convinced that we'll see glimpse-exclusive or gimp-exclusive plugins anytime in the near future.

Maybe gimpPlugins could globally expose an overridable function similar to pluginDerivation with an optional argument to the gimp/glimpse package? So in glimpsePlugins we could say something like

{
  # simple case for just swapping gimp with glimpse
  gap = gimpPlugins.gap.override { gimpPackage = glimpse; };

  # complex case where overriding makes no sense and an entirely new plugin is constructed.
  fourier = gimpPlugins.buildGimpPlugin {
    pname = "fourier";
    version = "0.4.3";
    gimpPackage = glimpse;
    postPatch = ...;
  };
}

Another way could be to keep gimpPlugins as it is and use overrideAttrs for every gimpPlugin, but I guess that could turn out messy.

What do you think? I have not much insight into this topic, so I guessed I'd ask you first before trying to implement it.

Pushed aa96bd2 to #103361, which should allow gimpPlugins.overrideScope (final: prev: { gimp = glimpse; }). But not sure how much sense it makes since Glimpse is expected to diverge from GIMP.

Thanks, I think that does the job. Much less code now.

Depends now on #103361

@jtojnar as you already did quite a bit ot review work, would you mind taking another look on this PR? Or maybe @ashkitten as the original author? Would be nice to get this merged eventually.

@erictapen please fix the merge conflict.

Fixed the conflict. Took me a while, but glimpse (as well as gimp) need autoconf 2.69.

Multiple people eyeballed this PR and I'm going to maintain all of its code anyway. So without further objections I'll merge this in two days.

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.

Result of nixpkgs-review pr 103288 run on x86_64-darwin 1

error: --- Error ----------------------------------------------------------------------------------------------------------------------------------------------------------------------- nixbuilder for '/nix/store/0vbvca424p055qyr7qqgibrjrjcwrg0b-python-2.7.18-env.drv' failed with exit code 2; last 1 log lines:                                                                    cannot create directory `/nix/store/k0azjfpdlcdx2rv3akdyxsf3zzxw86p6-python-2.7.18-env/': No such file or directory at /nix/store/gn7c8979lw55dg9np9gly7ylhiakbdim-builder.pl line 235.   error: --- Error ----------------------------------------------------------------------------------------------------------------------------------------------------------------------- nix1 dependencies of derivation '/nix/store/vlqm8d2y02cpainvfj96yxnq8g4xq7m4-glimpse-0.2.0.drv' failed to build 

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.

Result of nixpkgs-review pr 103288 run on x86_64-linux 1