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
cmt: init at 1.17 #82683
cmt: init at 1.17 #82683
Conversation
@jonringer Does anyone know why I can't request review on this? It's greyed 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.
is there supposed to be headers? all that's exported is a .so
$ tree ./results/cmt
./results/cmt
└── lib
└── ladspa
└── cmt.so
no idea, but I've noticed github having issues for a while |
Thanks again! |
Excellent question! But no, these are plugins. Nothing is built against them. |
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 you mind squashing the changes commit into the original commit. Otherwise LGTM
Not at all! |
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.
LGTM
[1 built, 0.0 MiB DL]
https://github.com/NixOS/nixpkgs/pull/82683
1 package built:
cmt
[nix-shell:/home/jon/.cache/nixpkgs-review/pr-82683-1]$ nix path-info -Sh ./results/cmt
/nix/store/kdvk7lwcdgcj2k03djjh1w7im5694768-cmt 37.5M
@GrahamcOfBorg build cmt |
Motivation for this change
Introduction of new package.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
(New pkg -- no dependents)./result/bin/
)nix path-info -S
before and after)