| 1 Feb 2024 |
 Pol | Jan Tojnar: Could you approve https://github.com/fossar/nix-phps/pull/324 and https://github.com/fossar/nix-phps/pull/323 ? | 07:50:09 |
| Pol | Jan Tojnar: https://github.blog/changelog/2024-01-30-github-actions-macos-14-sonoma-is-now-available/ | 08:20:28 |
| Pol | There's an issue with PHP 8.1 on darwin. The libxml patch is applied here: https://github.com/fossar/nix-phps/pull/321/files#diff-7d838ade1f0467c8bab1cdc2fc34fd867b20eb61cf6e26f50599dde42d776f27R48 but in the build log (https://pipelinesghubeus23.actions.githubusercontent.com/kOCT2mqNiaOhTutxCPe8hYXDv6z0C4UWNeYg9Obg2gmPygfshU/_apis/pipelines/1/runs/2590/signedlogcontent/18?urlExpires=2024-02-01T08%3A56%3A58.9001059Z&urlSigningMethod=HMACV1&urlSignature=OLmIQFW%2F0eJgM%2FfYIoALRA43HCG0X5tttWkPjpG3LHo%3D), I don't see it applied anywhere. Do you have a clue? | 08:59:12 |
| Pol | Someone with a darwin machine is free to help in here :) | 08:59:22 |
| Pol | I think I discovered an issue in phps.nix | 09:04:56 |
| Pol | 
Download image.png | 09:06:13 |
| Pol | Running nix build .#php81 -L doesn't throw the error. | 09:06:32 |
| Pol | I see, that is only applied to PHP versions we maintain in fossar/nix-phps, not in PHP versions that currently are in nixpkgs. | 09:10:53 |
| Pol | That's confusing. | 09:11:07 |
| Pol | Priority PR to merge to fix all of this shit: https://github.com/NixOS/nixpkgs/pull/283175#pullrequestreview-1855951367 | 09:15:27 |
| Pol | Jan Tojnar: We also need this new PR https://github.com/NixOS/nixpkgs/pull/285511 | 09:38:01 |
| Pol | Jan Tojnar: shouldn't we do this instead (instwad of the patch in https://github.com/NixOS/nixpkgs/pull/283175) ? https://github.com/NixOS/nixpkgs/pull/285542 | 13:15:40 |
| Pol | * Jan Tojnar: shouldn't we do this instead (instead of the patch in https://github.com/NixOS/nixpkgs/pull/283175) ? https://github.com/NixOS/nixpkgs/pull/285542 | 13:15:47 |
| Pol | Since the issue in only with Darwin AND php81 | 13:18:18 |
| Pol | I'm entering in a meeting, I'll check your replies later. | 13:18:27 |
 Jan Tojnar | In reply to @drupol:matrix.org
Jan Tojnar: shouldn't we do this instead (instead of the patch in https://github.com/NixOS/nixpkgs/pull/283175) ? https://github.com/NixOS/nixpkgs/pull/285542 That is a hacky workaround that might mask actual issues (especially when we add more patches). Patch is a more proper solution. | 13:20:24 |
| Jan Tojnar | And it is issue with Darwin just for now. It will be an issue everywhere once we bump gcc. | 13:21:13 |
| Pol | OK ! Fair enough. | 14:21:37 |
| Pol | Shall we continue with this PR then ? https://github.com/NixOS/nixpkgs/pull/283175 | 14:21:44 |
| 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 |
