!jngDrdMgndWibPCYsR:nixos.org

Nix PHP

75 Members
A room for PHP developers running on Nix21 Servers

Load older messages


SenderMessageTime
19 May 2023
@tgerbet:matrix.orgtgerbet
In reply to @jtojnar:matrix.org
or we could just optimize the fetcher so that downloading dev dependencies is not pain
For production env you do not want the dev deps. Some might do things that are fine in dev but sketchy for production if you have a mistake in your configuration.
21:36:26
@tgerbet:matrix.orgtgerbetThe famous eval-stdin.php from PHPUnit comes to mind21:36:47
@tgerbet:matrix.orgtgerbet * The famous eval-stdin.php from PHPUnit comes to my mind 21:36:55
@drupol:matrix.orgPolyeah... true.21:37:42
@jtojnar:matrix.orgJan Tojnar tgerbet: but composer install already defaults to --dev so you will need to run composer install --no-dev anyway 21:38:33
@tgerbet:matrix.orgtgerbetHum yes indeed 👍️21:39:12
@drupol:matrix.orgPol Excellent Jan Tojnar 22:32:00
@drupol:matrix.orgPolTomorrow morning I will update everything22:32:19
@jtojnar:matrix.orgJan Tojnar tried optimizing the fetching in composition-c4 by fetching just a single commit (which Git supposedly allows) but Nix does not allow fetching commit refs – it will assume the ref is a head if it does not start with refs/ 22:36:03
@drupol:matrix.orgPolI have an idea.22:41:02
@drupol:matrix.orgPolOh no, forget it.22:41:11
@drupol:matrix.orgPolI went downstairs, I wanted to try what you did on nixpkgs :)22:43:30
@drupol:matrix.orgPolAnd it works.22:44:44
@drupol:matrix.orgPolwow.22:44:46
@drupol:matrix.orgPol I think there would be a way to avoid downloading all the -dev stuff. 23:00:37
@drupol:matrix.orgPol The idea would be to create a wrapping composer.json that would require the package we want to build. A custom repo would also be added in that composer.json, pointing to the path of the package source. 23:01:34
@drupol:matrix.orgPolI will try that idea at some point.23:01:41
@drupol:matrix.orgPolRedacted or Malformed Event23:15:24
@drupol:matrix.orgPol Building composer and phpunit works in the branch: https://github.com/NixOS/nixpkgs/pull/232450 23:22:25
@drupol:matrix.orgPol Thanks Jan Tojnar ! 23:22:44
@drupol:matrix.orgPolSadly, it won't be possible to package psalm :( 23:51:41
@drupol:matrix.orgPolimage.png
Download image.png
23:51:45
20 May 2023
@jtojnar:matrix.orgJan Tojnar Gaël Reyrol: I have updated the Python script to try to use if for impure derivations https://github.com/fossar/composition-c4/pull/5 01:45:20
@drupol:matrix.orgPol I updated the drupol/composer-local-repo-plugin: https://github.com/drupol/composer-local-repo-plugin/ 07:52:21
@drupol:matrix.orgPol Jan Tojnar: Ideally we should not have derivation like phpstan/phpstan in the store when installing composr. 14:06:58
@drupol:matrix.orgPol * Jan Tojnar: Ideally we should not have derivation like phpstan/phpstan in the store when installing composer. 14:07:01
@drupol:matrix.orgPolHow can we prevent that?14:07:06
@drupol:matrix.orgPol(thanks for replying to Jordi btw!)14:07:27
@jtojnar:matrix.orgJan Tojnar Pol: why not? 14:07:30
@drupol:matrix.orgPol

Because

  1. It's polluting the store (not eco-friendly)
  2. It's absolutely not needed in Composer
14:08:09

Show newer messages


Back to Room ListRoom Version: 6