nrk/phpiredis

[RFE] improve installation doc

Opened this issue · 2 comments

Proposal 1

  • split out installation instruction from README.md to INSTALL.md

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)

nrk commented

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 commented

PS: PECL account has been approved yesterday!

Not much done with it since, though! ;)

I know this is an old issue now - I stumbled across it whilst looking for a DLL for phpiredis. Any chance of making a compiled version for those of us just looking to meet platform requirements in our local dev environments?

Cheers.