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
afterburn: init at 4.6.0 #68680
afterburn: init at 4.6.0 #68680
Conversation
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.
nix-review
passes on NixOS
diff LGTM (few suggestions above)
binary works
leaf package
Please don't merge yet! Still a WIP. For one I want to ship the systemd units that afterburn ships |
systemd services will probably have to be done as nixos modules, unless you're talking about some daemon programs |
Yes but when possible we tend to use the upstream units and set them using the |
e1d2f76
to
ed31e3d
Compare
sup with this? |
In stuck on the comments by @edolstra about not using fixed output derivations for rust (he commented about that in the NixOS module PR). I haven't checked out carnix yet, perhaps that can help |
fyi getting page not found on https://github.com/coreos/cargo |
How do we want to proceed with this? @vyp We're referring to |
a6f69bd
to
f97d9f2
Compare
77ce955
to
11fd28a
Compare
This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch). Result of 1 package built:
|
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 are also some initrd
-specific pieces of afterburn e.g. https://github.com/coreos/afterburn/blob/master/dracut/30afterburn/afterburn-hostname.service
but i guess we should address those in the nixos module ; not here.
So LGTM (though I cant approve as I originally opened this)
This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch). Result of 1 package built:
|
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @