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
klipper: init at 0.8.0 #98917
klipper: init at 0.8.0 #98917
Conversation
795c04f
to
6754732
Compare
efa7d8d
to
fb8ef7a
Compare
fb8ef7a
to
79392d9
Compare
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.
Last nit I have is: I would separate this into two commits: klipper: init at 0.8.0
, and nixos/klipper: init
.
(Brownie points for using lib
instead of stdenv.lib
:) )
79392d9
to
6a56424
Compare
Done! |
0b8751b
to
fc80004
Compare
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.
Diff LGTM, thanks!
fc80004
to
b49fdd7
Compare
8fb8fd8
to
24dce9e
Compare
Alright, I've tested this today and it works 🥳 |
24dce9e
to
4f817bb
Compare
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.
Just some minor comments but looks good otherwise.
4f817bb
to
3ab38c9
Compare
a2cb9df
to
dcef04d
Compare
dcef04d
to
97eadef
Compare
message = "Option klipper.octoprintIntegration requires Octoprint to be enabled on this system. Please enable services.octoprint to use it."; | ||
}]; | ||
|
||
environment.etc."klipper.cfg".source = format.generate "klipper.cfg" cfg.settings; |
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 know I'm several years late, but why are we putting the klipper.cfg in /etc? We later pass the path to klipper, which means it shouldn't need to be in /etc?
Motivation for this change
I need this to run the 3D printer I'm building. The firmware runs a daemon on a RPi that talks/commands it, which is what is being packaged here.
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)