nrk
commented
7 years ago Makes sense, will start making these changes next week.
By the way, unfortunately building on Windows is not supported by hiredis (well the parser part do work but the rest is not compatible). Actually Microsoft has a fork of Redis that ships with a win32-compatible version of hiredis but unfortunately it's not maintained as a separate project, maybe it's possible to make it work by building from sources but it's probably a bit complicated.
PS: PECL account has been approved yesterday!
Rohaq
Proposal 1
Rational: when installed (from package or from pecl), this documentation is no more needed (so doesn't need to be tagged as role="doc" in package.xml, role="src" is enough)
Proposal 2
Document the different way to install
Installation from sources (current content)
Installation from pecl: pecl install phpiredis
Binary Installation, as lot of people prefer to rely on this method when possible
Windows users can use the "DLL" link from the pecl page (if supported, as I don't see the config.w32 for now)
Fedora users can use the php-phpiredis (https://apps.fedoraproject.org/packages/php-phpiredis) from official repository and: dnf install php-phpiredis
RHEL / CentOS users can use the php-phpiredis package from EPEL repository (https://fedoraproject.org/wiki/EPEL): yum install php-phpiredis
Other distributions can be added later on maintainer/user request.
Of course, we have to wait for pecl approval. For discussion (and I can open a PR... even if my English is probably not the better)