| 23 Sep 2024 |
 Genghiz | So firefly-iii seems to be fixed. | 09:41:06 |
 Pol | Done, I just fixed the typo | 09:41:28 |
| Genghiz | Once you merge I’ll close the firefly-iii fix PR. | 09:41:29 |
| Pol | Can you try it again? | 09:41:40 |
| Pol | And post some feedback in the PR ? | 09:41:45 |
| Genghiz | Same PR number? | 09:41:48 |
| Pol | yeap | 09:41:54 |
| Genghiz | 2 mins | 09:42:10 |
| Genghiz | Fwiw the tests work, but I’ll add a properly formatted review message in a bit so just merge post that I think. | 10:21:19 |
| Genghiz | This is a backwards compatible change so I would recommend a backport as well. | 10:21:35 |
| Genghiz | In reply to @genghiz:cdw.go7box.xyz
Fwiw the tests work, but I’ll add a properly formatted review message in a bit so just merge post that I think. Done. | 10:35:26 |
| Pol | Merged ~! | 10:59:16 |
| Pol | * Merged ! | 10:59:17 |
 patka | What about the comment I made? | 11:02:52 |
| Pol | Which comment? Did I missed something? | 11:03:55 |
| patka | I made a comment on the PR, but it still says pending without a submit button so dont know what went wrong. I'm on mobile in the train now so just
But v1 had COMPOSER_DISABLE_NETWORK = "1"; and I think you forgot to add that in your PR? Or was it purposefully left out? | 11:10:51 |
| Pol | Left out on purpose ;) | 11:11:43 |
| Pol | No worries | 11:11:44 |
| Genghiz | In reply to @genghiz:cdw.go7box.xyz
https://github.com/NixOS/nixpkgs/pull/341746 @drupol:matrix.orgHave changed this to not set the variable. It’s a version bump rn. Do have a look.
| 11:47:51 |
| Pol | So we can now safely say that buildComposerProject2 is stable :) | 12:53:13 |
| patka | Awesome! | 13:03:56 |
| patka | I'll slowly start updating the packages that already use it (and other packages) this week then.
But I don't need to be in this channel then. Still trying to limit input and thus as much Matrix rooms as possible :) | 13:15:30 |
| patka left the room. | 13:15:34 |
 hexa | dear php maintainers, is there any requirement for the phpfpm service to run as root? | 15:34:39 |
| hexa | it seems to fork workers that run as specific users | 15:43:39 |
| hexa | just wondering why the fpm needs to run as root | 15:43:46 |
| Genghiz | It doesn’t. | 15:44:35 |
| Genghiz | I’ve run it as a specific user for a long time on Debian. | 15:44:45 |
| Genghiz | It also radically simplifies the systemd service because you can comfortably enable more lockdown options. | 15:45:23 |
| Genghiz | Never done it on nixos because it doesn’t make a huge difference. | 15:45:40 |
