!jngDrdMgndWibPCYsR:nixos.org

Nix PHP

80 Members
A room for PHP developers running on Nix22 Servers

Load older messages


SenderMessageTime
19 May 2023
@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
@jtojnar:matrix.orgJan Tojnar well, I guess we could return includeDev flag but only use it to decide what to fetch but still include everything in the repository’s packages.json 14:09:25
@jtojnar:matrix.orgJan Tojnarthen the paths for dev packages would not exist but it would not matter14:10:11
@jtojnar:matrix.orgJan Tojnar * then the paths for dev packages would not exist but it would not matter as long as user always uses --no-dev 14:10:22
@drupol:matrix.orgPolI don't know if this would work.14:12:42

Show newer messages


Back to Room ListRoom Version: 6