Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Fix warning in redis.c for sentinel config load #603

Merged
merged 1 commit into from Jul 25, 2012

Conversation

Projects
None yet
2 participants
Contributor

mrb commented Jul 24, 2012

Currently if you load redis-sentinel without a config file, it tells you to use redis-server instead. This fixes the warning.

Took a suggestion from @pietern to use argv[0] - this was initially to avoid checking redis.sentinel_mode, but I like this compromise because it provides both the exact string used to start the binary + the right name of the conf file.

Contributor

mrb commented Jul 25, 2012

@antirez Updated with your suggestion

@antirez antirez added a commit that referenced this pull request Jul 25, 2012

@antirez antirez Merge pull request #603 from mrb/fix_sentinel_config_warning
Fix warning in redis.c for sentinel config load
42c5718

@antirez antirez merged commit 42c5718 into antirez:unstable Jul 25, 2012

Owner

antirez commented Jul 25, 2012

Thank you, merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment