22 Sep 2024 |
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 |
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 | 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 | Oopsie, thanks! | 07:23:59 |
patka | 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 | * 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 |
Pol | Quick question, how do you usually test PR ? Do you cherry pick the commit in your branch or do another trick ? | 07:32:17 |
Genghiz | In reply to @drupol:matrix.org Quick question, how do you usually test PR ? Do you cherry pick the commit in your branch or do another trick ? I use nixpkgs-review | 09:37:35 |
Pol | But in this case, you need to test this against fireflii, how do you do then? | 09:37:58 |
Genghiz | Nixpkgs-review pr -p firefly-iii
| 09:38:22 |
Pol | nice | 09:39:29 |