Browse files

Dont use logger in options parsing, fixes #1335

  • Loading branch information...
1 parent 0da83d6 commit 6bcca735fe6d35fe4b4ca55b02660e6c892850c9 @mperham committed Nov 13, 2013
Showing with 3 additions and 3 deletions.
  1. +3 −2 lib/sidekiq/cli.rb
  2. +0 −1 test/config.yml
View
5 lib/sidekiq/cli.rb
@@ -326,8 +326,9 @@ def parse_config(cfile)
end
ns = opts.delete(:namespace)
if ns
- Sidekiq.logger.warn("namespace should be set in your ruby initializer, is ignored in config file")
- Sidekiq.logger.warn("config.redis = { :url => ..., :namespace => '#{ns}' }")
+ # logger hasn't been initialized yet, puts is all we have.
+ puts("namespace should be set in your ruby initializer, is ignored in config file")
+ puts("config.redis = { :url => ..., :namespace => '#{ns}' }")
end
opts
end
View
1 test/config.yml
@@ -3,7 +3,6 @@
:require: ./test/fake_env.rb
:pidfile: /tmp/sidekiq-config-test.pid
:logfile: /tmp/sidekiq.log
-:namespace: sidekiq
:concurrency: 50
:queues:
- [<%="very_"%>often, 2]

0 comments on commit 6bcca73

Please sign in to comment.