| 22 Jun 2023 |
 Pol | here we go: | 07:01:29 |
| Pol | version = "8.3.0";
hash = "";
base = (callPackage ./generic.nix (_args // {
inherit version hash;
})).overrideAttrs (oldAttrs: {
src = (fetchurl {
url = "https://downloads.php.net/~eric/php-8.3.0alpha2.tar.xz";
hash = "";
});
});
| 07:01:32 |
| Pol | Got a new comment: https://github.com/php/php-src/issues/11408#issuecomment-1601745012 | 07:02:52 |
| Pol | I'm compiling alpha2 | 07:07:12 |
| Pol | I'll see if fileinfo is building | 07:07:38 |
| Pol | But is still there in alpha2 | 07:08:29 |
| Pol | Trying the proposed patch. | 07:08:34 |
| Pol | Nope, same issue | 07:16:12 |
| Pol | version = "8.3.0";
hash = "";
base = (callPackage ./generic.nix (_args // {
inherit version hash;
})).overrideAttrs (oldAttrs: {
src = (fetchurl {
url = "https://downloads.php.net/~eric/php-8.3.0alpha2.tar.xz";
hash = "sha256-YLCxgiDcsBisOmAodf0h8HyaCIh+4i1Q7QZw/h4KR5I=";
});
patches = [
./fileinfo.patch
];
});
| 07:16:32 |
| Pol | tgerbet: I need to leave, I'll be back in the afternoon | 07:31:34 |
| Pol | I just posted this comment: https://github.com/php/php-src/issues/11408#issuecomment-1602150492 | 07:31:48 |
| Pol | We'll see... | 07:31:52 |
| Pol | Ok I got it working | 09:31:01 |
| Pol | Going to post a PR in a few minutes. | 09:31:07 |
| Pol | Ready for review: https://github.com/NixOS/nixpkgs/pull/239151 | 10:15:35 |
 tgerbet | I see in your change that I forgot to cleanup some lib.versionAtLeast php.version "8.1" conditions with the PHP 8.0 removal 😓 | 14:44:34 |
| Pol | Ca arrive même aux meilleurs ;)😁 | 15:42:06 |
| Pol | And also, there's some leftovers of php 8.0 in php-probe.nix (blackfire extension). | 17:47:21 |
| Pol | There's an issue with the php-probe extension... | 18:39:18 |
| Pol | https://gist.github.com/GrahamcOfBorg/eb307c2a2e15412c65011c546ceb982d | 18:39:21 |
| Pol | What's the flag to add in the metadata to disable it for php 8.3 ? I used broken but it's not the good one I think. | 18:40:15 |
| Pol | That issue is preventing me from merging PHP 8.3 right now | 18:44:36 |
| tgerbet | Maybe move the blackfire attribute into something optional? Something like
// lib.optionalAttrs (!(lib.versionAtLeast php.version "8.3")) {
blackfire = ...
}
| 19:07:42 |
| Pol | Where ? I haven't seen that | 19:16:18 |
| 23 Jun 2023 |
| Pol | PHP 8.3 PR is ready for your review/merge at: https://github.com/NixOS/nixpkgs/pull/239151 | 06:57:55 |
| Pol | I'm not satisfied with how the blackfire extension is managed, but I guess this is something I will likely refine very soon | 06:58:51 |
| Pol | Anything left to do in here before I leave ? ^^ | 16:32:11 |
| tgerbet | Too tired after a day of meetings but I will take a look tomorrow 🙂 | 16:38:56 |
| tgerbet | You might want to fetchpatch the patch fixing the build of fileinfo, it just got merged upstream | 16:39:44 |
| Pol | Oh good idea. Doing it right now. | 16:44:04 |
