p5-RedisDB / RedisDB

Perl extension to access Redis
22 stars 11 forks source link

Win32 Compatibility #1

Closed sanko closed 13 years ago

sanko commented 13 years ago

Hi!

You asked for some testing on Windows over on PerlMonks and I haven't gone too deep yet but here are a few things I've run in to. BTW, I'm using the port of redis found here: http://github.com/dmajkic/redis/downloads

t/RedisServer.pm is looking for File::Spec->catfile($_, 'redis-server') but on Windows the full filename includes an extension so it would look like [...]redis-server.exe.

Socket::MSG_DONTWAIT which you use in Redis::_recv_data_nb is undefined on Win32. An obvious (but my least favorite) choice may be to use select( ... ) first but you may just want to use the non-blocking two-step found at the end of Pedro Melo's Redis module.

trinitum commented 13 years ago

Thanks, I just pushed v0.15 which hopefully fixes these issues.