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
php: added argon2 option for php 7.2+ #46785
Conversation
cc @etu |
@GrahamcOfBorg build php72 |
Success on x86_64-linux (full log) Attempted: php72 Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: php72 Partial log (click to expand)
|
@GrahamcOfBorg build php71 |
Success on x86_64-linux (full log) Attempted: php71 Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: php71 Partial log (click to expand)
|
@Mic92 Looks good to me, I'm fine with backporting this since it's supported by upstream and is in general a good feature. I don't know about other's opinions though. If you backport this, make sure to not bring the php72 and php71 updates to 7.1.10 and 7.1.22 because of this: https://github.com/NixOS/nixpkgs/blob/master/pkgs/development/interpreters/php/default.nix#L40-L43 |
@aanderse One note though, it would be nice if you can amend the commit message to something like |
Just to be sure, I should be basing this change against the staging-18.09 branch, not the release-18.09 branch? |
@aanderse Now you have three commits instead of one https://github.com/NixOS/nixpkgs/pull/46785/commits You seem to have keeped your original commit instead of amending it and then causing a merge. I don't have merge rights, that's why I ping @Mic92 about merging/backporting things. But he can probably sort out your commits as well unless you do it. |
Motivation for this change
The Argon2 algorithm is not enabled in php on NixOS.
I would like this change to be included with 18.09 if possible.
http://php.net/manual/en/password.constants.php
https://bugs.php.net/bug.php?id=76360
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)