From f092b7db45e01b459aa899e8c0d076b97c137c16 Mon Sep 17 00:00:00 2001 From: Matthias Hecker Date: Tue, 31 Mar 2020 14:30:08 +0200 Subject: refactor: logger moved away from ircbot (exp.) This moves the logger management thread/queue to a seperate singleton. It removes the explicit stopping/restarting of the logging thread since the thread should behave like a daemon thread anyway. Still needs to be tested to work in daemonize. --- bin/rbot | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) (limited to 'bin') diff --git a/bin/rbot b/bin/rbot index a8dd39c4..86f907cf 100755 --- a/bin/rbot +++ b/bin/rbot @@ -61,17 +61,13 @@ opts = GetoptLong.new( ["--version", "-v", GetoptLong::NO_ARGUMENT] ) -$debug = $DEBUG $daemonize = false opts.each {|opt, arg| - $debug = true if(opt == "--debug") $daemonize = true if(opt == "--background") $opts[opt.sub(/^-+/, "")] = arg } -$cl_loglevel = $opts["loglevel"].to_i if $opts["loglevel"] - if ($opts["trace"]) set_trace_func proc { |event, file, line, id, binding, classname| if classname.to_s == $opts["trace"] @@ -116,6 +112,13 @@ if ($opts["help"]) end if(bot = Irc::Bot.new(ARGV.shift, :argv => orig_opts)) + # setup logger based on command line arguments + loglevel = $opts['loglevel'] ? $opts['loglevel'].to_i : nil + loglevel = $opts['debug'] ? 0 : nil + if loglevel + Irc::Bot::LoggerManager.instance.set_level(loglevel) + end + # just run the bot bot.mainloop end -- cgit v1.2.3