!jngDrdMgndWibPCYsR:nixos.org

Nix PHP

82 Members
A room for PHP developers running on Nix24 Servers

Load older messages


SenderMessageTime
12 Jun 2023
@drupol:matrix.orgPolTrue, but I have no clue on how to integrate that.17:33:41
@drupol:matrix.orgPol Do you think adding a composerHome input to composer derivation is a good idea? 17:34:01
@drupol:matrix.orgPolI just pushed the changes for the composer keys ! https://github.com/NixOS/nixpkgs/pull/225401/commits/9e1540b6f6f7055af9710aa1c8c83b9966953cce17:34:24
@tgerbet:matrix.orgtgerbet
In reply to @drupol:matrix.org
Do you think adding a composerHome input to composer derivation is a good idea?
Probably not, it will break too many things (e.g. access to GH OAuth access token)
17:38:55
@drupol:matrix.orgPolMmhm I don't know17:40:15
@tgerbet:matrix.orgtgerbetIt looks like we only get those warnings/errors because we are using the phar and in this situation it expects to be able to upgrade itself https://github.com/composer/composer/blob/bbea3e5284fd859373c292c7f696f9647f9a59ef/src/Composer/Command/DiagnoseCommand.php#L15217:41:33
@drupol:matrix.orgPolActually, we can reproduce the same issue with the src version, let me do a poc17:42:25
@drupol:matrix.orgPolOh no you're right17:43:34
@drupol:matrix.orgPolimage.png
Download image.png
17:43:37
@drupol:matrix.orgPolIt only does that with the PHAR indeed.17:43:45
@drupol:matrix.orgPolSo sad it doesnt use proper XDG standard.17:44:29
@drupol:matrix.orgPolSo we could install those keys in a specific place and be done with it.17:44:44
@tgerbet:matrix.orgtgerbet Yep. Since it is more of a Composer issue than something related to the plugin I would just let not bother with the keys in the plugin. It's not like a composer diagnose will be issue while building the PHP packages 😄 17:53:16
@drupol:matrix.orgPolI just wanted to be just pixel perfect... "just in case of... "17:58:04
@tgerbet:matrix.orgtgerbetUnpacking the phar archive seems to work https://github.com/NixOS/nixpkgs/pull/23741019:02:36
@drupol:matrix.orgPolLet me be completely honest with this19:21:23
@drupol:matrix.orgPolI don't think we should do that... 19:21:36
@drupol:matrix.orgPol I believe we should let the phar untouched. 19:21:56
@drupol:matrix.orgPolIf something has to be fixed, it's in Composer, not in here19:22:12
@drupol:matrix.orgPolLet me comment in the PR so we can continue there.19:23:51
@drupol:matrix.orgPol tgerbet: Hope no hard feeling on this 20:02:10
@drupol:matrix.orgPol * tgerbet: Hope no hard feelings on this 20:02:14
@tgerbet:matrix.orgtgerbetOh no20:02:26
@tgerbet:matrix.orgtgerbetCompletely fine :)20:02:41
13 Jun 2023
@marcus_yallow:matrix.org@marcus_yallow:matrix.org joined the room.13:15:02
14 Jun 2023
@hexa:lossy.networkhexa left the room.16:15:06
15 Jun 2023
@shyim:matrix.orgShyimAs someone packaged already PHP 8.3alpha? :) 21:20:18
@tgerbet:matrix.orgtgerbet Pol has tried but it encountered some issues. I did not had time to give it a try yet 21:26:21
@drupol:matrix.orgPol=> https://github.com/php/php-src/issues/1140821:27:01
@drupol:matrix.orgPol Jan Tojnar: 21:27:06

Show newer messages


Back to Room ListRoom Version: 6