| 18 Feb 2024 |
Pol | Created an issue: https://github.com/nikstur/bombon/issues/71 | 10:39:45 |
Pol | patka: do you plan to update this PR https://github.com/NixOS/nixpkgs/pull/289625#event-11837388469 ? | 10:43:11 |
@patka_123:matrix.org | Oh, thought I already did that. That was what the 👍 was for. But probably didn't push. Give me a sec | 10:44:22 |
@patka_123:matrix.org | Done | 10:45:31 |
Pol | Merci :) | 10:45:50 |
Pol | Merged! | 10:46:27 |
| 19 Feb 2024 |
Pol | youhou ! | 09:19:05 |
Pol | https://github.com/NixOS/nixpkgs/pull/285828 | 09:24:30 |
Pol | I added the required logic to detect if a composer update needs to be triggered. Now, thanks to that, we can install correctly php-parallel-lint ! | 09:25:03 |
Pol | Elis 🌱: Now it's ready for review :D | 09:25:13 |
Pol | Why Psalm need COMPOSER_ROOT_VERSION : https://github.com/vimeo/psalm/issues/10721 | 20:59:09 |
Pol | In Nix, when building a PHP derivation, the ``.gitfolder doesn't exist, therefore Composer is not able to infer the version... and it fails. SettingCOMPOSER_ROOT_VERSION` fix that. | 21:06:07 |
Pol | * In Nix, when building a PHP derivation, the .git folder doesn't exist, therefore Composer is not able to infer the version... and it fails. SettingCOMPOSER\_ROOT\_VERSION fix that. | 21:06:23 |
Pol | * In Nix, when building a PHP derivation, the .git folder doesn't exist, therefore Composer is not able to infer the version... and it fails. SettingCOMPOSER_ROOT_VERSION fix that. | 21:06:30 |
Pol | patka, tgerbet , Gaël Reyrol : I'm looking for reviewers on this: https://github.com/NixOS/nixpkgs/pull/285828 | 21:37:28 |
@patka_123:matrix.org | I'll just be honest, I probably don't have the means to review this properly the coming few days | 22:21:52 |
| 20 Feb 2024 |
Pol | Totally fine, it's just open-source, we do what we can do at our best effort :) no worries! Hope everything's fine though. | 10:33:09 |
Pol | Oh damn... I've broken my pants in https://github.com/NixOS/nixpkgs/pull/285828 | 11:45:33 |
Pol | I upgraded a couple of derivations and switch to using buildComposerProject wrapper | 11:45:50 |
Pol | I hope it's ok to do that into that PR. | 11:45:57 |
Pol | Once this PR will be merged, there will be only one PHAR in Nix: Composer (only used to bootstrap Composer sources) | 21:59:37 |
Pol | * Once this PR will be merged, there will be only one PHAR in Nixpkgs: Composer (only used to bootstrap Composer sources) | 21:59:46 |
| 21 Feb 2024 |
Pol | I'm still looking for reviewers at https://github.com/NixOS/nixpkgs/pull/285828 ! :) | 19:32:31 |
tgerbet | I will try to take a look but it's unlikely to happen before this week end | 21:18:45 |
Pol | Okidocky | 21:50:08 |
| 23 Feb 2024 |
Pol | Oh yeah ! The Composer bug we have with 2.7.0 has been flagged as a bug by Jordi! | 11:19:36 |
Pol | Link: https://github.com/composer/composer/pull/11850 | 11:19:39 |
Pol | Also, this PR has been merged this morning: https://github.com/composer/composer/pull/11858 | 11:20:00 |
Pol | Issue fixed in Composer ! \o/ https://github.com/composer/composer/pull/11850 | 13:20:11 |
Pol | I'll update the PR when 2.7.2 will be out. | 13:23:55 |