!jngDrdMgndWibPCYsR:nixos.org

Nix PHP

76 Members
A room for PHP developers running on Nix22 Servers

Load older messages


SenderMessageTime
9 Nov 2022
@etu:failar.nuetuSomehow we have lost the maintainers arrays from phpPackages packages14:37:48
@etu:failar.nuetu
~/code/nixpkgs $ nix repl .
nix-repl> :p php.packages.composer.meta
{ available = true; broken = false; insecure = false; mainProgram = "composer"; name = "php-composer-2.4.4"; outputsToInstall = [ "out" ]; position = "/home/etu/code/nixpkgs/pkgs/development/php-packages/composer/default.nix:7"; unfree = false; unsupported = false; }
14:38:30
@etu:failar.nuetuHowever I can see it in the code, so my best guess is that it has something to do with https://github.com/NixOS/nixpkgs/commit/58919907a73f2bd0329f8943875d7bab9c5eca8e14:41:26
@etu:failar.nuetu Yep, I can confirm locally that removing the setting of meta.mainProgram in the mkDerivation definition gets us the maintainers list back. 14:42:48
@drupol:matrix.orgPolThis is maybe the reason why I was not ping anymore14:48:01
@drupol:matrix.orgPolThere's this PR that I haven't seen: https://github.com/NixOS/nixpkgs/pull/19803714:49:26
@etu:failar.nuetuYeah, I RSS the updates regarding packages that I maintain via repology and just got a whole bunch of updates that I stopped receiving updates for all php packages. So I was curious about that 😉14:49:47
@drupol:matrix.orgPolnice finding14:50:15
@jtojnar:matrix.orgJan Tojnar maybe we should just drop the mkDerivation 18:21:24
@jtojnar:matrix.orgJan Tojnar I do not really see a point prefixing the package name with php https://github.com/ilkecan/nixpkgs/commit/06e0ede06ec1b2e97637b93e2b00c1eb9c71d3a6 18:21:43
10 Nov 2022
@drupol:matrix.orgPolWe need senior review here: https://github.com/NixOS/nixpkgs/pull/19847013:01:35
@drupol:matrix.orgPolIt modifies the PHP stuff13:01:44
11 Nov 2022
@ma27:nicht-so.sexyma27friendly ping re the PR linked above, would be cool to get some more reviewers on that :) 22:43:16
12 Nov 2022
@drupol:matrix.orgPol Thanks etu! 08:01:57
@etu:failar.nuetu Pol: I was thinking about this NTS/ZTS thing 10:08:31
@drupol:matrix.orgPolyep10:08:39
@drupol:matrix.orgPolme too :)10:08:42
@etu:failar.nuetuWhy is it depending on isDarwin?10:08:57
@etu:failar.nuetuIs it because ZTS doesn't work on darwin or something like that?10:09:15
@drupol:matrix.orgPolNo, zts was mostly used because of mod_php on apache10:09:47
@etu:failar.nuetuThen it's weird that it depends on isDarwin 😛10:10:24
@etu:failar.nuetuSo we would want... NTS by default, right?10:19:46
@etu:failar.nuetuAnd then maybe a separate attribute name for php with ZTS10:19:58
@etu:failar.nuetu like mod_php80 mod_php81 mod_php or something? And then use that by default for apache. 10:20:46
@drupol:matrix.orgPolyes10:21:45
@etu:failar.nuetu Pol: You feel up for doing those changes to your draft? 10:28:42
@etu:failar.nuetu Pol: And yeah, stop make it depending on isDarwin 10:28:56
@drupol:matrix.orgPolI can try :)10:48:57
@drupol:matrix.orgPol Job done etu ! https://github.com/NixOS/nixpkgs/pull/194172 21:57:31
@drupol:matrix.orgPolI like the change21:57:37

There are no newer messages yet.


Back to Room ListRoom Version: 6