credis-test get wrong "info" result under redis 1.2.3 / cygwin about credis HOT 5 CLOSED

The info return code -97 states that there has been a protocol error.

I guess it is because you're talking to a Redis server of version "1.2.3". 
Currently 
Credis only support the commands supported by Redis version 1.02.

Perhaps Credis should tell this in a better way than just returning an error?

I've modified the latest trunk version of credis to work with the most recent 
redis server protocol (I'm using 1.3 
for testing, but I think it would also work with 1.2.3).  I haven't added 
support for any new commands yet, but 
the existing credis-test program now runs successfully.

One way to detect which protocol to use would be to issue an INFO command as 
part of the credis_connect() 
function to find out the version of the server we are talking to.  You only 
need to parse and check the 
redis_version key.  We could then add version information to the REDIS struct 
so that other api functions (such as 
credis_keys() and cr_push()) could dynamically check it and send the 
appropriate protocol.

I posted my renegade version of credis (see previous comment) to my github 
account at:
http://github.com/itfrombit/credis
in case anyone wants to check it out.  This includes the version check in the 
credis_connect() call, so it should 
fall back to the 1.0.2 protocol if it detects an older server.

I can also post an svn diff patch if you are interested.

Cool. I briefly looked at your changes. I don't think a patch is necessary. 
I'll add 
this to the trunk. Thanks.

Fixed in revision 30. Verified on Redis 1.02 and 1.2.6.

Actually, the working copy of Redis is not compatible with 1.2.6. Therefore I 
took 
another approach, letting credis parse as many fields as it knows, hence 
decoupling the 
strict dependency to the Redis server allowing for a more flexible 
compatibility(?).