Re: [PATCH] Add php

[Hartmut Goebel <h.goebel@crazy-compilers.com>]

Am 17.11.2016 um 01:01 schrieb Marius Bakke:
> If there are no further comments, I will push this in the next few days:

Thanks for your work on this. It seems to be laborious.

I don't have time to inspect not to test the patch, so just some
thought. I'm not a php-user, but wanted to install a php-driven website
on Guix some weeks ago and had a look on what else would be required.

  * Does this php package include what other distributions call
    "php-cli", "php-fpm" and "php-cgi"? Maybe you want to mention it in
    the synopsis. (Addendum: I just found in
    <https://packages.debian.org/source/jessie/php5> that it looks like
    these are part of the package.)
  * As far as I understood, "pear" and "pecl" are important for
    installing php packages. Are these part of your package, too?


  * What do you think about adding "Packaging guideline" for "PHP
    Modules" section to the manual?
  * Do yo think we need some php build system?


-- 
Regards
Hartmut Goebel

| Hartmut Goebel          | h.goebel@crazy-compilers.com               |
| www.crazy-compilers.com | compilers which you thought are impossible |