Remove flake-inputs; refactor and split out package #39
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.
Besides removing a dependency for the project, this refactors the flake to be a bit cleaner.
Using
flake-utils
makes expressing "for each system" type expression simple, but it also gets in the way of items that don't need this type of expression, such as themodule.nix
. Thus we had to define an improper module expression, which makes using the module in another expression more complicated as well.So instead we create
forAllSystems
andnixpkgsFor
functions explicitly. This means a little more involved package expressions, but that hide less from the user. The module can now beimport
'ed and it can be used as a module in other machine configurations without extra treatment. I stole this idea from this.I also took the opportunity to split out the main package expression into its own file, for clarity.