| 19 May 2023 |
Jan Tojnar | tgerbet: but composer install already defaults to --dev so you will need to run composer install --no-dev anyway | 21:38:33 |
tgerbet | Hum yes indeed 👍️ | 21:39:12 |
Pol | Excellent Jan Tojnar | 22:32:00 |
Pol | Tomorrow morning I will update everything | 22:32:19 |
Jan Tojnar | tried optimizing the fetching in composition-c4 by fetching just a single commit (which Git supposedly allows) but Nix does not allow fetching commit refs – it will assume the ref is a head if it does not start with refs/ | 22:36:03 |
Pol | I have an idea. | 22:41:02 |
Pol | Oh no, forget it. | 22:41:11 |
Pol | I went downstairs, I wanted to try what you did on nixpkgs :) | 22:43:30 |
Pol | And it works. | 22:44:44 |
Pol | wow. | 22:44:46 |
Pol | I think there would be a way to avoid downloading all the -dev stuff. | 23:00:37 |
Pol | The idea would be to create a wrapping composer.json that would require the package we want to build. A custom repo would also be added in that composer.json, pointing to the path of the package source. | 23:01:34 |
Pol | I will try that idea at some point. | 23:01:41 |
Pol | Redacted or Malformed Event | 23:15:24 |
Pol | Building composer and phpunit works in the branch: https://github.com/NixOS/nixpkgs/pull/232450 | 23:22:25 |
Pol | Thanks Jan Tojnar ! | 23:22:44 |
Pol | Sadly, it won't be possible to package psalm :( | 23:51:41 |
Pol |  Download image.png | 23:51:45 |
| 20 May 2023 |
Jan Tojnar | Gaël Reyrol: I have updated the Python script to try to use if for impure derivations https://github.com/fossar/composition-c4/pull/5 | 01:45:20 |
Pol | I updated the drupol/composer-local-repo-plugin: https://github.com/drupol/composer-local-repo-plugin/ | 07:52:21 |
Pol | Jan Tojnar: Ideally we should not have derivation like phpstan/phpstan in the store when installing composr. | 14:06:58 |
Pol | * Jan Tojnar: Ideally we should not have derivation like phpstan/phpstan in the store when installing composer. | 14:07:01 |
Pol | How can we prevent that? | 14:07:06 |
Pol | (thanks for replying to Jordi btw!) | 14:07:27 |
Jan Tojnar | Pol: why not? | 14:07:30 |
Pol | Because
- It's polluting the store (not eco-friendly)
- It's absolutely not needed in Composer
| 14:08:09 |
Jan Tojnar | well, I guess we could return includeDev flag but only use it to decide what to fetch but still include everything in the repository’s packages.json | 14:09:25 |
Jan Tojnar | then the paths for dev packages would not exist but it would not matter | 14:10:11 |
Jan Tojnar | * then the paths for dev packages would not exist but it would not matter as long as user always uses --no-dev | 14:10:22 |
Pol | I don't know if this would work. | 14:12:42 |