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
Update php packages #65846
Update php packages #65846
Conversation
version = "3.1.5"; | ||
pthreads = if isPhp73 then pthreads32-dev else pthreads32; | ||
|
||
pthreads32 = assert (pkgs.config.php.zts or false); assert !isPhp73; buildPecl 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.
assert (pkgs.config.php.zts or false);
I don’t know how to make it work correctly
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.
What is the resulting behavior that you need?
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.
If enable ztsSupport (config.php.zts or true) need build pthreads. If (pkgs.config.php.zts or false) - disable build
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.
But not if it's not php73 I assume?
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.
yes
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.
to php 7.3 need pthreads 3.2.0-dev
cc @etu |
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've have ran through the whole nix-review. Everything did build on my system, except for the oci8 packages. But those failed due to missing source archives. And I'm not willing to get those because it requires an oracle account.
The php upgrades should also be backported to 19.03 |
|
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.
Almost only patch updates, a couple positive reviews, and it's been a week already.
(cherry picked from commit 7819d3f) This is a security release which also contains several bug fixes. http://www.php.net/ChangeLog-7.php#7.2.21
(cherry picked from commit cb9846c) This is a security release which also contains several bug fixes. http://www.php.net/ChangeLog-7.php#7.3.8
I picked the PHP bumps to 19.03, but I haven't looked if anything else needs to. |
Motivation for this change
Update php packages:
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)Notify maintainers
cc @