| 27 Nov 2022 |
 etu | I don't have an error message other than "it crashed quickly" | 19:33:30 |
| etu | If I run the same program by full path with same command line flag, it starts up fine. | 19:33:56 |
| etu | Since both... systems are built from the same nix config locked to the same versions... the only real thing that differs is the... hostname :D | 19:35:06 |
 Pol | Oh | 19:48:42 |
| Pol | Intelephense on vscode ? | 19:48:48 |
| Pol | Never had issues with it :S | 19:48:53 |
| Pol | I wish it could be more active and provide more updates | 19:49:03 |
| Pol | I even paied the license to support the dev | 19:49:14 |
| etu | In reply to @drupol:matrix.org
Intelephense on vscode ? No, emacs | 20:32:13 |
 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 |
