| 27 Apr 2022 |
 ramsey | That looks like it successfully built PHP, but it’s failing when running the PEAR installer. Is there a PHP binary at sapi/cli/php? If so, what do you see when you run ./sapi/cli/php -v? | 19:35:30 |
| ramsey | This is the error:
Warning: fopen(): Unable to find the wrapper "https" - did you forget to enable it when you configured PHP?
What was your full ./configure command? It should be in a file named config.nice. | 19:40:52 |
| ramsey | Is OpenSSL installed? If not, or if configure couldn’t find it, maybe you built PHP without the HTTPS wrapper? | 19:42:23 |
| 28 Apr 2022 |
 Pol | Trying to answer your questions tonight | 16:46:25 |
| Pol | etu: Could you review this? https://github.com/NixOS/nixpkgs/pull/170614 | 16:47:27 |
| Pol | Download php-master.build.log | 17:08:33 |
| Pol | ramsey: Here's the full build log ^^ | 17:08:43 |
| Pol | Regarding OpenSSL: checking for OpenSSL support... no | 17:15:39 |
| Pol | Going to see if I can fix this. | 17:15:51 |
| ramsey | Yep. Looks like configure couldn’t find it, and it’s also not in the options provided to configure | 17:29:31 |
| Pol | checking for OpenSSL support... yes
checking for openssl >= 1.0.2... yes
checking for phar openssl support... no
| 17:41:19 |
| Pol | I'm progressing. | 17:41:22 |
| Pol | No I need to find how to enable "phar openssl support" | 17:41:33 |
| Pol | Do you know what is the flag to enable this? ramsey ? | 17:52:05 |
| Pol | Tried on Google, but couldn't find anything relevant yet. | 17:53:48 |
 etu | In reply to @drupol:matrix.org
etu: Could you review this? https://github.com/NixOS/nixpkgs/pull/170614 Question: Does the usage of this package actually depend on which version of PHP it's wrapped with? | 18:54:37 |
| Pol | Jan Tojnar: I'm trying to fix this issue and I notice something. | 18:07:33 |
| Pol |
- In here: https://github.com/fossar/nix-phps/blob/php-master/pkgs/phps.nix#L438 curl is enabled.
- When I try to build
php-master: nix build --override-input php-src github:php/php-src/PHP-8.1 -L .#php-master
- Then it fails
- Then I check the logs:
nix log /nix/store/azipihspd62sqzdlhgyg5y2kyfp81xpx-php-8.1.7.pre+date=20220427181447.drv | grep -i 'curl' | less
| 18:08:55 |
| Pol | The output is: | 18:08:58 |
| Pol | checking for cURL support... no | 18:09:08 |
| Pol | How is it possible? | 18:09:14 |
| Pol | Normally, curl should be enabled since curl extension is in the list of php-master derivation. | 18:14:46 |
| ramsey | Do you have the —enable-phar option? | 18:25:15 |
| ramsey | I don’t know about any options to enable openssl support with phar. I thought it was automatic | 18:27:53 |
| Pol | 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 |
| Pol | WDYT ? | 19:03:34 |
| etu | 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 |
| Pol | 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 | Yes. | 21:30:24 |
| Pol | 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 |
