| 27 Nov 2022 |
 hexa | works on helix 😄 | 20:42:42 |
| hexa | can't say I like php tho | 20:42:50 |
 Pol | Just ignore the coding style... and use phpcsfixer... | 21:10:05 |
| hexa | that's the plan 😄 | 22:46:32 |
| 29 Nov 2022 |
 tgerbet | Hey,
After encountering a segfault while running Psalm I saw that JIT SEAlloc in PCRE was re-enabled in https://github.com/NixOS/nixpkgs/pull/184634 (I disabled it in https://github.com/NixOS/nixpkgs/pull/184634)
The issue in the PHP bug tracker is indeed closed because it was considered there was nothing left to do on their side but it is still problematic.
Basically it breaks when the app fork(). This is mentioned in the PCRE README "It does not support fork() operation" https://www.pcre.org/readme.txt
As I said in my initial PR this option is likely to cause more troubles than the advantages it may bring. Especially since PHP 8 has a JIT on its own, running it in restricted env is likely to require more than enabling the PCRE JIT SEAlloc option.
How would you like to proceed on this etu Pol ?
| 16:41:13 |
| tgerbet | I drafted a PR extending the existing PCRE test to demonstrate the issue https://github.com/NixOS/nixpkgs/pull/203634 | 17:54:51 |
| 30 Nov 2022 |
 etu | I guess we should include that test and a fix for it and backport it to 22.11 as well then :) | 05:58:00 |
| Pol | I think the same. | 07:08:43 |
| tgerbet | Thanks for the confirmation, PR updated :) | 14:35:53 |
| etu | Drunk github-actions bot is drunk: https://github.com/NixOS/nixpkgs/commit/9126bcf60802d2263dfe0d4fac3e2c31947f3c07#commitcomment-91555612 | 15:06:56 |
| Pol | Oh it has already been merged | 15:11:30 |
| Pol | The filename was erroneous :S | 15:11:35 |
| Pol | Oops sorry forget what I said. | 15:12:02 |
| Pol | Thanks for the PR tgerbet !!! | 15:12:17 |
| tgerbet | Thanks for the fast merge 😄 | 15:17:05 |
| 1 Dec 2022 |
| hexa changed their display name from hexa to hexa (22.11 now). | 13:09:13 |
| hexa changed their display name from hexa (22.11 now) to hexa. | 14:39:21 |
| 5 Dec 2022 |
| Pol | Fifth day of the Advent Of Code | 08:29:13 |
| Pol | I'm having fun :) | 08:29:18 |
| 6 Dec 2022 |
| Pol | Posting my solutions here: https://github.com/drupol/aoc2022 | 09:21:40 |
| 7 Dec 2022 |
| Pol | You crazy etu ! | 13:50:28 |
| Pol | I had a good laugh when I saw the notification :D | 13:50:49 |
| Pol | You already did it while it's not yet announced... beautiful :D | 13:51:04 |
| etu | Well, you know... I happen to have been around a while and I know that they have put it in mirrors the day before in the past | 13:51:53 |
| etu | So I tried it | 13:51:56 |
| etu | And I did see this today: https://stitcher.io/blog/new-in-php-82 | 13:52:05 |
| etu | So it reminded me that it's probably there now :) | 13:52:16 |
| Pol | I don't read sticher's blog | 13:53:10 |
| etu | I saw it on Mastodon :) | 13:53:21 |
| etu | But I've seen their posts in the past and they are usually good | 13:53:34 |
