13 Aug 2024 |
Pol | haha nice ! :D | 18:22:43 |
Pol | I will reuse it :D | 18:22:48 |
piotrkwiecinski | I updated comments in https://github.com/NixOS/nixpkgs/pull/334411 | 18:22:57 |
Pol | I have a question for you with these patches. | 18:23:43 |
Pol | you know I'm maintaining that thingie here: https://github.com/loophp/php-src-nix/ | 18:24:02 |
Pol | Basically this project automatically maintain itself. I like it. | 18:24:13 |
Pol | However, I noticed sometimes that when we set lower bounds for patches in nixpkgs , it sometimes makes issue in my project. | 18:24:37 |
Pol | So I was wondering if you correctly set the version boundaries so it can be cleanly applied to the PHP versions we maintain in nixos/nixpkgs but also in my project. | 18:25:15 |
piotrkwiecinski | My fix should apply to all php8.1 versions | 18:25:50 |
Pol | As you can see, it's quite broken: https://github.com/loophp/php-src-nix/actions/runs/10364035720 | 18:25:54 |
Pol | Oh damn, I just notice now that we will have an issue with the next PHP84 version... https://github.com/loophp/php-src-nix/actions/runs/10364035720/job/28688711194 | 18:26:47 |
Pol | In reply to @piotrkwiecinski:matrix.org My fix should apply to all php8.1 versions Cool, I validate your PR then. | 18:27:06 |
Pol | Done. | 18:27:33 |
piotrkwiecinski | also is it ok to go to master? | 18:28:30 |
piotrkwiecinski | it has fair bit of rebuilds | 18:28:36 |
Pol | yes master is fine. | 18:29:01 |
piotrkwiecinski | > calling 'postUnpack' function hook '_updateSourceDateEpochFromSourceRoot'
> Running phase: patchPhase
> applying patch /nix/store/r2bw91ma55jqnlv51iwck17f05i1c76x-fix-paths-php7.patch
> patching file ext/gettext/config.m4
| 18:29:07 |
piotrkwiecinski | this one is in generic.nix | 18:29:16 |
Pol | yes, since quite a while | 18:30:15 |
piotrkwiecinski | https://github.com/php/php-src/commit/c2d6edc4854b036a94e28b495ccbc1b652649b02 | 18:57:21 |
piotrkwiecinski | I think it's the cause | 18:57:26 |
piotrkwiecinski | for the gettext failure | 18:57:48 |
piotrkwiecinski | we'll have to prepare a new patch for 8.4 | 18:58:07 |
piotrkwiecinski | alpha4 was released 2 weeks ago and doesn't have this code yet | 19:08:58 |
piotrkwiecinski | but beta1 was tagged today | 19:09:12 |
piotrkwiecinski | I'll see if I can prepare the patch for gettext later this week | 19:09:35 |
14 Aug 2024 |
piotrkwiecinski | damn patches :D | 13:16:03 |
piotrkwiecinski | I have a fix on local for gettext but ext-dom fails with beta1 build
> Running phase: patchPhase
> applying patch /nix/store/hqrpdj07y97vwiyy29jizcsmvx9hdnys-8d7365b6f009ba43e305d6459013ac4fbed7c606.diff?full_index=1
> patching file ext/dom/tests/modern/extensions/Element_substitutedNodeValue.phpt
> Reversed (or previously applied) patch detected! Assume -R? [n]
> Apply anyway? [n]
> Skipping patch.
> 2 out of 2 hunks ignored -- saving rejects to file ext/dom/tests/modern/extensions/Element_substitutedNodeValue.phpt.rej
| 13:17:05 |
piotrkwiecinski | maybe its because try to apply a patch to for alpha1 | 13:18:59 |
Pol | #facepalm | 13:21:24 |