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
Add new package "ncpamixer" #34564
Add new package "ncpamixer" #34564
Conversation
make PREFIX=$out | ||
''; | ||
|
||
meta = with stdenv.lib; { |
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.
Will you be the maintainer of this expression?
@GrahamcOfBorg build ncpamixer |
Failure on x86_64-darwin (full log) Partial log (click to expand)
|
Success on x86_64-linux (full log) Partial log (click to expand)
|
Success on aarch64-linux (full log) Partial log (click to expand)
|
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 check the contributing guidelines on how to name commits.
nativeBuildInputs = [ cmake pkgconfig ]; | ||
|
||
configurePhase = '' | ||
make PREFIX=$out |
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.
While this works I think the proper approach would be:
--- a/pkgs/applications/audio/ncpamixer/default.nix
+++ b/pkgs/applications/audio/ncpamixer/default.nix
@@ -16,7 +16,11 @@ stdenv.mkDerivation rec {
nativeBuildInputs = [ cmake pkgconfig ];
configurePhase = ''
- make PREFIX=$out
+ make PREFIX=$out build/Makefile
+ '';
+
+ buildPhase = ''
+ make build
'';
meta = with stdenv.lib; {
Otherwise you are building the project in the configure phase.
f50423c
to
38b5214
Compare
I fixed the commit message and applied andir patch. |
@StijnDW yes, you can use a separate commit for that but as part of this PR. |
Motivation for this change
The only package I know of that has user a user friendly way to adjust pulse audio volume from a command line interface.
It works the same way as alsamixer but mimics the user interface from pavucontrol.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)