21 Aug 2024 |
piotrkwiecinski | so we have to sort the array after it's generated | 19:18:04 |
Pol | https://github.com/composer/composer/blob/main/src/Composer/Autoload/AutoloadGenerator.php#L735C15-L735C35 | 19:19:03 |
Pol | Oh wait. | 19:19:15 |
Pol | you're right. sorry. | 19:19:26 |
Pol | I guess it's a missing sort somewhere before that. | 19:20:01 |
22 Aug 2024 |
Pol | piotrkwiecinski: I cannot reproduce the issue with the patch you provided. | 04:33:45 |
Pol | I get a stable hash for phpdocumentor with it: sha256-LFrkJaFfMzNWOgeUXrNLLURpMt0eNqCUHcTYnIcQJqk= | 04:34:17 |
Pol | OK, got a working patch, plus a refactoring, plus the tests updated. | 05:21:27 |
Pol | Let me know if this is OK for you: https://github.com/composer/composer/pull/12090 | 05:29:02 |
Pol | Allrighty, I won't be able to take care of this today, I have to switch to something else now. | 05:46:22 |
Pol | I'll check here from time to time without being able to produce code if needed. | 05:46:40 |
piotrkwiecinski | Thank you Pol for picking it up. | 06:46:19 |
Pol | You did the job mate... I just created the PR to fix that issue ASAP. | 06:46:37 |
Pol | piotrkwiecinski: https://github.com/composer/composer/pull/12090#discussion_r1726756464 | 10:16:38 |
piotrkwiecinski | I saw the comment | 10:18:57 |
piotrkwiecinski | $psr0 = $this->parseAutoloadsType($packageMap, 'psr-0', $rootPackage);
$psr4 = $this->parseAutoloadsType($packageMap, 'psr-4', $rootPackage);
$classmap = $this->parseAutoloadsType(array_reverse($sortedPackageMap), 'classmap', $rootPackage);
$files = $this->parseAutoloadsType($sortedPackageMap, 'files', $rootPackage);
$exclude = $this->parseAutoloadsType($sortedPackageMap, 'exclude-from-classmap', $rootPackage);
| 10:18:59 |
piotrkwiecinski | psr4 doesn't seem to use sorted map | 10:19:16 |
piotrkwiecinski | maybe this is the problem | 10:19:56 |
piotrkwiecinski | I won't be able to test it for a couple hours | 10:20:08 |
piotrkwiecinski | I have the last day before my annual leave and have to wrap up few things at work | 10:20:38 |
piotrkwiecinski | https://github.com/composer/composer/blob/main/src/Composer/Autoload/AutoloadGenerator.php#L569-L582 | 10:22:33 |
piotrkwiecinski | it's the block to debug | 10:23:08 |
24 Aug 2024 |
| Rogier Knoester joined the room. | 15:25:03 |
27 Aug 2024 |
@genghiz:talk.go7box.xyz | Hey guys, can someone have a look at this PHP PR? Packages firefly-iii data importer https://github.com/NixOS/nixpkgs/pull/331823 | 19:42:47 |
28 Aug 2024 |
fsagbuya | I also would like to follow up on this PR: https://github.com/NixOS/nixpkgs/pull/336044. Pol | 02:48:41 |
2 Sep 2024 |
Pol | piotrkwiecinski: Did you had the time to revisit the composer issue ? | 17:25:08 |
Pol | On my side, big nope :( | 17:25:13 |
6 Sep 2024 |
piotrkwiecinski | Pol: I just finished my vacations. Didn't have access to my laptop for almost 2 weeks. | 16:54:58 |
7 Sep 2024 |
Pol | In reply to @piotrkwiecinski:matrix.org Pol: I just finished my vacations. Didn't have access to my laptop for almost 2 weeks. Lucky you | 09:04:28 |
13 Sep 2024 |
| @genghiz:talk.go7box.xyz left the room. | 15:29:43 |