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
kconf: init at 1.10.1 #117265
kconf: init at 1.10.1 #117265
Conversation
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
pkgs/top-level/all-packages.nix
Outdated
@@ -30461,4 +30461,6 @@ in | |||
lc3tools = callPackage ../development/tools/lc3tools {}; | |||
|
|||
zktree = callPackage ../applications/misc/zktree {}; | |||
|
|||
kconf = callPackage ../applications/networking/cluster/kconf { }; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please sort this entry into a category and there alphabetically.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tried to find a pattern, but there are networking/cluster packages spread across the entire file. Can you point to where this entry should go?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There should be a headline like ```### NETWORK`` and yeah we need to programmatically sort this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I couldn't find a ### NETWORK
header, but I moved it near some other Kubernetes-related tools. Is this better?
@SuperSandro2000, would you mind taking another look at this? Thanks. |
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)