!jngDrdMgndWibPCYsR:nixos.org

Nix PHP

74 Members
A room for PHP developers running on Nix22 Servers

Load older messages


SenderMessageTime
16 May 2024
@drupol:matrix.orgPolHad to work on this yesterday and today at work.12:37:23
@drupol:matrix.orgPolI finally came up with this interface: https://github.com/NixOS/nixpkgs/pull/308059/commits/29d9c8a602875fe096eb35fc60dd1b1ece510e8d12:37:32
@drupol:matrix.orgPolThis works, but I don't know if it's the best way to do that.12:37:53
@drupol:matrix.orgPolFeel free to hack and participate in the PR at https://github.com/NixOS/nixpkgs/pull/30805912:38:06
@drupol:matrix.orgPolThanks to this, I can fix some bad design from version 1: https://github.com/NixOS/nixpkgs/pull/308059/commits/133d96cb6a70582d65ae8161906c4c4823457bd113:16:38
@drupol:matrix.orgPolI will most probably split that PR in multiple PR, trying to backport the good thing before introducing the version 2.13:16:57
@drupol:matrix.orgPolA lot of work ahead to avoid breaking everything.13:17:04
@drupol:matrix.orgPolAny small help is very welcome.13:17:11
@drupol:matrix.orgPolHere's some backport for the builder version 1: https://github.com/NixOS/nixpkgs/pull/31223115:07:28
@drupol:matrix.orgPolI think I'm done with that PR.16:14:42
@drupol:matrix.orgPol 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
@drupol:matrix.orgPolRedacted or Malformed Event16:16:52
17 May 2024
@drupol:matrix.orgPol 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
@drupol:matrix.orgPolGiven 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_123:matrix.org@patka_123:matrix.orgI will gladly review it, but I currently don't have the bandwith. And you probably want someone more experienced as well09:43:04
@drupol:matrix.orgPolNo worries it's fine10:07:11
@piotrkwiecinski:matrix.orgpiotrkwiecinskiHi 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
@drupol:matrix.orgPolv2 is not going to be out "soon".12:13:37
@drupol:matrix.orgPolThe 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:matrix.orgpiotrkwiecinskiI 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_123:matrix.org@patka_123:matrix.orgThere are not many. If you want you could go through the list to find out if something needs to be updated?12:21:54
@drupol:matrix.orgPolI would not advise to update hashes now. 12:28:27
@drupol:matrix.orgPolI would wait a bit I stabilize the new V2 builder.12:28:39
@drupol:matrix.orgPolIf you really would like to help is by reviewing PR in the proper order because I can only progress from there.12:29:07
@drupol:matrix.orgPol * I would not advise to update hashes now or use the v2 of the builder now.12:31:03
@drupol:matrix.orgPolI would advise reviewers to review one commit at a time: https://github.com/NixOS/nixpkgs/pull/312231/commits12:49:37
20 May 2024
@genghiz:talk.go7box.xyz@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
@genghiz:talk.go7box.xyz@genghiz:talk.go7box.xyz * Pol: I've updated the firefly-iii module and updated the package 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:15
@drupol:matrix.orgPolcool stuff !12:29:13
@drupol:matrix.orgPol catman: Would you be encline to help me packaging a Drupal service? 12:29:31

Show newer messages


Back to Room ListRoom Version: 6