It took an ungodly amount of time but I may have done it:
1. Fix the build error in ncdns.goModules due to a module rename in a
downstream dependency
2. Add lock files to avoid the same kind of problem that already broke
the build of ncdns.goModules.
Note that due to the nature of the x509-compressed module the source
code dependes on the Go stdlib, so the goModules derivation
is still not really fully reproducible, but it comes closer.
3. Make ncdns behave more like a normal Go module and use the built-in
phases of goBuildModule.
In 787af0f79f
I had to change ${go-modules} to $goModules to allow overrideAttrs to work;
However, env vars cannot contain -, so i had to change go-modules too.
This in turn broke nix-update because it uses the go-modules attr.
Instead of making nix-update more complicated, make go-modules naming match cargoDeps.
`fd --type f | xargs sd '\bgo-modules\b' 'goModules'`
and revert change to pkgs/applications/misc/dstask/default.nix
and pkgs/servers/http/dave/default.nix
and pkgs/os-specific/darwin/plistwatch/default.nix
release note added