-
-
Notifications
You must be signed in to change notification settings - Fork 14.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove empty go vendor deps #126682
Remove empty go vendor deps #126682
Conversation
Should we blacklist that hash and throw an error if it is used? |
An empty vendor derivation doesn't seem to be a real problem. By turning it into an error, we are creating a potential problem. Whether we merge or close this pr is rather inconsequential. If this little discovery doesn't lead to any relevant insights for the go maintainers, that's also ok and they can ignore this. |
Result of 3 packages built successfully:
1 suggestion:
Result of 3 packages built successfully:
1 suggestion:
|
I would make it a lint check instead in nixpkgs-hammering instead. |
Opened a PR for this #212024. |
@Artturin might be interesting to add to https://github.com/nix-community/nixpkgs-lint |
Motivation for this change
A little discovery I've made when introducing the
emptyDirectory
package #126680: these vendor directories are empty.cc @NixOS/golang in case this is new information that's useful somehow.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)