| 12 Nov 2022 |
 @etu:failar.nu | So we would want... NTS by default, right? | 10:19:46 |
| @etu:failar.nu | And then maybe a separate attribute name for php with ZTS | 10:19:58 |
| @etu:failar.nu | like mod_php80 mod_php81 mod_php or something? And then use that by default for apache. | 10:20:46 |
 Pol | yes | 10:21:45 |
| @etu:failar.nu | Pol: You feel up for doing those changes to your draft? | 10:28:42 |
| @etu:failar.nu | Pol: And yeah, stop make it depending on isDarwin | 10:28:56 |
| Pol | I can try :) | 10:48:57 |
| Pol | Job done etu ! https://github.com/NixOS/nixpkgs/pull/194172 | 21:57:31 |
| Pol | I like the change | 21:57:37 |
| @etu:failar.nu | My thinking is, why over complicate it with new attributes and all. Apache users are used to that it needs building (that could be improved as a separate thing) and the unit test for Apache well just work since it enables the Apache module and changes the required parameters. | 21:59:37 |
| @etu:failar.nu | It should probably end up in the change log for the upcoming release though | 22:00:13 |
| Pol | yup | 22:19:43 |
| Pol | I'll take care of the changelog tomorrow | 22:24:03 |
| 13 Nov 2022 |
| Pol | I just created a draft PR so that we can merge PHP 8.2.0 as soon as it is out (8th December): https://github.com/NixOS/nixpkgs/pull/201000 | 09:37:17 |
| Pol | I don't know if we can submit a RC though. | 09:42:26 |
| Pol | Because if we can, that PR is definitely valid. | 09:42:35 |
| Pol | etu: Can you please refresh my memory when it comes to updating the changelog for the release? | 09:43:10 |
| @etu:failar.nu | Pol: I was thinking something like this:
diff --git a/nixos/doc/manual/release-notes/rl-2211.section.md b/nixos/doc/manual/release-notes/rl-2211.section.md
index c4bc7e7db09..d61cf296755 100644
--- a/nixos/doc/manual/release-notes/rl-2211.section.md
+++ b/nixos/doc/manual/release-notes/rl-2211.section.md
@@ -53,6 +53,10 @@ In addition to numerous new and upgraded packages, this release has the followin
- PHP now defaults to PHP 8.1, updated from 8.0.
+- PHP is now built `NTS` (Non-Thread Safe) style by default, for Apache and
+ `mod_php` usage we still enable `ZTS` (Zend Thread Safe). This has been a
+ common practice for a long time in other distributions.
+
- `protonup` has been aliased to and replaced by `protonup-ng` due to upstream not maintaining it.
- Perl has been updated to 5.36, and its core module `HTTP::Tiny` was patched to verify SSL/TLS certificates by default.
| 10:41:56 |
| @etu:failar.nu | Pol: And then you have to run the script ./nixos/doc/manual/md-to-db.sh to generate the docbook entries as well | 10:43:13 |
| Pol | Will do it now. | 10:43:29 |
| Pol | Job done. | 10:47:36 |
| Pol | Also, wdyt about that? https://github.com/NixOS/nixpkgs/pull/201000 | 10:47:47 |
| @etu:failar.nu | I'll look at it soon :) | 10:50:30 |
| @etu:failar.nu | I think the NTS PR should be marked as ready for review | 10:50:45 |
| @etu:failar.nu | And then that we let it eval and such | 10:50:52 |
| @etu:failar.nu | In reply to @drupol:matrix.org
Also, wdyt about that? https://github.com/NixOS/nixpkgs/pull/201000 Also, it seems to build so I'm fine with that. It's also nice to have it before the branch-off since that means that the release will contain the new version from release day. | 10:53:09 |
| Pol | yes :) | 10:53:50 |
 Andreas Schrägle | In reply to @etu:failar.nu
Also, it seems to build so I'm fine with that. It's also nice to have it before the branch-off since that means that the release will contain the new version from release day. does nixpkgs have a policy on pre-release software? | 10:55:13 |
| @etu:failar.nu | Well, I think it depends | 10:55:46 |
| @etu:failar.nu | It's not too uncommon with pinned packages to specific commits and such. And in this case it "will become a stable release" before the release is out. | 10:56:26 |
