| 19 May 2023 |
 Pol | Feel free to hack into it | 21:29:47 |
 tgerbet | In reply to @drupol:matrix.org
Maybe we can just skip the package if the package is not in require section. Not exactly because you need to know the whole tree to determine if a transitive dep is needed for a prod dependency or not | 21:30:12 |
| Pol | I'm now going to bed, I've worked in my house the whole day, the baseboards are almost done, I need to get some sleep | 21:30:25 |
| Pol | In reply to @tgerbet:matrix.org
Not exactly because you need to know the whole tree to determine if a transitive dep is needed for a prod dependency or not Indeed, I should have removed my answer, I wrote too quickly. | 21:30:48 |
 Jan Tojnar | I think I might have only used source in composition-c4 because I though that builtins.fetchGit does not require output hash but it does currently | 21:31:08 |
| 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 |
