ncdns: fix ncdns.goModules derivation, 2022-10-07 -> 2024-05-18 #331542
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of changes
It took an ungodly amount of time but I may have done it:
Fix the build error in ncdns.goModules due to a module rename in a
downstream dependency
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.
Make ncdns behave more like a normal Go module and use the built-in
phases of goBuildModule.
Bonus update
Fixes #331188
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
ncdns.tests