| 1 Feb 2024 |
 Pol | The tests you ran this morning are green. | 14:21:58 |
 Jan Tojnar | Pol: would be also nice to check if fetchpatch cannot be used. | 16:53:24 |
| Pol | Jan Tojnar: Do we need 1 or 2 patches in https://github.com/NixOS/nixpkgs/pull/283175 | 16:57:51 |
| Jan Tojnar | One should be enough, just mention it in two places | 16:59:46 |
| Jan Tojnar | Or maybe we could reuse php patches in extensions using php src | 17:00:18 |
| Pol | My head is going to explode man... | 17:05:41 |
| Pol | So let's summarize a little bit | 17:07:56 |
| Pol | I would like to unblock nix-phps today if possible. | 17:08:07 |
| Pol | I've been dedicating a couple of hours to this, and I really wish I could do something else :) | 17:08:26 |
| Pol | So, for now, regarding this PR: https://github.com/NixOS/nixpkgs/pull/283175 | 17:08:44 |
| Pol | There's one simple patch, patching at the same time libxml and soap PHP extensions. | 17:09:05 |
| Pol | Tests seems to pass as seen on the PR. | 17:09:22 |
| Pol | Should we do something else? | 17:09:28 |
| Pol | I just pushed a commit to use the patch from php-src. | 17:28:39 |
| Pol | * I just pushed a commit to use the patch from github:php/php-src. | 17:28:55 |
| Jan Tojnar | Not a huge fan of the ellipsis, maybe removeAttrs fetchpatch from _attrs | 17:36:38 |
| Jan Tojnar | We can probably ignore soap for now since it is non-fatal | 17:39:04 |
| Pol | Right | 17:39:28 |
| Jan Tojnar | But it should be as easy as adding `patches = php.unwrapped.patches or [] ++ args.patches or []` to `mkExtension` in `php-packages.nix` | 17:42:05 |
| Pol | I removed the ellipsis | 17:43:27 |
| Pol | Now, what's left in there? | 17:43:52 |
| Pol | Jan Tojnar: Are you ok with this one ? https://github.com/NixOS/nixpkgs/pull/285511 | 18:32:35 |
| Jan Tojnar | In reply to @drupol:matrix.org
Jan Tojnar: Are you ok with this one ? https://github.com/NixOS/nixpkgs/pull/285511 Not really for me to decide, Nixpkgs PHP maintainers will bear the cost of extra code not needed for Nixpkgs | 18:53:22 |
| Pol | Ok but since I will most probably manage that, ... | 19:05:34 |
| Pol | I guess it's ok if I merge it ? | 19:05:40 |
| Jan Tojnar | In that case, it is probably fine. Just note that in the past, we have always cleaned references to PHP versions not available in Nixpkgs. Hence the common use of `builtins.filter` in nix-phps | 19:18:11 |
| Pol | Right let's do that. | 19:18:58 |
| 2 Feb 2024 |
| Pol | Elis 🌱: For you: https://github.com/NixOS/nixpkgs/pull/285828 | 15:58:15 |
| Pol | Redacted or Malformed Event | 16:03:48 |
| Pol | Asking for a repository composer.lock is sometimes successful, sometimes not... https://github.com/php-parallel-lint/PHP-Parallel-Lint/issues/153 | 18:09:03 |
