| 17 Sep 2024 |
 Pol | Sorry can you rephrase? | 15:28:10 |
 @genghiz:talk.go7box.xyz | I was joking that I will now come to you with issues in my python code. | 15:29:34 |
| @genghiz:talk.go7box.xyz | (I was not being serious of course) | 15:29:40 |
| @genghiz:talk.go7box.xyz | Because you have been very nice in helping me with PHP in the past and I will take advantage of your good nature. ^_^ | 15:30:06 |
| Pol | If I can help, I'll gladly do it since improving something in nixpkgs will potentially benefit me at some point too ;) | 15:32:32 |
| @genghiz:talk.go7box.xyz left the room. | 17:16:57 |
|  Genghiz joined the room. | 17:18:44 |
| 20 Sep 2024 |
| Genghiz | https://github.com/NixOS/nixpkgs/pull/341746 | 17:38:47 |
| Genghiz | https://github.com/NixOS/nixpkgs/pull/342556 | 17:38:57 |
| Genghiz | Would appreciate a merge on these. :) | 17:39:05 |
| Genghiz | In reply to @genghiz:talk.go7box.xyz
I'll do that I thought I'd do this but I think it's a bit more complicated than I had expected because it would be good to expose the option as well. The way I was thinking of doing it would hardcode it. | 17:40:08 |
| Genghiz | In reply to @genghiz:cdw.go7box.xyz
Would appreciate a merge on these. :) Both related to firefly-iii | 17:46:16 |
| Genghiz | Hence the ping in this channel | 17:46:23 |
| Genghiz changed their display name from genghiz to Catman. | 17:49:53 |
| Genghiz changed their display name from Catman to Genghiz. | 17:50:09 |
| 22 Sep 2024 |
|  patka joined the room. | 13:49:24 |
| patka | 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 | 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 | * 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 | * 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 | I'll PR it tonight and share it here :) | 15:17:44 |
| patka | https://github.com/NixOS/nixpkgs/pull/343762 | 17:02:06 |
| Pol | excellent. | 17:14:24 |
| patka | 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 | 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 |
