Bug #13040
closed
syslog/logger uses "require 'logger'" which is interpreted as circular dependency
Added by lemsx1 (Luis Mondesi) over 7 years ago.
Updated almost 7 years ago.
Description
while using the 'syslog/logger' require from Ruby it works well, however if one tries to use it via "test" it complaints about circular require and the constant "Logger" not defined. After a little digging, I realized that 'syslog/logger' has a line that says require 'logger' which could mean itself!
The fix is to use require_relative '../logger' as this is what it intends in order to piggy back on Logger:
Current code for syslog/logger:
-
frozen_string_literal: false¶
- require 'syslog'
- require 'logger'
Fixed code:
-
frozen_string_literal: false¶
- require 'syslog'
- require_relative '../logger'
'syslog/logger.rb' and 'logger.rb' are different files.
What do you mean by "test"?
- Status changed from Open to Feedback
Nobuyoshi Nakada wrote:
'syslog/logger.rb' and 'logger.rb' are different files.
What do you mean by "test"?
For example, I setup a new Rails 5.0.0.1 app and minitest for use with "spec" via a test_helper file, nothing out of the norm. Then ran "rake test" and saw a complaint (warning) about "circular dependency". If I turn the warnings off, then you won't see the message, but would fail saying logger constant is not defined. It took me a little while to realize that both syslog/logger and "logger" are both in the path while "require 'logger'" is called from syslog/logger.
And like I said above, require_relative disambiguate the call and makes it very clear that you mean to require the "logger" from the previous directory.
While my app runs I do not get this warning or error, it only happens in "test mode". Whatever minitest does to run tests causes Ruby to get confused.
Am I the only person having this problem? This is in different code sources we are maintaining. Here is the latest. In a new project I setup test with minitest/spec
- added pp $LOAD_PATH to my test_helper.rb
- rake test
$> rake test
["/Users/lmondesi/Projects/new_project/lib",
"/Users/lmondesi/Projects/new_project/lib/new_project",
"/usr/local/lib/ruby/gems/2.4.0/gems/rake-12.0.0/lib",
"/usr/local/lib/ruby/gems/2.4.0/gems/did_you_mean-1.1.0/lib",
"/usr/local/lib/ruby/gems/2.4.0/gems/minitest-5.10.1/lib",
"/usr/local/lib/ruby/site_ruby/2.4.0",
"/usr/local/lib/ruby/site_ruby/2.4.0/x86_64-darwin16",
"/usr/local/lib/ruby/site_ruby",
"/usr/local/lib/ruby/vendor_ruby/2.4.0",
"/usr/local/lib/ruby/vendor_ruby/2.4.0/x86_64-darwin16",
"/usr/local/lib/ruby/vendor_ruby",
"/usr/local/Cellar/ruby/2.4.1_1/lib/ruby/2.4.0",
"/usr/local/Cellar/ruby/2.4.1_1/lib/ruby/2.4.0/x86_64-darwin16"]
/usr/local/Cellar/ruby/2.4.1_1/lib/ruby/2.4.0/syslog/logger.rb:81:in <class:Logger>': uninitialized constant Logger (NameError) from /usr/local/Cellar/ruby/2.4.1_1/lib/ruby/2.4.0/syslog/logger.rb:48:in
<top (required)>'
from /usr/local/Cellar/ruby/2.4.1_1/lib/ruby/2.4.0/rubygems/core_ext/kernel_require.rb:55:in require' from /usr/local/Cellar/ruby/2.4.1_1/lib/ruby/2.4.0/rubygems/core_ext/kernel_require.rb:55:in
require'
Ok, I was able to reproduce this now.
- create a new Rails Project
- create an initializer under configs/initializer named "delayed_job.rb" with:
require 'syslog/logger'
Delayed::Worker.delay_jobs = !Rails.env.test?
Delayed::Worker.logger = Rails.logger
hack to reopen syslog in local1¶
Syslog::Logger.syslog.close
Syslog::Logger.syslog = Syslog.open('bmanage', Syslog::LOG_ODELAY, Syslog::LOG_LOCAL1)
- run bin/rake test or bundle exec rake test
Essentially, this was required with older libraries of Syslog::Logger. Now that Syslog::Logger allows one to change facility when initializing an instance, there is no need for this code.
After removing this code, the bug went away!
You can now close this issue.
- Status changed from Feedback to Rejected
Also available in: Atom
PDF
Like0
Like0Like0Like0Like0Like0Like0