Sender | Message | Time |
---|---|---|
17 Aug 2024 | ||
Pol | submit the PR against loophp/php-src-nix | 08:52:07 |
Pol | If you want I can make you maintainer too | 08:52:15 |
piotrkwiecinski | I'll PR it against loophp once it's ready. If you don't mind me being a maintainer. I'm happy to help. | 08:55:20 |
piotrkwiecinski | I'm going to use php for at least a couple years | 08:55:53 |
piotrkwiecinski | I start to think maybe we should rework patches in nixpkgs | 09:44:03 |
piotrkwiecinski | Overlay relies on https://github.com/piotrkwiecinski/php-src-nix/blob/main/src/makePhpPackage.nix#L13 | 09:45:15 |
piotrkwiecinski | this means version specific patches from nixpkgs aren't applied | 09:45:36 |
piotrkwiecinski | we would have to move https://github.com/piotrkwiecinski/nixpkgs/blob/fa088a9477b35d8e36c8204c35fc574eaffe0190/pkgs/development/interpreters/php/8.1.nix#L7-L31 to generic.nix | 09:46:12 |
piotrkwiecinski | it would make backporting easier | 09:46:28 |
Pol | yeah there are some improvements to be made. | 18:09:57 |
Pol | Just added you as maintainer | 18:10:03 |
piotrkwiecinski | Got it. Thank you. I have 59 builds passing. Still 49 to go. It'll probably take at least a couple days. | 18:18:25 |
Pol | there's no hurry | 18:44:07 |
18 Aug 2024 | ||
@genghiz:talk.go7box.xyz | 19:03:37 | |
20 Aug 2024 | ||
fsagbuya joined the room. | 08:58:03 | |
fsagbuya | Hi. Is there a good way to extend a certain package with a
However, I'm not sure if this is the proper way. Is there an extension management system for | 09:05:28 |
fsagbuya | * Hi. Is there a good way to extend a certain package with a
However, I'm not sure if this is the proper way. Is there an extension management system for | 09:05:57 |
fsagbuya | * Hi. Is there a good way to extend a certain package with a
However, I'm not sure if this is the proper way. Is there an extension management system for | 09:06:13 |
Pol | I think this is the way to go. Either that or a .patch . | 09:14:53 |
Pol | Isn't this supposed to be fixed upstream? | 09:15:20 |
fsagbuya | The upstream package is only the base installation; however, these are good extensions to add for the Flarum package, and that is how we used it. I only added this in my local nixpkgs because some users may not want to enable it. I can push these packages if it's alright. Although, I'm thinking there might be a way to make this optional. | 09:19:34 |
fsagbuya | In reply to @drupol:matrix.organ optional .patch is a good idea too | 09:22:51 |
Pol | composer doesn't have, like python, a way to compose groups of dependencies. | 09:41:45 |
Pol | So I guess using a patch is the way to go. | 09:41:53 |
Pol | I think the buildComposerProject2 will help this. | 09:42:06 |
Pol | https://github.com/NixOS/nixpkgs/pull/308059 | 09:42:39 |
fsagbuya | In reply to @drupol:matrix.orgWill try to add an optional patch upstream for the meantime until the buildComposerProject2 is merged. Thanks! | 09:46:18 |
Pol | especially this: https://github.com/NixOS/nixpkgs/pull/308059/files#diff-bead697169711cba8bf1ffc5a302d4dbe3f7d72513c9043cd431aba2c885885bR41 | 09:47:25 |
piotrkwiecinski | I reckon we should merge https://github.com/NixOS/nixpkgs/pull/308059 | 14:13:32 |
piotrkwiecinski | and start using it in the wild | 14:13:42 |