| 18 Apr 2023 |
Shyim | In reply to @philipp:xndr.de I'll give it a real-world spin but I'm not optimistic. magento will probably start to try writing into a file in the nix store and die. I work for Shopware and we recently started with read-only deployments with Docker. There are so many even composer packages which writes to disk like pdf libraries. I guess read-only will be a thing in future with PHP. Symfony recommends Platform.SH and they do read-only deployments so maybe we have in future every application out of the box compatible as those Hosters require it :) | 15:55:20 |
Shyim | Did someone tried to package Frankenphp with Nix? I don’t really understand the stuff with CGO and PHP dev :D | 15:56:05 |
Pol | It's raining PHP PRs !
- https://github.com/NixOS/nixpkgs/pull/226911
- https://github.com/NixOS/nixpkgs/pull/226908
- https://github.com/NixOS/nixpkgs/pull/226824
| 19:31:55 |
| 19 Apr 2023 |
Pol | Nope | 04:15:24 |
Pol | In reply to @shyim:matrix.org I work for Shopware and we recently started with read-only deployments with Docker. There are so many even composer packages which writes to disk like pdf libraries. I guess read-only will be a thing in future with PHP. Symfony recommends Platform.SH and they do read-only deployments so maybe we have in future every application out of the box compatible as those Hosters require it :) I will try to package Shopware with the new builder once it's done ! Looking forward to push this thing! | 04:15:51 |
Pol | Elis 🌱: Last one! https://github.com/NixOS/nixpkgs/pull/226824 | 07:53:08 |
Pol | Elis 🌱: https://github.com/NixOS/nixpkgs/pull/227066#pullrequestreview-1392278346 | 15:19:50 |
Pol | Merci ❤️ | 15:33:08 |
Shyim | I packaged Roadrunner for Nix. Looking for reviewers :) https://github.com/NixOS/nixpkgs/pull/227144 | 21:41:57 |
| 21 Apr 2023 |
Pol | Hey PHP ! Yesterday I finally made the changes in this PR https://github.com/NixOS/nixpkgs/pull/225401 to enable hooks. That was quite complicated for me since I was not aware of those hooks in Nix overall, so, a lot of back and forth with some community members to tell me where to look for. The PR has been updated with those hooks now and it works like before. There is only one derivation which is broken "#wp-cli". Apparently, it requires wrapProgram in its postInstall hook, but it cannot find it despite it is in nativeBuildInputs. I guess it's a minor issue, but I'll make a pause today on that thing because I've been spending too much time on it lately. Feel free to try the branch and eventually help me with that wp-cli derivation! Enjoy your friday all ! | 07:25:39 |
Pol | Thanks Jan Tojnar going to try your suggestion ! | 07:32:09 |
Pol | Thanks, it fixed the issue :) | 07:39:47 |
Pol | You're amazing, thanks ! | 07:39:51 |
Pol | PR is back to green ! :) | 07:52:29 |
Jan Tojnar | Pol: you will also want to do previousAttrs.buildPhase or ''runHook ...'' for the phases | 07:54:01 |
Pol | Oooh ! you're right. | 07:54:16 |
Pol | As usual. | 07:54:17 |
Pol | Going to do it now. | 07:54:20 |
Jan Tojnar | the empty phases are a bit ugly but I do not see a way around them | 07:55:01 |
Pol | We could override them in the composer-setup-hook.sh I guess? | 07:55:26 |
Pol | Instead of hooking on preConfigurePhase, we override the phase itself ? | 07:55:46 |
Jan Tojnar | yes, but then if people pass configurePhase to buildPhpProject, they will lose the setupHook | 07:56:31 |
Jan Tojnar | * yes, but then if people pass configurePhase to buildPhpProject, they will lose the setupHook functionality | 07:56:46 |
Jan Tojnar | unless they call it explicitly | 07:57:02 |
Jan Tojnar | which maybe would not be a problem if documented. IDK, we are breaking a new ground so the usage patterns are not developed yet. | 07:57:53 |
Jan Tojnar | * which maybe would not be a problem if documented. IDK, we are breaking a new ground so the usage patterns are not well developed yet. | 07:58:11 |
Pol | Job done. Fixed. | 07:59:35 |
Jan Tojnar | and also strictDeps`` | 08:05:24 |
Jan Tojnar | * and also strictDeps | 08:05:30 |
Jan Tojnar | it really shows that mkDerivation was not designed with this in mind | 08:06:28 |