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
tacacs+: init 4.0.4.28 #64595
tacacs+: init 4.0.4.28 #64595
Conversation
@GrahamcOfBorg build tacacs+ |
I"m not certain, how to proceed on the licensing dimension of this PR.
|
cc @adisbladis |
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.
This seems like a package that might benefit from also having a NixOS service. If that is true, were you interested in working on that as well?
ebed406
to
3c213e9
Compare
Yes, I'd like to work on a service after this PR. |
Since the license appears to be bespoke (I couldn't find any uses of it
outside this software), I think we should just use licenses.free rather
than adding it to licenses.nix.
|
460b6fe
to
aeec3d9
Compare
aeec3d9
to
b1bf3a8
Compare
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 👍
@GrahamcOfBorg build tacacs+ |
Motivation for this change
Adding package for a TACACS+ implementation.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)