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
fverb: init at unstable-2020-06-09 #85776
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.
Leave notes about why unstable. Maybe change line in all-packages.nix
@GrahamcOfBorg build fverb
@drewrisinger Thanks, all done. |
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.
Looked at the repo, and it looks fairly out-of-date/inactive. Makes me semi wary of packaging this.
Also, in terms of naming, there seems to be some precedent for naming LV2 plugins as name-lv2
or name.lv2
. I really don't have much familiarity with this ecosystem, just an observation. See e.g. https://github.com/NixOS/nixpkgs/blob/84cf00f98031e93f389f1eb93c4a7374a33cc0a9/pkgs/applications/audio/ir.lv2/default.nix
It does build locally via nixpkgs-review pr 85776
. Untested
https://github.com/NixOS/nixpkgs/pull/85776
1 package built:
fverb
The repo is pretty new, yes, but not inactive or out of date. Therefore I would argue: this package is mature, and should be included. As for the name: I named the package ir.lv2 because upstream is called that. |
You are correct. My mistake, moving too quick & making assumptions. Ack. Changing to approved. |
Convention is to write dates as YYYY-MM-DD, b/c then versions are strictly monotonically increasing left to right. Please change the commit message/title. |
}: | ||
stdenv.mkDerivation rec { |
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.
nitpicking, but...
}: | |
stdenv.mkDerivation rec { | |
}: | |
stdenv.mkDerivation rec { |
@drewrisinger drew All done! If you are interested, I have a bunch of other, mostly audio related, PR's open! ;) |
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
- Builds via OfBorg (aarch64, amd64)
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
Thanks. |
Thank you! |
Motivation for this change
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)