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: Enable mysqlnd as default #44117
Conversation
@GrahamcOfBorg build php56 php70 php71 |
MySQL Native Driver was implemented by PHP back in 5.3.0 and has been default in most distributions for a very long time. The option was added in 41cd4f2 and I don't see any reason why it would default to false. Overview of mysqlnd by PHP: https://secure.php.net/manual/en/mysqlnd.overview.php
Success on x86_64-linux (full log) Attempted: php56, php70, php71 Partial log (click to expand)
|
I thought that the bot would build php72 as well since that's the default @GrahamcOfBorg build php72 |
Success on aarch64-linux (full log) Attempted: php56, php70, php71 Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: php72 Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: php Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: php72 Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: php Partial log (click to expand)
|
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.
Looks good to me! Currently, I have to override this for my deployments, which entails recompiling php on the webservers..
Motivation for this change
MySQL Native Driver was implemented by PHP back in 5.3.0 and has been
default in most distributions for a very long time.
The option was added in 41cd4f2 and I don't see any reason why it would default to false.
Overview of mysqlnd by PHP: https://secure.php.net/manual/en/mysqlnd.overview.php
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)cc @globin @FRidh @adisbladis @srhb @elitak @talyz