| 5 May 2024 |
Pol | Maybe this is something to do indeed. | 08:06:41 |
Pol | I dont know how I'm going to do that yet in Nix. | 08:06:54 |
@genghiz:talk.go7box.xyz | Just get a base64 hash of the package name | 08:07:05 |
Pol | in the shell? | 08:07:34 |
@genghiz:talk.go7box.xyz | Yup | 08:07:37 |
Pol | I'll see how we can do that. | 08:07:52 |
@genghiz:talk.go7box.xyz | Or of the composer.lock | 08:08:00 |
@genghiz:talk.go7box.xyz | You can just run something like base64 composer.lock | head -c32 to get a prefix | 08:08:55 |
Pol | Wait | 08:10:11 |
Pol | I have something better. | 08:10:15 |
@genghiz:talk.go7box.xyz | I trust you | 08:10:23 |
@genghiz:talk.go7box.xyz | 😄 | 08:10:40 |
Pol | jq -r -c 'try ."content-hash"' < composer.lock | 08:11:06 |
@genghiz:talk.go7box.xyz | Even better. | 08:12:47 |
Pol | Pushed the changes. | 08:13:11 |
Pol | Thanks for the suggestion | 08:13:27 |
@genghiz:talk.go7box.xyz | No worries | 08:13:34 |
@genghiz:talk.go7box.xyz | You might need to add jq as a buildInput or something. If you don't want to do that, you can probably also use md5sum composer.lock | awk '{ print $1 }' | 08:20:13 |
Pol | It's already in there | 08:22:11 |
Pol | Look at the PR :) | 08:24:59 |
Pol | Oops I did a stupid thing. | 08:25:37 |
Pol | Let me fix that. | 08:25:42 |
Pol | Done. | 08:29:29 |
Pol | Now you can test it properly | 08:32:59 |
Pol | By just using buildComposerProjectNext. | 08:33:08 |
| andmuz joined the room. | 11:39:09 |
Pol | I made further improvements | 12:09:47 |
Pol | Definitely much better than before, and also faster. | 12:09:59 |
Pol | Jan Tojnar: I implemented the changes you suggested from https://github.com/NixOS/nixpkgs/pull/308608 in https://github.com/NixOS/nixpkgs/pull/308059 In the last commit, I'm trying to make the upgrade of psysh, but I end up in a infinite recursion. I've been spending the last hour trying to figure out workarounds, but I'm running out of ideas, do you think you could have a look? | 12:23:11 |
Pol | The overriding part is:
composerRepository = finalAttrs.composerRepository.overrideAttrs (oldAttrs: {
preBuild = ''
setComposeRootVersion
composer config platform.php 7.4
composer require --no-update symfony/polyfill-iconv symfony/polyfill-mbstring
composer require --no-update --dev roave/security-advisories:dev-latest
'';
});
| 14:38:07 |