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
nmon: init at 16g #33285
nmon: init at 16g #33285
Conversation
@grahamc build nmon |
@GrahamcOfBorg build nmon |
Success on x86_64-linux (full log) Partial log (click to expand)
|
Success on aarch64-linux (full log) Partial log (click to expand)
|
Failure on x86_64-darwin (full log) Partial log (click to expand)
|
@@ -0,0 +1,11 @@ | |||
source $stdenv/setup |
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.
Nit: this is an old-school way of packaging; it's way more common to inline the builder in the Nix expression/rely on the standard build sequence. In this case, I think you can get away with setting buildPhase
and installPhase
(setting buildCommand
could work too, but then you need more boilerplate).
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.
Do we still have ancient documentation, which show examples like 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.
this is nice for understanding, but there should be a warning sign or something: https://github.com/NixOS/nixpkgs/blob/master/doc/stdenv.xml#L90
@vquintin I also prepared a derivation for https://github.com/NixOS/nixpkgs/compare/master...sveitser:nmon?expand=1 |
yes it would. You can open a pull request, if you want. |
the other pull request was merged. |
Motivation for this change
This PR adds nmon to the nixpkgs.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)