| 14 Apr 2023 |
Jan Tojnar | then it should be usable in shebangs just fine | 19:45:54 |
Pol | whot ?! | 19:46:06 |
Jan Tojnar | replace makeWrapper with makeBinaryWrapper | 19:46:21 |
Pol | Oh it's not done yet, going to do it and test. | 19:46:37 |
Jan Tojnar | also for the installation into $out/bin, instead of manual symlinking, is there a way to convince composer install to create the links there instead of in vendor/.bin? Then we would not even need to bother with postInstall. | 19:48:13 |
Pol | Let's do this fine tuning later on | 19:48:45 |
Pol | I want to make sure that I'm doing something stupid in this PR | 19:48:58 |
Pol | I'm pretty excited about it | 19:49:07 |
Pol | Jan Tojnar: THANKS ! It solved the issue :) | 19:52:10 |
| 15 Apr 2023 |
Pol | I replace most of PHP PHAR distribution derivation with their source. The only remaining one is psalm, I can't get it working. | 17:06:26 |
Pol | Find the PR at https://github.com/NixOS/nixpkgs/pull/225401 | 17:07:08 |
Pol | Reviews are welcome | 17:07:17 |
Pol | There is also a remark about deployer in the OP message. | 17:07:37 |
Pol | Jan Tojnar: unable to use the tag in `src` for deployer. It's missing from the repo. | 18:07:28 |
Jan Tojnar | Pol: notice the ^ | 18:07:58 |
Pol | Omg | 18:08:04 |
Pol | So smart | 18:08:12 |
Pol | Will do it | 18:08:15 |
Pol | After lunch. | 18:08:21 |
Pol | Fixed ! | 19:04:44 |
Jan Tojnar | Pol: regarding the programs, https://getcomposer.org/doc/articles/vendor-binaries.md#can-vendor-binaries-be-installed-somewhere-other-than-vendor-bin- | 19:18:08 |
Pol | yes? | 19:18:17 |
Jan Tojnar | COMPOSER_BIN_DIR should work | 19:18:47 |
Jan Tojnar | or is that just for dependencies? | 19:19:23 |
Pol | Wait, I don't understand what you're trying to explain here | 19:24:31 |
Pol | I don't think this is a good idea | 19:35:47 |
Pol | For example, if we take PHPUnit, | 19:36:02 |
Pol | The binary is not inside bin/ | 19:36:14 |
Pol | https://github.com/sebastianbergmann/phpunit/blob/main/composer.json | 19:36:34 |
Pol | That said, there is a bin attribute in the composer.json. | 19:37:09 |