| 13 Aug 2024 |
 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 |
| 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 |
