DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG· RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT <http://issues.apache.org/bugzilla/show_bug.cgi?id=37965>. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND· INSERTED IN THE BUG DATABASE.
http://issues.apache.org/bugzilla/show_bug.cgi?id=37965 ------- Additional Comments From [EMAIL PROTECTED] 2005-12-21 02:04 ------- Rev 358162 adds the SyslogAppenderTest to the test suite and addresses the API changes identified by the clirr target. There are several issues with the 1.3 implementation that have not been addressed. First, I'm uncertain if the 1.3 implementation was functional. The implementation of activateOptions did not call super.activateOptions() which resulted in any attempted appending to be stopped with AppenderSkeleton's check that the appender had been activated. log4j 1.2's API had a facilityPrinting property that is now supported but is not referenced in the 1.3 code and no obvious place to add equivalent behavior. The 1.3 branch changed the behavior of activateOptions so that a call on an imcompletely configured appender would throw an IllegalStateException. In 1.2, an incomplete configuration would result the ErrorHandler being called on the first log attempt. The testAppenderNoHost test was changed so that it expects the 1.3 behavior. The unit tests exercise the API as much as possible, but do nothing to check that anything ever made it to the syslog in either the 1.2 or 1.3 branches. -- Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee. --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
