| 19 May 2023 |
tgerbet | In reply to @jtojnar:matrix.org or we could just optimize the fetcher so that downloading dev dependencies is not pain For production env you do not want the dev deps. Some might do things that are fine in dev but sketchy for production if you have a mistake in your configuration. | 21:36:26 |
tgerbet | The famous eval-stdin.php from PHPUnit comes to mind | 21:36:47 |
tgerbet | * The famous eval-stdin.php from PHPUnit comes to my mind | 21:36:55 |
Pol | yeah... true. | 21:37:42 |
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 |