| 9 Oct 2023 |
 Pol | But this is building a docker image. I guess you won't have trouble transposing that for NixOS. | 20:19:33 |
 w | I'll read it | 21:16:48 |
| 16 Oct 2023 |
| Pol | PR To review/merge:
- https://github.com/NixOS/nixpkgs/pull/261175
| 11:30:41 |
| Pol | PR to review/merge:
- https://github.com/NixOS/nixpkgs/pull/261175
- https://github.com/NixOS/nixpkgs/pull/261429
- https://github.com/NixOS/nixpkgs/pull/261413
| 15:43:09 |
| 17 Oct 2023 |
| Pol | PR to review:
- https://github.com/NixOS/nixpkgs/pull/261582
| 16:18:13 |
| 20 Oct 2023 |
|  ramsey left the room. | 16:39:00 |
| Pol | PR to review:
- https://github.com/NixOS/nixpkgs/pull/262388
| 21:55:30 |
| 23 Oct 2023 |
|  globin joined the room. | 10:00:21 |
|  globin left the room. | 10:00:27 |
| globin set a profile picture. | 14:27:51 |
| Pol | Looking for feedback and/or thumbs up here: https://github.com/bobthecow/psysh/issues/767 | 19:08:03 |
| 29 Oct 2023 |
| Pol | I made a presentation about Nix and PHP in Munich 2 days ago, find the slides here: https://github.com/drupol/ipc2023/releases/tag/v19-ea1ffb1 | 10:20:01 |
| 2 Nov 2023 |
 philipp | Somebody here made a composer builder that relied on a composer.lock file, instead of a vendorHash, right? I can't seem to find it, so I'm happy for any pointers. | 12:01:45 |
| philipp | Ah, this is the one: https://github.com/fossar/composition-c4 | 12:04:43 |
| 3 Nov 2023 |
| Pol | Some with a Mac could confirm this bug ? https://github.com/fossar/nix-phps/issues/305 Just run nix shell github:fossar/nix-phps#php73 to confirm or infirm please! | 08:03:43 |
| Pol | In reply to @philipp:xndr.de
Somebody here made a composer builder that relied on a composer.lock file, instead of a vendorHash, right? I can't seem to find it, so I'm happy for any pointers. The new Composer/Builder in nixpkgs doesn't work for you? Happy to hear the story! | 08:07:28 |
| philipp | In reply to @drupol:matrix.org
The new Composer/Builder in nixpkgs doesn't work for you? Happy to hear the story! Need to work it with old php versions for a few weeks, kind of a bootstrapping problem. The new builder will hopefully be used next month. | 08:10:41 |
| Pol | Okidocky | 08:11:41 |
| Pol | Glad to hear :) | 08:11:45 |
| philipp | Really looking forward to use the version in nixpkgs, that should finally enable us to have not god-awful deployment methods. | 08:24:42 |
| Pol | Yes that was something awaited for a long time | 09:47:36 |
| Pol | I grow more white hairs while doing it as well :D | 09:47:58 |
| 4 Nov 2023 |
 Gaël Reyrol | Hey Pol , I've ran into a surprising issue with buildPhpFromComposer when playing with PHP ini option ffi.preload using .user.ini. It seems if I set it to a local file, the complete chain of derivations fails when building composer from it because the local file is not included in the derivations such as composer-local-repo-plugin. The built PHP derivation has the option but the file is not included in the derivation. When thinking about it is completely logic but it is not trivial 😅 | 14:46:36 |
| Gaël Reyrol | I've set up a test repository and a more detailed issue here: https://github.com/gaelreyrol/opentelemetry-php-nix/issues/1 | 14:47:15 |
| Pol | In reply to @Zevran:matrix.org
I've set up a test repository and a more detailed issue here: https://github.com/gaelreyrol/opentelemetry-php-nix/issues/1 I hope to find time to test this. if you find the issue in the meantime, let me know! | 23:08:59 |
| Gaël Reyrol | Don't worry it is not urgent! When I think about it, I don't see an easy way of fixing it :/ | 23:16:01 |
| 6 Nov 2023 |
| philipp | Pol: Since you are looking for feedback and I started trying to use the new composer build environment in in nixpkgs: The docs are somewhat unclear on whether a vendorHash is always required and why that is, I think. | 08:45:35 |
| Pol | OK, that means there's room for improvements. The vendorHash is required just like in other builder (rust, nodejs, etc etc). | 08:46:47 |
| Pol | The vendorHash correspond to the vendor directory hash that Composer builds for the PHP app. | 08:47:09 |
| Pol | It is always required. | 08:49:47 |
