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: add lts version #22358
asterisk: add lts version #22358
Conversation
@yorickvP, thanks for your PR! By analyzing the history of the files in this pull request, we identified @DerTim1, @auntieNeo and @fpletz to be potential reviewers. |
I can check this out in a few hours... |
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.
Compilation in NixOS is ok and works with my Asterisk-Config.
pkgs/servers/asterisk/default.nix
Outdated
lts = ./lts.nix; | ||
}; | ||
in | ||
{ stdenv, pkgs, |
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.
I think this is unusual style. The only differences between the two versions (lts & stable) are version and hash. So it would be smarter to switch inside one nix-file, compare for example PostgreSQL https://github.com/NixOS/nixpkgs/blob/master/pkgs/servers/sql/postgresql/default.nix
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.
okay, I changed it to be inside one nix-file like postgresql.
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.
Great, looks good to me!
Motivation for this change
Asterisk has a few different versions available. Some of them might be more suitable for server use than others. I added a way to specify which asterisk version to use, and also a way to fetch external dependencies (it was doing it at build time, leading to hydra failures).
I'm not sure if what I'm doing is the correct way.
I also added myself as a maintainer to this package.
(cc: @grahamc, @DerTim1, @fpletz )
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)