search

nrk / phpiredis

PHP extension for Redis based on Hiredis
BSD 2-Clause "Simplified" License
490 stars 67 forks source link

pls support php7 #43

Closed dkeepersun closed 8 years ago

dkeepersun commented 8 years ago

can't use at php7 now . pls support it. thanks ! :+1:

enleur commented 8 years ago

+1

drefined commented 8 years ago

There's currently a PR for this https://github.com/nrk/phpiredis/pull/39. I'm patiently waiting. :+1:

dkeepersun commented 8 years ago

+2

quantverse commented 8 years ago

:+1: +1

Orion1 commented 8 years ago

+1

dkeepersun commented 8 years ago

+3

moechofe commented 8 years ago

+1

ellisio commented 8 years ago

+1 would love to have this enabled on production again. :)

mmucklo commented 8 years ago

What happened to this package - is anyone maintaining it? Should someone fork and should we all move on?

nrk commented 8 years ago

@mmucklo this library is not abandoned, but it's been months since the last time I had time for my opensource projects and I myself am quite unhappy about the situation. That said, #39 is a starting point for PHP7 compatibility but it's a bit messy so I'm not so keen about merging it as-is. Unfortunately +1s and similar stuff don't help, contributions do. Actual help to improve that PR would be more than appreciated and would speed things up. Thanks.

rchunping commented 8 years ago

+1 please support php7 it's a good extension, we use this 3 years

muratGG commented 8 years ago

+1 This is the only extension keeping us from PHP 7.

Cysioland commented 8 years ago

:+1:

Barbery commented 8 years ago

+1

plessmann commented 8 years ago

I also vote for this feature :+1:

nrk commented 8 years ago

Closing this issue as we are almost there! See #50 for details.