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
oathToolkit: 2.6.2 -> 2.6.6 #120610
oathToolkit: 2.6.2 -> 2.6.6 #120610
Conversation
@schnusch Do you wanna be a maintainer for this package? |
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.
The commit message should be oathToolkit: 2.6.2 -> 2.6.6
since that's what the attribute is called.
let | ||
securityDependency = | ||
if stdenv.isDarwin then xmlsec | ||
else pam; | ||
|
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.
let | |
securityDependency = | |
if stdenv.isDarwin then xmlsec | |
else pam; |
|
||
let | ||
securityDependency = | ||
if stdenv.isDarwin then xmlsec | ||
else pam; | ||
|
||
in stdenv.mkDerivation { | ||
name = "oath-toolkit-2.6.2"; | ||
in 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.
in stdenv.mkDerivation rec { | |
stdenv.mkDerivation rec { |
printf "gdoc_MANS =\ngdoc_TEXINFOS =\n" > libpskc/man/Makefile.gdoc | ||
touch ChangeLog | ||
''; | ||
buildInputs = [ securityDependency ]; |
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.
buildInputs = [ securityDependency ]; | |
buildInputs = [ (if stdenv.isDarwin then xmlsec else pam) ]; |
@SuperSandro2000 That really doesn't make a difference. |
It removes a variable which is not required because first I thought securityDepedency is a packageand missing in inputs. Also the beauty of nix of that syntactic changes don't change anything about the package. |
Motivation for this change
oath-toolkit-2.6.2 is quite out of date, especially oathtool's syntax to read keys from files is missing.
By building the tarball from https://savannah.nongnu.org/projects/oath-toolkit/ instead of https://gitlab.com/oath-toolkit/oath-toolkit all the changes do not seem necessary anymore.
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)