16 May 2024 |
Pol | Here's some backport for the builder version 1: https://github.com/NixOS/nixpkgs/pull/312231 | 15:07:28 |
Pol | I think I'm done with that PR. | 16:14:42 |
Pol | I'm happy enough... until someone will make a review and will kill my modjo :D looking at you Jan Tojnar ! :D :D :D | 16:15:13 |
Pol | Redacted or Malformed Event | 16:16:52 |
17 May 2024 |
Pol | This morning I rewrote the PR https://github.com/NixOS/nixpkgs/pull/312231, the new builder look like the existing builder now, with finalAttrs override stuff. Everything works great, I'm extremelly satisfied of the result. | 08:38:46 |
Pol | Given the amount of work in this PR, I'll merge it in one week if I don't have feedback. The easiest way to review it is going commit per commit. | 08:39:17 |
patka (new account: @patka:envs.net) | I will gladly review it, but I currently don't have the bandwith. And you probably want someone more experienced as well | 09:43:04 |
Pol | No worries it's fine | 10:07:11 |
piotrkwiecinski | Hi everyone. When https://github.com/NixOS/nixpkgs/pull/308059 is ready feel free to ping me and I'm happy to help with migrating php package to v2 builder, so you can focus on more tasks. | 12:11:19 |
Pol | v2 is not going to be out "soon". | 12:13:37 |
Pol | The PR https://github.com/NixOS/nixpkgs/pull/312231 needs to comes first, then I'll rebase the PR for the V2 builder on it. | 12:13:59 |
piotrkwiecinski | I saw it as well but I'm not experienced enough to review it. I can help with reviewing PHP package updates if that helps. | 12:18:38 |
patka (new account: @patka:envs.net) | There are not many. If you want you could go through the list to find out if something needs to be updated? | 12:21:54 |
Pol | I would not advise to update hashes now. | 12:28:27 |
Pol | I would wait a bit I stabilize the new V2 builder. | 12:28:39 |
Pol | If you really would like to help is by reviewing PR in the proper order because I can only progress from there. | 12:29:07 |
Pol | * I would not advise to update hashes now or use the v2 of the builder now. | 12:31:03 |
Pol | I would advise reviewers to review one commit at a time: https://github.com/NixOS/nixpkgs/pull/312231/commits | 12:49:37 |
20 May 2024 |
@genghiz:talk.go7box.xyz | Pol: I've updated the firefly-iii module and updated it to version 6.1.16. I think it's a minor enough update for it to wait until the release and then get backported. | 12:18:08 |