23 Sep 2024 |
Pol | I just notice a small typo | 09:39:35 |
Pol | I will fix it after that PR is merged. | 09:39:42 |
Pol | setComposeRootVersion -> setComposerRootVersion | 09:40:01 |
Pol | setComposeEnvVariables -> setComposerEnvVariables | 09:40:12 |
Pol | Let me do it within this PR | 09:40:23 |
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 |