| 22 Sep 2024 |
|  @patka:envs.net joined the room. | 13:49:24 |
| @patka:envs.net | Hello 👋
Currently packages built with buildComposerProject2 can't properly be updated because of the autoloader bug.
The bug seems to be in the apcu-autoloader that is enabled as a flag in the new composer builder, but wasn't used in the old one.
What about removing the flag from the new version until it's fixed upstream? It's not that big of a deal. Then we can atleast fix and update packages. | 13:54:37 |
 Pol | Yes this is because of that flag indeed, but I would have prefered to have it fixed upstream though | 14:23:35 |
| @patka:envs.net | Yeah, sure. Me too
But it wasnt enabled in the first composer builder. The packages that use the second builder are broken.
So we can either move the v2 packages back to v1, or temporarily disable the broken flag in v2 that wasn't even used in v1.
We can't not update the packages that currently use v2 until upstream decides to fix it | 14:26:54 |
| @patka:envs.net | * Yeah, sure. Me too
But it wasnt enabled in the first composer builder. The packages that use the second builder are broken.
So we can either move the v2 packages back to v1, or temporarily disable the broken flag in v2 that wasn't even used in v1.
We can't not update the packages that currently use v2 until upstream decides to sometimes maybe possibly fix it. As soon as upstream fixes it we enable acpu-autoloader again
| 14:36:02 |
| @patka:envs.net | * Yeah, sure. Me too
But it wasnt enabled in the first composer builder. The packages that use the second builder are broken.
So we can either move the v2 packages back to v1, or temporarily disable the broken flag in v2 that wasn't even used in v1.
We can't not update the packages that currently use v2 until upstream decides to sometime maybe possibly fix it. As soon as upstream fixes it we enable acpu-autoloader again
| 14:36:18 |
| Pol | fair enough. I agree. | 15:07:44 |
| @patka:envs.net | I'll PR it tonight and share it here :) | 15:17:44 |
| @patka:envs.net | https://github.com/NixOS/nixpkgs/pull/343762 | 17:02:06 |
| Pol | excellent. | 17:14:24 |
| @patka:envs.net | When that's merged I'll bump all php packages. Probably the psysh issue will then also be fixed | 17:36:32 |
| Pol | Thank you patka Just merged it. | 19:03:26 |
| 23 Sep 2024 |
| @patka:envs.net | I fixed nix-update, but my PR still has an error in a (python) test. I tested it manually and it works, just this test is somehow broken.
Maybe anyone here feels like having a look? https://github.com/Mic92/nix-update/pull/283 | 06:27:27 |
| Pol | patka: Do you have a clue for this? https://github.com/NixOS/nixpkgs/pull/341746#pullrequestreview-2321219018 | 06:52:27 |
 Genghiz | @drupol:matrix.org I’ve added PR 343898 which solves this issue.
| 07:17:08 |
| Genghiz | It should clear it up for all packages built like firefly-iii | 07:17:23 |
| Genghiz | But if possible do run the check locally and make sure of it. | 07:17:55 |
| Pol | Genghiz: I have some thing a bit more advanced | 07:17:58 |
| Genghiz | Oo | 07:18:03 |
| Genghiz | Tell me more? | 07:18:09 |
| Pol | Let me share it in a PR in 2 minutes | 07:18:27 |
| Genghiz | Awesome. | 07:18:34 |
| @patka:envs.net | So this means I don't have to look and there's already a solution? | 07:19:58 |
| Pol | https://github.com/NixOS/nixpkgs/pull/343900 | 07:21:21 |
| Pol | patka: Yeah :) | 07:21:30 |
| Pol | patka: I've also fixed this: https://github.com/NixOS/nixpkgs/pull/343900/commits/3bcaabbb7119cb06f7f79770baf2e11ad293955c | 07:21:45 |
| @patka:envs.net | Oopsie, thanks! | 07:23:59 |
| @patka:envs.net | Just have a small question on the PR above. Apart from that it looks good. Genghiz would you be able to test that it works with the firefly-iii work? | 07:25:48 |
| @patka:envs.net | * Just have a small question on the PR above. Apart from that it looks good. Genghiz would you be able to test/validate that it works with the firefly-iii work? | 07:26:05 |
| Genghiz | In reply to @patka:envs.net
Just have a small question on the PR above. Apart from that it looks good. Genghiz would you be able to test/validate that it works with the firefly-iii work? Gimme an hour I’ll do that. | 07:29:47 |
