Issue #5096 has been updated by knu (Akinori MUSHA).


Oh, of course.  I didn't carefully read the latest patch.

I've revised the implementation and the tests as attached.

Here's the list of what I changed:

- Reduced map constants.  You shouldn't need this many.

- Changed Syslog::Logger::SYSLOG to Syslog::Logger.syslog using a class variable as storage.  I didn't like the way the constant was used.

- Changed to call syslog methods with ('%s', message) instead of manually replacing %'s.

- Changed a regexp so it exactly matches (ANSI & XTerm) color escape sequences.

Please merge it if it's OK, and you have my approval.
----------------------------------------
Feature #5096: offer Logger-compatibility for syslog ext
https://bugs.ruby-lang.org/issues/5096#change-26668

Author: normalperson (Eric Wong)
Status: Assigned
Priority: Low
Assignee: knu (Akinori MUSHA)
Category: 
Target version: 


There's http://rubygems.org/gems/SyslogLogger, but I would like to have
ths compatibility by default.  Having to remember to install the
oddly-capitalized SyslogLogger gem makes things harder than it should
be.

Importing the SyslogLogger gem would solve this issue.


-- 
http://bugs.ruby-lang.org/