!jngDrdMgndWibPCYsR:nixos.org

Nix PHP

74 Members
A room for PHP developers running on Nix21 Servers

Load older messages

SenderMessageTime
28 Apr 2022
@ramsey:matrix.orgramsey Do you have the —enable-phar option? 18:25:15
@ramsey:matrix.orgramsey I don’t know about any options to enable openssl support with phar. I thought it was automatic 18:27:53
@drupol:matrix.orgPol
In reply to @etu:semi.social
Question: Does the usage of this package actually depend on which version of PHP it's wrapped with?
I don't know, I checked but it's not in the documentation. I think you should add it as it is and focus on a refactoring and update all the packages at once. 		19:01:15
@drupol:matrix.orgPolWDYT ?19:03:34
@etu:semi.socialetu
In reply to @drupol:matrix.org
I don't know, I checked but it's not in the documentation. I think you should add it as it is and focus on a refactoring and update all the packages at once.
I don't agree since that would mean that it would break the configuration for people who just started using it. Just because a tool is written in PHP doesn't automatically make it valid to package it three times, once for every version of PHP we support. If it's a tool that doesn't care about PHP version it doesn't belong in php.packages, if it cares, like composer for example. It absolutely belongs there. 		21:26:51
@drupol:matrix.orgPol
In reply to @etu:semi.social
Question: Does the usage of this package actually depend on which version of PHP it's wrapped with?
It does not, I just checked. Do you want me to move the package to some other places? 		19:33:22
@etu:semi.socialetuYes.21:30:24
@drupol:matrix.orgPol
In reply to @etu:semi.social
I don't agree since that would mean that it would break the configuration for people who just started using it. Just because a tool is written in PHP doesn't automatically make it valid to package it three times, once for every version of PHP we support. If it's a tool that doesn't care about PHP version it doesn't belong in php.packages, if it cares, like composer for example. It absolutely belongs there.
True. Will make the change tomorrow morning, I'm already in bed. 		21:30:27
29 Apr 2022
@etu:semi.socialetu
In reply to @drupol:matrix.org
True. Will make the change tomorrow morning, I'm already in bed.
But I like your suggestion to move out packages that doesn't depend on different versions, so I'd say that would be worth doing after the new release branch is branched off :) 		07:06:34
@drupol:matrix.orgPol
In reply to @etu:semi.social
I don't agree since that would mean that it would break the configuration for people who just started using it. Just because a tool is written in PHP doesn't automatically make it valid to package it three times, once for every version of PHP we support. If it's a tool that doesn't care about PHP version it doesn't belong in php.packages, if it cares, like composer for example. It absolutely belongs there.
Fixed ! https://github.com/NixOS/nixpkgs/pull/170614 		07:50:07
@etu:semi.socialetu
In reply to @drupol:matrix.org
Fixed ! https://github.com/NixOS/nixpkgs/pull/170614
The commit message should now be: pdepend: init at 2.10.3 		07:58:55
@drupol:matrix.orgPoloops. You're right.07:59:04
@drupol:matrix.orgPolFixing it right now.07:59:11
@drupol:matrix.orgPolJob done.07:59:31
@ramsey:matrix.orgramsey Pol: did you solve the PEAR installation issue 19:17:48
@drupol:matrix.orgPol ramsey: yes, I succeeded to build using --without-pear ! The rest of the issue is related to directory naming, something I guess related to Nix. 19:49:02
@ramsey:matrix.orgramsey Interesting. I believe if you use --without-pear, the pecl installer tool will not be available, so PECL extensions have to be manually installed. 19:49:41
@ramsey:matrix.orgramsey this is because pecl uses pear 19:50:07
@drupol:matrix.orgPolTo be tested, in Nix, each extension are built separately (AFAIK). 19:50:27
@drupol:matrix.orgPolThe build system is some kind of genius system, and I haven't understood everything.19:50:49
@drupol:matrix.orgPolExtensibility comes at a cost :)19:52:28
@ramsey:matrix.orgramseythis is why I've kinda given up on Nix for right now... it's far too over my head to understand 🙂 19:56:12
@ramsey:matrix.orgramseyI might come back to it later19:56:16
@drupol:matrix.orgPolOh, nose. Feel free to ask any kind of question, all the people here are very keen to give advice and help. They are impressively patient as well (I can relate !).19:59:14
@drupol:matrix.orgPol * Oh, nose. Feel free to ask any kind of question about nix in here, all the people here are very keen to give advice and help. They are impressively patient as well (I can relate !).19:59:24
@drupol:matrix.orgPol On my side, I'm glad to have you here ramsey ! 20:16:39
30 Apr 2022
@drupol:matrix.orgPol

PR to reviews:

  • https://github.com/NixOS/nixpkgs/pull/170962
  • https://github.com/NixOS/nixpkgs/pull/170614

If there are brave souls to have a look at those, it would be super nice ! Thanks and have a nice weekend !

08:44:13
2 May 2022
@drupol:matrix.orgPol Jan Tojnar: looks like your night was extremely short ! Going to review what you did in the train in 1h. 05:06:47
@drupol:matrix.orgPol
In reply to @etu:semi.social
The commit message should now be: pdepend: init at 2.10.3
Hi etu, I fixed the stuff, could you have a look at it? 		08:46:16
@drupol:matrix.orgPol

Hi stephank (Stéphan Kochen) !
I just got this error:

 $ composer nixify

In Command.php line 301:
                                                                                               
  [TypeError]                                                                                  
  Return value of "Nixify\NixifyCommand::execute()" must be of the type int, "null" returned.  
                                                                                               

Exception trace:
  at phar:///nix/store/vn3jg9v1gnchw3c62yi7i58m88qf3pql-php-composer-2.3.5/libexec/composer/composer.phar/vendor/symfony/console/Command/Command.php:301
 Symfony\Component\Console\Command\Command->run() at phar:///nix/store/vn3jg9v1gnchw3c62yi7i58m88qf3pql-php-composer-2.3.5/libexec/composer/composer.phar/vendor/symfony/console/Application.php:1015
 Symfony\Component\Console\Application->doRunCommand() at phar:///nix/store/vn3jg9v1gnchw3c62yi7i58m88qf3pql-php-composer-2.3.5/libexec/composer/composer.phar/vendor/symfony/console/Application.php:299
 Symfony\Component\Console\Application->doRun() at phar:///nix/store/vn3jg9v1gnchw3c62yi7i58m88qf3pql-php-composer-2.3.5/libexec/composer/composer.phar/src/Composer/Console/Application.php:334
 Composer\Console\Application->doRun() at phar:///nix/store/vn3jg9v1gnchw3c62yi7i58m88qf3pql-php-composer-2.3.5/libexec/composer/composer.phar/vendor/symfony/console/Application.php:171
 Symfony\Component\Console\Application->run() at phar:///nix/store/vn3jg9v1gnchw3c62yi7i58m88qf3pql-php-composer-2.3.5/libexec/composer/composer.phar/src/Composer/Console/Application.php:130
 Composer\Console\Application->run() at phar:///nix/store/vn3jg9v1gnchw3c62yi7i58m88qf3pql-php-composer-2.3.5/libexec/composer/composer.phar/bin/composer:88
 require() at /nix/store/vn3jg9v1gnchw3c62yi7i58m88qf3pql-php-composer-2.3.5/libexec/composer/composer.phar:29

nixify
13:36:36

There are no newer messages yet.

Back to Room ListRoom Version: 6