NodeRedis/node-redis-parser

hiredis is not compiling in nodejs 12

rj-david opened this issue · 7 comments

rj-david commented

Is there any way that this package will no longer depend on hiredis? Hiredis is not building properly using node-gyp in nodejs 12

stelcodes commented

I'm getting the same issue when building with parcel

sookoll commented

Any news on this? As hiredis is pretty much abonded, it should be removed as dependency.

EDIT: Ah, v3 it is removed already, node-redis still uses 2.6 and that was causing problems.

swateek commented

@sookoll could you install "redis"? I didn't quite understand your solution. Help?

yosiat commented

If you can upgrade the latest node-redis you don't need hiredis, it uses this repo which is fastest than hiredis as authors claim.

If you are like me and stuck at some version, you can use this fork redis/hiredis-node#144 until the PR will get merged

swateek commented

I really didn't need hiredis. I moved to use "ioredis" and it works good for me.

Salakar commented

Hey all,

hiredis is no longer used in the latest parser, the the redis library has been updated to the latest parser version in redis/node-redis@78936ac (unreleased still, no eta).

Given that a fix is sat on master for this I'm going to go ahead and close this issue now. Thanks

ropre4 commented

It's been more than a year, can there be a new release to include this fix? redis-parser, and redis packages do not work on node 12 and node 14.

  • 👍1