14 Aug 2024 |
piotrkwiecinski | if I'm not mistaken 8.4 is scheduled for the next staging-next merge | 13:22:33 |
piotrkwiecinski | so we'll have to wait a bit till I can update it to beta1 | 13:22:51 |
piotrkwiecinski | I'll have update prepared | 13:23:15 |
Pol | In reply to @piotrkwiecinski:matrix.org if I'm not mistaken 8.4 is scheduled for the next staging-next merge it has been merged this morning, so it's coming very soon. | 13:24:14 |
piotrkwiecinski | Is this correct syntax ? | 13:27:59 |
piotrkwiecinski | patches = lib.optional (lib.versionOlder version "8.4.0beta1") [
./fix-paths-php7.patch
] ++ lib.optional (lib.versionAtLeast version "8.4.0beta1") [
./fix-paths-php84.patch
] ++ extraPatches;
| 13:28:00 |
piotrkwiecinski | if this works as expected i can PR it earlier | 13:28:50 |
piotrkwiecinski | ok seems to build | 13:31:37 |
Pol | no | 13:31:40 |
Pol | By using lib.optional <predicate> [ ] , you're wrapping twice. | 13:32:04 |
Pol | In this case, you have to use lib.optionals <predicate> [ ] | 13:32:20 |
Pol | (hense the s ) | 13:32:28 |
piotrkwiecinski | somehow it built without s too | 13:34:28 |
piotrkwiecinski | and applied the patch | 13:34:40 |
piotrkwiecinski | on I changed it to s | 13:35:53 |
piotrkwiecinski | so in theory it shouldn't break current 8.4.0alpha1 | 13:36:21 |
Pol | We can already use beta1 I think | 13:40:12 |
piotrkwiecinski | we have alpha | 13:40:24 |
Pol | beta1 : https://downloads.php.net/~calvinb/ | 13:40:45 |
piotrkwiecinski | alpha4 | 13:40:47 |
Pol | alpha4 and beta1 are available | 13:40:55 |
piotrkwiecinski | in our repo i mean | 13:40:58 |
Pol | ah ok | 13:41:02 |
Pol | #324831 php84: init at 8.4.0alpha1 ✅ staging-next ✅ master ✅ nixos-unstable-small ⏳nixos-unstable ⏳nixpkgs-unstable | 13:42:41 |
Pol | It will be soon available on nixos-unstable! | 13:42:54 |
piotrkwiecinski | it's in master so I can PR update to beta | 13:43:56 |
Pol | Yes | 13:44:54 |
piotrkwiecinski | Could you merge https://github.com/NixOS/nixpkgs/pull/334411 in meantime? I may have conflicts in php-packages.nix | 13:48:54 |
Pol | yes | 14:52:32 |
Pol | Done. | 14:55:24 |