| 16 Apr 2023 |
Pol | I'm not at ease with that yet | 16:32:14 |
Pol | I'm failing to see the big picture | 16:32:25 |
Pol | Do you mind helping me on that part? | 16:32:35 |
Pol | I can grant you access to the PR if you want to. | 16:32:49 |
Pol |  Download image.png | 16:40:24 |
Pol | Is this is what you have in mind? | 16:40:29 |
Pol | With something like this? | 16:44:53 |
Pol |  Download image.png | 16:44:56 |
Jan Tojnar | Pol: for simplicity, you could have a single composerSetupHook on Nixpkgs level | 17:11:31 |
Jan Tojnar | and then have configure, build and install functions that would replace configurePhase, buildPhase and installPhase respectively | 17:12:14 |
Jan Tojnar | plus a second function that would create the vendor variable | 17:13:17 |
Jan Tojnar | * plus a second function that would create the vendor variable in your PR | 17:13:23 |
Pol | right right, I'm trying but I can't get it working. I'll retry another day. Help is welcome on this. | 18:35:52 |
| 17 Apr 2023 |
Pol | PHP Bumps: https://github.com/NixOS/nixpkgs/pull/226605 -> needs review/merge. | 08:37:41 |
| etu changed their display name from etu to Elis 🌱. | 08:52:21 |
tgerbet | Thanks, I will do a nixpkgs-review run on x86_64 later today | 09:35:32 |
| 18 Apr 2023 |
Pol | Installing Drupal with Nix be like: | 10:31:35 |
Pol |  Download image.png | 10:31:38 |
Pol | Same for wordpress, symfony and other apps ! | 10:37:15 |
philipp | If only that worked for magento. | 10:45:49 |
Pol | Let me check | 11:29:58 |
Pol | Let me try :D | 11:30:18 |
Pol |  Download image.png | 11:34:17 |
Pol | Looks like its broken | 11:34:22 |
Pol | philipp: it works | 11:44:05 |
Pol |  Download image.png | 11:44:12 |
Pol | I just need to fix a small issue in build-php.nix and it's ok. | 11:44:27 |
philipp | 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. | 11:45:31 |
Pol | Ok that's another issue. But at least now we have a way to package magento2 :) | 11:48:11 |
Pol | Ok bug has been fixed in the PR ;) | 11:50:59 |