fixes for compilation on Solaris/SmartOS, other platforms

mdevilliers / redishappy

Redis Sentinel high availabillity daemon

Apache License 2.0

114 stars 23 forks source link

fixes for compilation on Solaris/SmartOS, other platforms #46

Closed doublerebel closed 8 years ago

doublerebel commented 8 years ago

Hello,

RedisHappy works great, with all tests passing, on Joyent's SmartOS (Solaris). It just took two minor changes. -race flag is not supported on Solaris, and InitLogging was missing from the default logger.

Thanks for your work!

doublerebel commented 8 years ago

The travis error is because vet has been deprecated as a separate package. I also received the error locally. It's the same as drone/drone#1565 and hashicorp/vault#1310. RedisHappy builds fine without pulling vet as a separate package. I can add that fix to this PR if you'd like.

mdevilliers commented 8 years ago

This look great - thanks.

I'll merge this and fix up the vet issue separately.

Thanks,

Mark