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
asterisk: update all versions #50874
Conversation
}: | ||
|
||
let | ||
common = {version, sha256, externals}: stdenv.mkDerivation rec { | ||
inherit version; | ||
name = "asterisk-${version}"; | ||
|
||
buildInputs = [ jansson libxml2 libxslt ncurses openssl sqlite utillinux dmidecode libuuid newt lua speex srtp wget curl iksemel ]; | ||
buildInputs = [ jansson libedit libxml2 libxslt ncurses openssl sqlite utillinux dmidecode libuuid newt lua speex srtp wget curl iksemel pkgconfig ]; |
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.
pkgconfig
belongs to nativeBuildInputs
. It would not hurt to add line breaks in this list.
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.
utillinux
is probably also a candidate for nativeBuildInputs
.
I think PR #48709 is then superfluous. |
Weird to end up with |
Are there any updates on this pull request, please? |
This PR can be closed. #55378 is already merged, so this here is superfluous. |
Add new Asterisk 16 branch, and update other releases.
/cc maintainers @auntie @DerTim1 @yorickvP
sandbox
innix.conf
on non-NixOS)