Skip to content

Commit f43e9b2

Browse files
ppkarwaszhulkoba
authored andcommitted
Remove JANSI dependency in 2.x (apache#3070)
This commit: - Removes support for the outdated [Jansi 1.x](http://fusesource.github.io/jansi/) version in `Console` appender. - Rewrites `JAnsiTextRenderer`, use in the `%m{ansi}` and `%ex{ansi}` pattern converters to use our internal ANSI support instead of Jansi. Fixes apache#1736.
1 parent 1dd9a95 commit f43e9b2

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

log4j-1.2-api/src/test/java/org/apache/log4j/config/AbstractLog4j1ConfigurationTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -117,8 +117,8 @@ private Layout<?> testConsole(final String configResource) throws Exception {
117117
final String name = "Console";
118118
final ConsoleAppender appender = configuration.getAppender(name);
119119
assertNotNull(
120-
appender, "Missing appender '" + name + "' in configuration " + configResource + " → " + configuration);
121-
assertEquals(true, getFollowProperty(appender), "follow");
120+
"Missing appender '" + name + "' in configuration " + configResource + " → " + configuration, appender);
121+
assertTrue("follow", getFollowProperty(appender));
122122
assertEquals(Target.SYSTEM_ERR, appender.getTarget());
123123
//
124124
final LoggerConfig loggerConfig = configuration.getLoggerConfig("com.example.foo");

0 commit comments

Comments
 (0)