 | Nix PHP | 76 Members |
| A room for PHP developers running on Nix | 22 Servers |
| Nix PHP | 76 Members |
| A room for PHP developers running on Nix | 22 Servers |
| Sender | Message | Time |
|---|---|---|
| 18 Apr 2023 | ||
 Pol | Ok bug has been fixed in the PR ;) | 11:50:59 |
 philipp | Cool! I'm looking forward to using this tool. | 13:02:07 |
| Pol | PHP PR: https://github.com/NixOS/nixpkgs/pull/226845 | 13:28:48 |
| Pol | I don't remember who told me to not add the platform meta attribute, | 13:29:14 |
 Shyim | In reply to @philipp:xndr.deI 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 !
| 19:31:55 |
| 19 Apr 2023 | ||
| Pol | Nope | 04:15:24 |
| Pol | In reply to @shyim:matrix.orgI 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 |