- logback-dev - mailman3.qos.ch

svn commit: r1730 - logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/net
by noreply.ceki＠qos.ch 04 Aug '08

04 Aug '08

Author: ceki Date: Mon Aug 4 22:49:22 2008 New Revision: 1730 Modified: logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/net/SocketAppenderTest.java Log: test now wait for the SimpleSocketServer to send a notification Modified: logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/net/SocketAppenderTest.java ============================================================================== --- logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/net/SocketAppenderTest.java (original) +++ logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/net/SocketAppenderTest.java Mon Aug 4 22:49:22 2008 @@ -9,12 +9,10 @@ */ package ch.qos.logback.classic.net; - import static org.junit.Assert.*; import java.util.Map; - import org.junit.Test; import org.slf4j.MDC; import org.slf4j.Marker; @@ -29,17 +27,20 @@ import ch.qos.logback.core.read.ListAppender; import ch.qos.logback.core.util.StatusPrinter; -public class SocketAppenderTest { +public class SocketAppenderTest { static final String LIST_APPENDER_NAME = "la"; + static final int JOIN__OR_WAIT_TIMEOUT = 200; + static final int SLEEP_AFTER_LOG = 100; + int port = 4561; LoggerContext lc = new LoggerContext(); LoggerContext serverLC = new LoggerContext(); ListAppender<LoggingEvent> la = new ListAppender<LoggingEvent>(); SocketAppender socketAppender = new SocketAppender(); - + private SimpleSocketServer simpleSocketServer; - + @Test public void startFailNoRemoteHost() { SocketAppender appender = new SocketAppender(); @@ -52,17 +53,16 @@ @Test public void recieveMessage() throws InterruptedException { fireServer(); + waitForServerToStart(); configureClient(); Logger logger = lc.getLogger(LoggerContext.ROOT_NAME); logger.debug("test msg"); - - Thread.sleep(100); - - // Wait max 2 seconds for mock server to finish. However, it should - // finish much sooner than that. + + Thread.sleep(SLEEP_AFTER_LOG); + simpleSocketServer.close(); - simpleSocketServer.join(2000); + simpleSocketServer.join(JOIN__OR_WAIT_TIMEOUT); assertTrue(simpleSocketServer.isClosed()); assertEquals(1, la.list.size()); @@ -74,16 +74,15 @@ @Test public void recieveWithContext() throws InterruptedException { fireServer(); + waitForServerToStart(); configureClient(); Logger logger = lc.getLogger(LoggerContext.ROOT_NAME); logger.debug("test msg"); + Thread.sleep(SLEEP_AFTER_LOG); - // Wait max 2 seconds for mock server to finish. However, it should - // finish much sooner than that. - Thread.sleep(100); simpleSocketServer.close(); - simpleSocketServer.join(2000); + simpleSocketServer.join(JOIN__OR_WAIT_TIMEOUT); assertTrue(simpleSocketServer.isClosed()); assertEquals(1, la.list.size()); @@ -104,6 +103,7 @@ @Test public void messageWithMDC() throws InterruptedException { fireServer(); + waitForServerToStart(); configureClient(); Logger logger = lc.getLogger(LoggerContext.ROOT_NAME); @@ -111,11 +111,9 @@ MDC.put("key", "testValue"); logger.debug("test msg"); - // Wait max 2 seconds for mock server to finish. However, it should - // finish much sooner than that. - Thread.sleep(100); + Thread.sleep(SLEEP_AFTER_LOG); simpleSocketServer.close(); - simpleSocketServer.join(2000); + simpleSocketServer.join(JOIN__OR_WAIT_TIMEOUT); assertTrue(simpleSocketServer.isClosed()); ListAppender<LoggingEvent> la = getListAppender(); assertEquals(1, la.list.size()); @@ -124,23 +122,23 @@ Map<String, String> MDCPropertyMap = remoteEvent.getMDCPropertyMap(); assertEquals("testValue", MDCPropertyMap.get("key")); } - + @Test public void messageWithMarker() throws InterruptedException { fireServer(); + waitForServerToStart(); + + //Thread.sleep(SLEEP_AFTER_SERVER_START); configureClient(); Logger logger = lc.getLogger(LoggerContext.ROOT_NAME); - + Marker marker = MarkerFactory.getMarker("testMarker"); logger.debug(marker, "test msg"); + Thread.sleep(SLEEP_AFTER_LOG); - Thread.sleep(100); - - // Wait max 2 seconds for mock server to finish. However, it should - // finish much sooner than that. simpleSocketServer.close(); - simpleSocketServer.join(2000); + simpleSocketServer.join(JOIN__OR_WAIT_TIMEOUT); assertTrue(simpleSocketServer.isClosed()); assertEquals(1, la.list.size()); @@ -151,6 +149,8 @@ @Test public void messageWithUpdatedMDC() throws InterruptedException { fireServer(); + waitForServerToStart(); + configureClient(); Logger logger = lc.getLogger(LoggerContext.ROOT_NAME); @@ -160,16 +160,13 @@ MDC.put("key", "updatedTestValue"); logger.debug("test msg 2"); + Thread.sleep(SLEEP_AFTER_LOG); - Thread.sleep(100); - - // Wait max 2 seconds for mock server to finish. However, it should - // finish much sooner than that. simpleSocketServer.close(); - simpleSocketServer.join(2000); + simpleSocketServer.join(JOIN__OR_WAIT_TIMEOUT); assertTrue(simpleSocketServer.isClosed()); ListAppender<LoggingEvent> la = getListAppender(); - + assertEquals(2, la.list.size()); // We observe the second logging event. It should provide us with @@ -187,16 +184,12 @@ logger.debug("test msg"); fireServer(); - synchronized (simpleSocketServer) { - simpleSocketServer.wait(1000); - } - Thread.yield(); // give the server a little more time - Thread.yield(); + waitForServerToStart(); logger.debug("test msg 2"); - Thread.yield(); // give the server a little more time + Thread.sleep(SLEEP_AFTER_LOG); simpleSocketServer.close(); - simpleSocketServer.join(2000); + simpleSocketServer.join(JOIN__OR_WAIT_TIMEOUT); StatusPrinter.print(lc); assertTrue(simpleSocketServer.isClosed()); assertEquals(1, la.list.size()); @@ -205,7 +198,13 @@ assertEquals("test msg 2", remoteEvent.getMessage()); assertEquals(Level.DEBUG, remoteEvent.getLevel()); } - + + private void waitForServerToStart() throws InterruptedException { + synchronized (simpleSocketServer) { + simpleSocketServer.wait(JOIN__OR_WAIT_TIMEOUT); + } + } + private void fireServer() throws InterruptedException { Logger root = serverLC.getLogger("root"); la.setName(LIST_APPENDER_NAME); @@ -217,12 +216,11 @@ Thread.yield(); } - ListAppender<LoggingEvent> getListAppender() { Logger root = serverLC.getLogger("root"); return (ListAppender<LoggingEvent>) root.getAppender(LIST_APPENDER_NAME); } - + private void configureClient() { lc = new LoggerContext(); lc.setName("test");

1 0

svn commit: r1729 - in logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic: . util
by noreply.ceki＠qos.ch 04 Aug '08

04 Aug '08

Author: ceki Date: Mon Aug 4 22:15:53 2008 New Revision: 1729 Modified: logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/LoggerPerfTest.java logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/util/BogoPerf.java Log: - adjust numbers according to the performance of the host CPU Modified: logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/LoggerPerfTest.java ============================================================================== --- logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/LoggerPerfTest.java (original) +++ logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/LoggerPerfTest.java Mon Aug 4 22:15:53 2008 @@ -24,25 +24,25 @@ String localhostName = null; static long NORMAL_RUN_LENGTH = 1000*1000; - static long REFERENCE_BIPS = 10000; + static long REFERENCE_BIPS = 9324; public void setUp() throws Exception { localhostName = InetAddress.getLocalHost().getCanonicalHostName(); } public void testBasic() { - loopBasic(NORMAL_RUN_LENGTH); - double avg = loopBasic(NORMAL_RUN_LENGTH); + basicDurationInNanos(NORMAL_RUN_LENGTH); + double avg = basicDurationInNanos(NORMAL_RUN_LENGTH); long referencePerf = 17; - BogoPerf.assertPerformance(avg, referencePerf, REFERENCE_BIPS); + BogoPerf.assertDuration(avg, referencePerf, REFERENCE_BIPS); //System.out.println("Average log time for disabled statements: "+avg+" nanos."); } public void testParameterized() { - loopBasic(NORMAL_RUN_LENGTH); - double avg = loopParameterized(NORMAL_RUN_LENGTH); + basicDurationInNanos(NORMAL_RUN_LENGTH); + double avgDuration = loopParameterized(NORMAL_RUN_LENGTH); long referencePerf = 36; - BogoPerf.assertPerformance(avg, referencePerf, REFERENCE_BIPS); + BogoPerf.assertDuration(avgDuration, referencePerf, REFERENCE_BIPS); //System.out.println("Average log time for disabled (parameterized) statements: "+avg+" nanos."); } @@ -52,10 +52,10 @@ double avg = loopNopFilter(NORMAL_RUN_LENGTH); //System.out.println("Average log time for disabled (NOPFilter) statements: "+avg+" nanos."); long referencePerf = 48; - BogoPerf.assertPerformance(avg, referencePerf, REFERENCE_BIPS); + BogoPerf.assertDuration(avg, referencePerf, REFERENCE_BIPS); } - double loopBasic(long len) { + double basicDurationInNanos(long len) { LoggerContext lc = new LoggerContext(); NOPAppender<LoggingEvent> mopAppender = new NOPAppender<LoggingEvent>(); mopAppender.start(); Modified: logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/util/BogoPerf.java ============================================================================== --- logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/util/BogoPerf.java (original) +++ logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/util/BogoPerf.java Mon Aug 4 22:15:53 2008 @@ -19,7 +19,7 @@ } /** - * compute bogoInstructions per second + * Compute bogoInstructions per second * * on a 3.2 Ghz Pentium D CPU (around 2007), we obtain about 10'000 bogoIPS. * @@ -69,20 +69,54 @@ return (2 * lastBogos + INITIAL_BOGO_IPS) / 3; } + /** + * Assertion used for values that decrease with faster CPUs, + * typically the time (duration) needed to perform a task. + * + * @param currentDuration + * @param referenceDuraion + * @param referenceBIPS + * @throws AssertionFailedError + */ + public static void assertDuration(double currentDuration, + long referenceDuraion, double referenceBIPS) + throws AssertionFailedError { + double ajustedDuration = adjustExpectedDuration(referenceDuraion, + referenceBIPS); + if (currentDuration > ajustedDuration * SLACK) { + throw new AssertionFailedError(currentDuration + " exceeded expected " + + ajustedDuration + " (adjusted), " + referenceDuraion + " (raw)"); + } + } + /** + * Assertion used for values that increase with faster CPUs, typically + * the number of operations accomplished per unit of time. + * + * @param currentPerformance + * @param referencePerformance + * @param referenceBIPS + * @throws AssertionFailedError + */ public static void assertPerformance(double currentPerformance, - long referencePerf, double referenceBIPS) + long referencePerformance, double referenceBIPS) throws AssertionFailedError { - double ajustedPerf = adjustExpectedValue(referencePerf, + double ajustedPerf = adjustExpectedPerformance(referencePerformance, referenceBIPS); - if (currentPerformance > ajustedPerf * SLACK) { - throw new AssertionFailedError(currentPerformance + " exceeded expected " - + ajustedPerf + " (adjusted), " + referencePerf + " (raw)"); + if (currentPerformance*SLACK < ajustedPerf) { + throw new AssertionFailedError(currentPerformance + " below expected " + + ajustedPerf + " (adjusted), " + referencePerformance + " (raw)"); } } - - private static double adjustExpectedValue(long referencePerf, + + private static double adjustExpectedPerformance(long referenceDuration, + double referenceBIPS) { + double currentBIPS = currentBIPS(); + return referenceDuration * (currentBIPS/referenceBIPS); + } + + private static double adjustExpectedDuration(long referenceDuration, double referenceBIPS) { double currentBIPS = currentBIPS(); - return referencePerf * (referenceBIPS / currentBIPS); + return referenceDuration * (referenceBIPS / currentBIPS); } }

1 0

svn commit: r1728 - in logback/trunk/logback-classic: . src/test/java/ch/qos/logback/classic src/test/java/ch/qos/logback/classic/util
by noreply.ceki＠qos.ch 04 Aug '08

04 Aug '08

Author: ceki Date: Mon Aug 4 19:00:40 2008 New Revision: 1728 Added: logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/util/BogoPerf.java Modified: logback/trunk/logback-classic/pom.xml logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/LoggerPerfTest.java logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/PackageTest.java Log: - performance test are not adjusted according to the performance of the host CPU Modified: logback/trunk/logback-classic/pom.xml ============================================================================== --- logback/trunk/logback-classic/pom.xml (original) +++ logback/trunk/logback-classic/pom.xml Mon Aug 4 19:00:40 2008 @@ -140,7 +140,6 @@ <exclude>**/PackageTest.java</exclude> <exclude>**/TestConstants.java</exclude> <exclude>**/SerializationPerfsTest.java</exclude> - <exclude>**/LoggerPerfTest.java</exclude> </excludes> </configuration> </plugin> Modified: logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/LoggerPerfTest.java ============================================================================== --- logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/LoggerPerfTest.java (original) +++ logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/LoggerPerfTest.java Mon Aug 4 19:00:40 2008 @@ -15,54 +15,44 @@ import junit.framework.TestCase; import ch.qos.logback.classic.spi.LoggingEvent; import ch.qos.logback.classic.turbo.NOPTurboFilter; +import ch.qos.logback.classic.util.BogoPerf; import ch.qos.logback.core.appender.NOPAppender; public class LoggerPerfTest extends TestCase { final static String KAL = "kal"; String localhostName = null; - static long RUN_LENGTH = 1000*1000*10; + static long NORMAL_RUN_LENGTH = 1000*1000; + + static long REFERENCE_BIPS = 10000; public void setUp() throws Exception { localhostName = InetAddress.getLocalHost().getCanonicalHostName(); } public void testBasic() { - loopBasic(RUN_LENGTH); - double avg = loopBasic(RUN_LENGTH); + loopBasic(NORMAL_RUN_LENGTH); + double avg = loopBasic(NORMAL_RUN_LENGTH); - System.out.println("Running on "+localhostName); - // check for performance on KAL only - if(KAL.equals(localhostName)) { - assertTrue(30 > avg); - } - System.out.println("Average log time for disabled statements: "+avg+" nanos."); + long referencePerf = 17; + BogoPerf.assertPerformance(avg, referencePerf, REFERENCE_BIPS); + //System.out.println("Average log time for disabled statements: "+avg+" nanos."); } public void testParameterized() { - - loopBasic(RUN_LENGTH); - double avg = loopParameterized(RUN_LENGTH); - - System.out.println("Running on "+localhostName); - // check for performance on KAL only - if(KAL.equals(localhostName)) { - assertTrue(30 > avg); - } - System.out.println("Average log time for disabled (parameterized) statements: "+avg+" nanos."); + loopBasic(NORMAL_RUN_LENGTH); + double avg = loopParameterized(NORMAL_RUN_LENGTH); + long referencePerf = 36; + BogoPerf.assertPerformance(avg, referencePerf, REFERENCE_BIPS); + //System.out.println("Average log time for disabled (parameterized) statements: "+avg+" nanos."); } public void testNOPFilter() { - loopNopFilter(RUN_LENGTH); - double avg = loopNopFilter(RUN_LENGTH); - - System.out.println("Running on "+localhostName); - System.out.println("Average log time for disabled (NOPFilter) statements: "+avg+" nanos."); - // check for performance on KAL only - if(KAL.equals(localhostName)) { - assertTrue(80 > avg); - } - + loopNopFilter(NORMAL_RUN_LENGTH); + double avg = loopNopFilter(NORMAL_RUN_LENGTH); + //System.out.println("Average log time for disabled (NOPFilter) statements: "+avg+" nanos."); + long referencePerf = 48; + BogoPerf.assertPerformance(avg, referencePerf, REFERENCE_BIPS); } double loopBasic(long len) { Modified: logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/PackageTest.java ============================================================================== --- logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/PackageTest.java (original) +++ logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/PackageTest.java Mon Aug 4 19:00:40 2008 @@ -16,6 +16,7 @@ public static Test suite() { TestSuite suite = new TestSuite(); suite.addTestSuite(LoggerContextTest.class); + suite.addTestSuite(LoggerPerfTest.class); suite.addTestSuite(DynamicLoggerContextTest.class); suite.addTestSuite(PatternLayoutTest.class); suite.addTestSuite(BasicLoggerTest.class); Added: logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/util/BogoPerf.java ============================================================================== --- (empty file) +++ logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/util/BogoPerf.java Mon Aug 4 19:00:40 2008 @@ -0,0 +1,88 @@ +package ch.qos.logback.classic.util; + +import java.util.Arrays; +import java.util.Random; + +import junit.framework.AssertionFailedError; + +public class BogoPerf { + + private static long NANOS_IN_ONE_SECOND = 1000 * 1000 * 1000; + private static int INITIAL_N = 1000; + private static int LAST_N = 100; + private static double INITIAL_BOGO_IPS = computeBogoIPS(INITIAL_N); + + private static int SLACK = 2; + + static { + System.out.println("Host runs at "+INITIAL_BOGO_IPS + " BIPS"); + } + + /** + * compute bogoInstructions per second + * + * on a 3.2 Ghz Pentium D CPU (around 2007), we obtain about 10'000 bogoIPS. + * + * @param N + * number of bogoInstructions to average over in order to + * compute the result + * @return bogo Instructions Per Second + */ + private static double computeBogoIPS(int N) { + long begin = System.nanoTime(); + + for (int i = 0; i < N; i++) { + bogoInstruction(); + } + long end = System.nanoTime(); + + // duration + double D = end - begin; + // average duration per instruction + double avgDPIS = D / N; + // System.out.println(D + " nanos for " + N + " instructions"); + // System.out.println(avgD + " nanos per instruction"); + + double bogoIPS = NANOS_IN_ONE_SECOND / avgDPIS; + // System.out.println(bogoIPS + " bogoIPS"); + + return bogoIPS; + } + + private static void bogoInstruction() { + Random random = new Random(100); + int len = 500; + int[] intArray = new int[len]; + for (int i = 0; i < len; i++) { + intArray[i] = random.nextInt(); + } + Arrays.sort(intArray); + } + + /** + * Computed the BogoIPS for this host CPU. + * + * @return + */ + public static double currentBIPS() { + double lastBogos = computeBogoIPS(LAST_N); + return (2 * lastBogos + INITIAL_BOGO_IPS) / 3; + } + + public static void assertPerformance(double currentPerformance, + long referencePerf, double referenceBIPS) + throws AssertionFailedError { + double ajustedPerf = adjustExpectedValue(referencePerf, + referenceBIPS); + if (currentPerformance > ajustedPerf * SLACK) { + throw new AssertionFailedError(currentPerformance + " exceeded expected " + + ajustedPerf + " (adjusted), " + referencePerf + " (raw)"); + } + } + + private static double adjustExpectedValue(long referencePerf, + double referenceBIPS) { + double currentBIPS = currentBIPS(); + return referencePerf * (referenceBIPS / currentBIPS); + } +}

1 0

benchmarking extension to junit?
by Ceki Gulcu 04 Aug '08

04 Aug '08

Hello all, Logback contains a small number of time-sensitive tests. The time needed to execute these tests depend on the capabilities of the host CPU. For example, if on my machine a test executes in 1 millisecond, I would expect it to finish in say 1*10 milliseconds on most machine -- allowing for 10 fold variation. Does anyone know of a junit extension which can help me normalize the coefficient, 10 in the previous example, so that it adapts to the speed of the host CPU? There is also the problem of the JIT compiler... Any recommendations on the subject? -- Ceki Gülcü

4 5

svn commit: r1727 - logback/trunk/logback-examples/src/main/java/chapter8
by noreply.ceki＠qos.ch 29 Jul '08

29 Jul '08

Author: ceki Date: Tue Jul 29 13:32:23 2008 New Revision: 1727 Removed: logback/trunk/logback-examples/src/main/java/chapter8/ Log: - profiler code moved to slf4j

1 0

svn commit: r1726 - in logback/trunk: . logback-core/src/test/java/ch/qos/logback/core/rolling logback-core/src/test/java/ch/qos/logback/core/rolling/helper logback-core/src/test/java/ch/qos/logback/core/util logback-examples logback-site/src/site/pages
by noreply.ceki＠qos.ch 28 Jul '08

28 Jul '08

Author: ceki Date: Mon Jul 28 23:42:22 2008 New Revision: 1726 Modified: logback/trunk/logback-core/src/test/java/ch/qos/logback/core/rolling/RenamingTest.java logback/trunk/logback-core/src/test/java/ch/qos/logback/core/rolling/SizeBasedRollingTest.java logback/trunk/logback-core/src/test/java/ch/qos/logback/core/rolling/TimeBasedRollingTest.java logback/trunk/logback-core/src/test/java/ch/qos/logback/core/rolling/helper/CompressTest.java logback/trunk/logback-core/src/test/java/ch/qos/logback/core/util/Constants.java logback/trunk/logback-examples/pom.xml logback/trunk/logback-site/src/site/pages/news.html logback/trunk/pom.xml Log: - test from the c.q.l.core.rolling package send their output to files under the target directory. See also http://www.qos.ch/pipermail/logback-dev/2008-July/002610.html - logback now depends on slf4j version 5.1.3-snapshot - lb-examples depends on slf4j-ext Modified: logback/trunk/logback-core/src/test/java/ch/qos/logback/core/rolling/RenamingTest.java ============================================================================== --- logback/trunk/logback-core/src/test/java/ch/qos/logback/core/rolling/RenamingTest.java (original) +++ logback/trunk/logback-core/src/test/java/ch/qos/logback/core/rolling/RenamingTest.java Mon Jul 28 23:42:22 2008 @@ -43,7 +43,7 @@ super.setUp(); layout = new EchoLayout<Object>(); - File target = new File(Constants.TEST_DIR_PREFIX + "output/test.log"); + File target = new File(Constants.OUTPUT_DIR_PREFIX + "test.log"); target.mkdirs(); target.delete(); } @@ -57,7 +57,7 @@ RollingFileAppender<Object> rfa = new RollingFileAppender<Object>(); rfa.setLayout(layout); rfa.setContext(context); - rfa.setFile(Constants.TEST_DIR_PREFIX + "output/test.log"); + rfa.setFile(Constants.OUTPUT_DIR_PREFIX + "test.log"); // rollover by the second String datePattern = "yyyy-MM-dd_HH_mm_ss"; @@ -65,7 +65,7 @@ String[] filenames = new String[2]; TimeBasedRollingPolicy tbrp = new TimeBasedRollingPolicy(); - tbrp.setFileNamePattern(Constants.TEST_DIR_PREFIX + "output/test-%d{" + datePattern + "}"); + tbrp.setFileNamePattern(Constants.OUTPUT_DIR_PREFIX + "test-%d{" + datePattern + "}"); //tbrp.setActiveFileName("src/test/output/test.log"); tbrp.setContext(context); tbrp.setParent(rfa); @@ -81,8 +81,8 @@ DelayerUtil.delayUntilNextSecond(50); rfa.doAppend("Hello 1"); - filenames[0] = Constants.TEST_DIR_PREFIX + "output/test-" + sdf.format(cal.getTime()); - filenames[1] = Constants.TEST_DIR_PREFIX + "output/test.log"; + filenames[0] = Constants.OUTPUT_DIR_PREFIX + "test-" + sdf.format(cal.getTime()); + filenames[1] = Constants.OUTPUT_DIR_PREFIX + "test.log"; for (int i = 0; i < filenames.length; i++) { //System.out.println("before i=" + i); Modified: logback/trunk/logback-core/src/test/java/ch/qos/logback/core/rolling/SizeBasedRollingTest.java ============================================================================== --- logback/trunk/logback-core/src/test/java/ch/qos/logback/core/rolling/SizeBasedRollingTest.java (original) +++ logback/trunk/logback-core/src/test/java/ch/qos/logback/core/rolling/SizeBasedRollingTest.java Mon Jul 28 23:42:22 2008 @@ -44,13 +44,13 @@ public void setUp() { { - File target = new File(Constants.TEST_DIR_PREFIX - + "output/sizeBased-test2.log"); + File target = new File(Constants.OUTPUT_DIR_PREFIX + + "sizeBased-test2.log"); target.mkdirs(); target.delete(); } { - File target = new File(Constants.TEST_DIR_PREFIX + "output/sbr-test3.log"); + File target = new File(Constants.OUTPUT_DIR_PREFIX + "sbr-test3.log"); target.mkdirs(); target.delete(); } @@ -80,8 +80,8 @@ sbtp.setMaxFileSize("100"); sbtp.start(); - fwrp.setFileNamePattern(Constants.TEST_DIR_PREFIX - + "output/sizeBased-test1.%i"); + fwrp.setFileNamePattern(Constants.OUTPUT_DIR_PREFIX + + "sizeBased-test1.%i"); try { fwrp.start(); fail("The absence of activeFileName option should have caused an exception."); @@ -103,8 +103,8 @@ rfa.setName("ROLLING"); rfa.setLayout(layout); rfa.setContext(context); - rfa.setFile(Constants.TEST_DIR_PREFIX - + "output/sizeBased-test2.log"); + rfa.setFile(Constants.OUTPUT_DIR_PREFIX + + "sizeBased-test2.log"); FixedWindowRollingPolicy swrp = new FixedWindowRollingPolicy(); swrp.setContext(context); @@ -116,8 +116,8 @@ // swrp.setActiveFileName(Constants.TEST_DIR_PREFIX // + "output/sizeBased-test2.log"); - swrp.setFileNamePattern(Constants.TEST_DIR_PREFIX - + "output/sizeBased-test2.%i"); + swrp.setFileNamePattern(Constants.OUTPUT_DIR_PREFIX + + "sizeBased-test2.%i"); swrp.setParent(rfa); swrp.start(); @@ -142,25 +142,25 @@ rfa.doAppend("hello"); } - assertTrue(new File(Constants.TEST_DIR_PREFIX - + "output/sizeBased-test2.log").exists()); - assertTrue(new File(Constants.TEST_DIR_PREFIX + "output/sizeBased-test2.0") + assertTrue(new File(Constants.OUTPUT_DIR_PREFIX + + "sizeBased-test2.log").exists()); + assertTrue(new File(Constants.OUTPUT_DIR_PREFIX + "sizeBased-test2.0") .exists()); - assertTrue(new File(Constants.TEST_DIR_PREFIX + "output/sizeBased-test2.1") + assertTrue(new File(Constants.OUTPUT_DIR_PREFIX + "sizeBased-test2.1") .exists()); // The File.length() method is not accurate under Windows if (!isWindows()) { - assertTrue(Compare.compare(Constants.TEST_DIR_PREFIX - + "output/sizeBased-test2.log", Constants.TEST_DIR_PREFIX + assertTrue(Compare.compare(Constants.OUTPUT_DIR_PREFIX + + "sizeBased-test2.log", Constants.TEST_DIR_PREFIX + "witness/rolling/sbr-test2.l")); - assertTrue(Compare.compare(Constants.TEST_DIR_PREFIX - + "output/sizeBased-test2.0", Constants.TEST_DIR_PREFIX + assertTrue(Compare.compare(Constants.OUTPUT_DIR_PREFIX + + "sizeBased-test2.0", Constants.TEST_DIR_PREFIX + "witness/rolling/sbr-test2.0")); - assertTrue(Compare.compare(Constants.TEST_DIR_PREFIX - + "output/sizeBased-test2.1", Constants.TEST_DIR_PREFIX + assertTrue(Compare.compare(Constants.OUTPUT_DIR_PREFIX + + "sizeBased-test2.1", Constants.TEST_DIR_PREFIX + "witness/rolling/sbr-test2.1")); } @@ -176,7 +176,7 @@ RollingFileAppender<Object> rfa = new RollingFileAppender<Object>(); rfa.setLayout(layout); rfa.setContext(context); - rfa.setFile(Constants.TEST_DIR_PREFIX + "output/sbr-test3.log"); + rfa.setFile(Constants.OUTPUT_DIR_PREFIX + "sbr-test3.log"); FixedWindowRollingPolicy fwrp = new FixedWindowRollingPolicy(); fwrp.setContext(context); @@ -186,8 +186,8 @@ sbtp.setMaxFileSize("100"); fwrp.setMinIndex(0); //fwrp.setActiveFileName(Constants.TEST_DIR_PREFIX + "output/sbr-test3.log"); - fwrp.setFileNamePattern(Constants.TEST_DIR_PREFIX - + "output/sbr-test3.%i.gz"); + fwrp.setFileNamePattern(Constants.OUTPUT_DIR_PREFIX + + "sbr-test3.%i.gz"); fwrp.setParent(rfa); fwrp.start(); rfa.setRollingPolicy(fwrp); @@ -211,11 +211,11 @@ rfa.doAppend("hello"); } - assertTrue(new File(Constants.TEST_DIR_PREFIX + "output/sbr-test3.log") + assertTrue(new File(Constants.OUTPUT_DIR_PREFIX + "sbr-test3.log") .exists()); - assertTrue(new File(Constants.TEST_DIR_PREFIX + "output/sbr-test3.0.gz") + assertTrue(new File(Constants.OUTPUT_DIR_PREFIX + "sbr-test3.0.gz") .exists()); - assertTrue(new File(Constants.TEST_DIR_PREFIX + "output/sbr-test3.1.gz") + assertTrue(new File(Constants.OUTPUT_DIR_PREFIX + "sbr-test3.1.gz") .exists()); if (!isWindows()) { Modified: logback/trunk/logback-core/src/test/java/ch/qos/logback/core/rolling/TimeBasedRollingTest.java ============================================================================== --- logback/trunk/logback-core/src/test/java/ch/qos/logback/core/rolling/TimeBasedRollingTest.java (original) +++ logback/trunk/logback-core/src/test/java/ch/qos/logback/core/rolling/TimeBasedRollingTest.java Mon Jul 28 23:42:22 2008 @@ -60,17 +60,17 @@ context.setName("test"); // Delete .log files { - File target = new File(Constants.TEST_DIR_PREFIX + "output/test4.log"); + File target = new File(Constants.OUTPUT_DIR_PREFIX + "test4.log"); target.mkdirs(); target.delete(); } { - File target = new File(Constants.TEST_DIR_PREFIX + "output/test5.log"); + File target = new File(Constants.OUTPUT_DIR_PREFIX + "test5.log"); target.mkdirs(); target.delete(); } { - File target = new File(Constants.TEST_DIR_PREFIX + "output/test6.log"); + File target = new File(Constants.OUTPUT_DIR_PREFIX + "test6.log"); target.mkdirs(); target.delete(); } @@ -93,7 +93,7 @@ TimeBasedRollingPolicy tbrp = new TimeBasedRollingPolicy(); tbrp.setContext(context); - tbrp.setFileNamePattern(Constants.TEST_DIR_PREFIX + "output/test1-%d{" + tbrp.setFileNamePattern(Constants.OUTPUT_DIR_PREFIX + "test1-%d{" + DATE_PATTERN + "}"); tbrp.setParent(rfa); tbrp.setCurrentTime(currentTime); @@ -105,7 +105,7 @@ String[] filenames = new String[3]; for (int i = 0; i < 3; i++) { - filenames[i] = Constants.TEST_DIR_PREFIX + "output/test1-" + filenames[i] = Constants.OUTPUT_DIR_PREFIX + "test1-" + sdf.format(cal.getTime()); cal.add(Calendar.SECOND, 1); } @@ -143,7 +143,7 @@ rfa1.setLayout(layout); TimeBasedRollingPolicy tbrp1 = new TimeBasedRollingPolicy(); - tbrp1.setFileNamePattern(Constants.TEST_DIR_PREFIX + "output/test2-%d{" + tbrp1.setFileNamePattern(Constants.OUTPUT_DIR_PREFIX + "test2-%d{" + DATE_PATTERN + "}"); tbrp1.setContext(context); tbrp1.setParent(rfa1); @@ -156,7 +156,7 @@ String[] filenames = new String[3]; for (int i = 0; i < 3; i++) { - filenames[i] = Constants.TEST_DIR_PREFIX + "output/test2-" + filenames[i] = Constants.OUTPUT_DIR_PREFIX + "test2-" + sdf.format(cal.getTime()); cal.add(Calendar.SECOND, 1); } @@ -179,7 +179,7 @@ TimeBasedRollingPolicy tbrp2 = new TimeBasedRollingPolicy(); tbrp2.setContext(context); - tbrp2.setFileNamePattern(Constants.TEST_DIR_PREFIX + "output/test2-%d{" + tbrp2.setFileNamePattern(Constants.OUTPUT_DIR_PREFIX + "test2-%d{" + DATE_PATTERN + "}"); tbrp2.setParent(rfa2); tbrp2.setCurrentTime(tbrp1.getCurrentTime()); @@ -213,7 +213,7 @@ TimeBasedRollingPolicy tbrp = new TimeBasedRollingPolicy(); tbrp.setContext(context); - tbrp.setFileNamePattern(Constants.TEST_DIR_PREFIX + "output/test3-%d{" + tbrp.setFileNamePattern(Constants.OUTPUT_DIR_PREFIX + "test3-%d{" + DATE_PATTERN + "}.gz"); tbrp.setParent(rfa); tbrp.setCurrentTime(currentTime); @@ -225,12 +225,12 @@ String[] filenames = new String[4]; for (int i = 0; i < 2; i++) { - filenames[i] = Constants.TEST_DIR_PREFIX + "output/test3-" + filenames[i] = Constants.OUTPUT_DIR_PREFIX + "test3-" + sdf.format(cal.getTime()) + ".gz"; cal.add(Calendar.SECOND, 1); } - filenames[3] = Constants.TEST_DIR_PREFIX + "output/test3-" + filenames[3] = Constants.OUTPUT_DIR_PREFIX + "test3-" + sdf.format(cal.getTime()); // System.out.println("Waiting until next second and 100 millis."); @@ -264,12 +264,12 @@ RollingFileAppender<Object> rfa1 = new RollingFileAppender<Object>(); rfa1.setContext(context); rfa1.setLayout(layout); - rfa1.setFile(Constants.TEST_DIR_PREFIX + "output/test4.log"); + rfa1.setFile(Constants.OUTPUT_DIR_PREFIX + "test4.log"); TimeBasedRollingPolicy tbrp1 = new TimeBasedRollingPolicy(); tbrp1.setContext(context); // tbrp1.setActiveFileName(Constants.TEST_DIR_PREFIX + "output/test4.log"); - tbrp1.setFileNamePattern(Constants.TEST_DIR_PREFIX + "output/test4-%d{" + tbrp1.setFileNamePattern(Constants.OUTPUT_DIR_PREFIX + "test4-%d{" + DATE_PATTERN + "}"); tbrp1.setParent(rfa1); tbrp1.setCurrentTime(currentTime); @@ -281,11 +281,11 @@ String[] filenames = new String[3]; for (int i = 0; i < 2; i++) { - filenames[i] = Constants.TEST_DIR_PREFIX + "output/test4-" + filenames[i] = Constants.OUTPUT_DIR_PREFIX + "test4-" + sdf.format(cal.getTime()); cal.add(Calendar.SECOND, 1); } - filenames[2] = Constants.TEST_DIR_PREFIX + "output/test4.log"; + filenames[2] = Constants.OUTPUT_DIR_PREFIX + "test4.log"; // System.out.println("Waiting until next second and 100 millis."); tbrp1.setCurrentTime(addTime(currentTime, 1100)); @@ -301,11 +301,11 @@ RollingFileAppender<Object> rfa2 = new RollingFileAppender<Object>(); rfa2.setContext(context); rfa2.setLayout(layout); - rfa2.setFile(Constants.TEST_DIR_PREFIX + "output/test4.log"); + rfa2.setFile(Constants.OUTPUT_DIR_PREFIX + "test4.log"); TimeBasedRollingPolicy tbrp2 = new TimeBasedRollingPolicy(); tbrp2.setContext(context); - tbrp2.setFileNamePattern(Constants.TEST_DIR_PREFIX + "output/test4-%d{" + tbrp2.setFileNamePattern(Constants.OUTPUT_DIR_PREFIX + "test4-%d{" + DATE_PATTERN + "}"); // tbrp2.setActiveFileName(Constants.TEST_DIR_PREFIX + "output/test4.log"); tbrp2.setParent(rfa2); @@ -336,11 +336,11 @@ RollingFileAppender<Object> rfa = new RollingFileAppender<Object>(); rfa.setContext(context); rfa.setLayout(layout); - rfa.setFile(Constants.TEST_DIR_PREFIX + "output/test5.log"); + rfa.setFile(Constants.OUTPUT_DIR_PREFIX + "test5.log"); TimeBasedRollingPolicy tbrp = new TimeBasedRollingPolicy(); tbrp.setContext(context); - tbrp.setFileNamePattern(Constants.TEST_DIR_PREFIX + "output/test5-%d{" + tbrp.setFileNamePattern(Constants.OUTPUT_DIR_PREFIX + "test5-%d{" + DATE_PATTERN + "}"); // tbrp.setActiveFileName(Constants.TEST_DIR_PREFIX + "output/test5.log"); tbrp.setParent(rfa); @@ -353,12 +353,12 @@ String[] filenames = new String[4]; for (int i = 0; i < 2; i++) { - filenames[i] = Constants.TEST_DIR_PREFIX + "output/test5-" + filenames[i] = Constants.OUTPUT_DIR_PREFIX + "test5-" + sdf.format(cal.getTime()); cal.add(Calendar.SECOND, 1); } - filenames[2] = Constants.TEST_DIR_PREFIX + "output/test5.log"; + filenames[2] = Constants.OUTPUT_DIR_PREFIX + "test5.log"; // System.out.println("Waiting until next second and 100 millis."); tbrp.setCurrentTime(addTime(currentTime, 1100)); @@ -386,11 +386,11 @@ RollingFileAppender<Object> rfa = new RollingFileAppender<Object>(); rfa.setContext(context); rfa.setLayout(layout); - rfa.setFile(Constants.TEST_DIR_PREFIX + "output/test6.log"); + rfa.setFile(Constants.OUTPUT_DIR_PREFIX+ "test6.log"); TimeBasedRollingPolicy tbrp = new TimeBasedRollingPolicy(); tbrp.setContext(context); - tbrp.setFileNamePattern(Constants.TEST_DIR_PREFIX + "output/test6-%d{" + tbrp.setFileNamePattern(Constants.OUTPUT_DIR_PREFIX + "test6-%d{" + DATE_PATTERN + "}.gz"); // tbrp.setActiveFileName(Constants.TEST_DIR_PREFIX + "output/test6.log"); tbrp.setParent(rfa); @@ -403,12 +403,12 @@ String[] filenames = new String[4]; for (int i = 0; i < 2; i++) { - filenames[i] = Constants.TEST_DIR_PREFIX + "output/test6-" + filenames[i] = Constants.OUTPUT_DIR_PREFIX + "test6-" + sdf.format(cal.getTime()) + ".gz"; cal.add(Calendar.SECOND, 1); } - filenames[2] = Constants.TEST_DIR_PREFIX + "output/test6.log"; + filenames[2] = Constants.OUTPUT_DIR_PREFIX + "test6.log"; // System.out.println("Waiting until next second and 100 millis."); tbrp.setCurrentTime(addTime(currentTime, 1100)); Modified: logback/trunk/logback-core/src/test/java/ch/qos/logback/core/rolling/helper/CompressTest.java ============================================================================== --- logback/trunk/logback-core/src/test/java/ch/qos/logback/core/rolling/helper/CompressTest.java (original) +++ logback/trunk/logback-core/src/test/java/ch/qos/logback/core/rolling/helper/CompressTest.java Mon Jul 28 23:42:22 2008 @@ -38,8 +38,8 @@ File dest = new File(Constants.TEST_DIR_PREFIX + "input/compress1.txt"); copy(source, dest); - File target = new File(Constants.TEST_DIR_PREFIX - + "output/compress1.txt.gz"); + File target = new File(Constants.OUTPUT_DIR_PREFIX + + "compress1.txt.gz"); target.mkdirs(); target.delete(); } @@ -47,8 +47,8 @@ File source = new File(Constants.TEST_DIR_PREFIX + "input/compress2.copy"); File dest = new File(Constants.TEST_DIR_PREFIX + "input/compress2.txt"); copy(source, dest); - File target = new File(Constants.TEST_DIR_PREFIX - + "output/compress2.txt.gz"); + File target = new File(Constants.OUTPUT_DIR_PREFIX + + "compress2.txt.gz"); target.mkdirs(); target.delete(); } @@ -56,8 +56,8 @@ File source = new File(Constants.TEST_DIR_PREFIX + "input/compress3.copy"); File dest = new File(Constants.TEST_DIR_PREFIX + "input/compress3.txt"); copy(source, dest); - File target = new File(Constants.TEST_DIR_PREFIX - + "output/compress3.txt.zip"); + File target = new File(Constants.OUTPUT_DIR_PREFIX + + "compress3.txt.zip"); target.mkdirs(); target.delete(); } @@ -69,35 +69,35 @@ public void test1() throws Exception { Compressor compressor = new Compressor(CompressionMode.GZ, Constants.TEST_DIR_PREFIX + "input/compress1.txt", - Constants.TEST_DIR_PREFIX + "output/compress1.txt.gz"); + Constants.OUTPUT_DIR_PREFIX + "compress1.txt.gz"); compressor.setContext(context); compressor.compress(); StatusPrinter.print(context.getStatusManager()); assertEquals(0, context.getStatusManager().getCount()); - assertTrue(Compare.gzCompare(Constants.TEST_DIR_PREFIX - + "output/compress1.txt.gz", Constants.TEST_DIR_PREFIX + assertTrue(Compare.gzCompare(Constants.OUTPUT_DIR_PREFIX + + "compress1.txt.gz", Constants.TEST_DIR_PREFIX + "witness/compress1.txt.gz")); } public void test2() throws Exception { Compressor compressor = new Compressor(CompressionMode.GZ, Constants.TEST_DIR_PREFIX + "input/compress2.txt", - Constants.TEST_DIR_PREFIX + "output/compress2.txt"); + Constants.OUTPUT_DIR_PREFIX + "compress2.txt"); compressor.setContext(context); compressor.compress(); StatusPrinter.print(context.getStatusManager()); assertEquals(0, context.getStatusManager().getCount()); - assertTrue(Compare.gzCompare(Constants.TEST_DIR_PREFIX - + "output/compress2.txt.gz", Constants.TEST_DIR_PREFIX + assertTrue(Compare.gzCompare(Constants.OUTPUT_DIR_PREFIX + + "compress2.txt.gz", Constants.TEST_DIR_PREFIX + "witness/compress2.txt.gz")); } public void test3() throws Exception { Compressor compressor = new Compressor(CompressionMode.ZIP, Constants.TEST_DIR_PREFIX + "input/compress3.txt", - Constants.TEST_DIR_PREFIX + "output/compress3.txt"); + Constants.OUTPUT_DIR_PREFIX + "compress3.txt"); compressor.setContext(context); compressor.compress(); StatusPrinter.print(context.getStatusManager()); Modified: logback/trunk/logback-core/src/test/java/ch/qos/logback/core/util/Constants.java ============================================================================== --- logback/trunk/logback-core/src/test/java/ch/qos/logback/core/util/Constants.java (original) +++ logback/trunk/logback-core/src/test/java/ch/qos/logback/core/util/Constants.java Mon Jul 28 23:42:22 2008 @@ -3,16 +3,7 @@ public class Constants { -// static { -// File f = new File(""); -// String path = ""; -// String absolute = f.getAbsolutePath(); -// if (!absolute.endsWith("logback-core")) { -// path = "logback-core/"; -// } -// TEST_DIR_PREFIX = path + "src/test/"; -// } - public static final String TEST_DIR_PREFIX = "src/test/"; - + public static final String OUTPUT_DIR_PREFIX = "target/test-output/"; + } Modified: logback/trunk/logback-examples/pom.xml ============================================================================== --- logback/trunk/logback-examples/pom.xml (original) +++ logback/trunk/logback-examples/pom.xml Mon Jul 28 23:42:22 2008 @@ -48,6 +48,11 @@ </dependency> <dependency> + <groupId>org.slf4j</groupId> + <artifactId>slf4j-ext</artifactId> + </dependency> + + <dependency> <groupId>javax.servlet</groupId> <artifactId>servlet-api</artifactId> <scope>compile</scope> Modified: logback/trunk/logback-site/src/site/pages/news.html ============================================================================== --- logback/trunk/logback-site/src/site/pages/news.html (original) +++ logback/trunk/logback-site/src/site/pages/news.html Mon Jul 28 23:42:22 2008 @@ -55,6 +55,14 @@ developpers list. + <a href="http://jira.qos.ch/browse/LBCORE-32">LBCORE-32</a> + + + + <a href="http://jira.qos.ch/browse/LBCLASSIC-53">LBCLASSIC-53</a> + + + <hr width="80%" align="center" /> Modified: logback/trunk/pom.xml ============================================================================== --- logback/trunk/pom.xml (original) +++ logback/trunk/pom.xml Mon Jul 28 23:42:22 2008 @@ -36,7 +36,7 @@ <properties>  - <slf4j.version>1.5.2</slf4j.version> + <slf4j.version>1.5.3-SNAPSHOT</slf4j.version> <consolePlugin.version>1.1.0</consolePlugin.version> <retrotranslator.verify>false</retrotranslator.verify> </properties> @@ -78,7 +78,12 @@ <artifactId>log4j-over-slf4j</artifactId> <version>${slf4j.version}</version> </dependency> - + <dependency> + <groupId>org.slf4j</groupId> + <artifactId>slf4j-ext</artifactId> + <version>${slf4j.version}</version> + </dependency> + <dependency> <groupId>janino</groupId> <artifactId>janino</artifactId>

1 0

svn commit: r1725 - logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic
by noreply.ceki＠qos.ch 28 Jul '08

28 Jul '08

Author: ceki Date: Mon Jul 28 23:19:41 2008 New Revision: 1725 Modified: logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/AllTest.java Log: - stopwatch code migrated to slf4j Modified: logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/AllTest.java ============================================================================== --- logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/AllTest.java (original) +++ logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/AllTest.java Mon Jul 28 23:19:41 2008 @@ -31,7 +31,6 @@ suite.addTest(ch.qos.logback.classic.db.PackageTest.suite()); suite.addTest(ch.qos.logback.classic.spi.PackageTest.suite()); suite.addTest(ch.qos.logback.classic.turbo.PackageTest.suite()); - suite.addTest(ch.qos.logback.classic.stopwatch.PackageTest.suite()); return suite; }

1 0

svn commit: r1724 - in logback/trunk/logback-classic/src: main/java/ch/qos/logback/classic/stopwatch test/java/ch/qos/logback/classic/stopwatch
by noreply.ceki＠qos.ch 28 Jul '08

28 Jul '08

Author: ceki Date: Mon Jul 28 20:51:34 2008 New Revision: 1724 Removed: logback/trunk/logback-classic/src/main/java/ch/qos/logback/classic/stopwatch/ logback/trunk/logback-classic/src/test/java/ch/qos/logback/classic/stopwatch/ Log: profiler code moved to slf4j

1 0

[JIRA] Commented: (LBCORE-42) add option to FileAppender to auto-create parent directories
by Rob Ross (JIRA) 27 Jul '08

27 Jul '08

[ http://jira.qos.ch/browse/LBCORE-42?page=com.atlassian.jira.plugin.system.i… ] Rob Ross commented on LBCORE-42: -------------------------------- I agree with the above. I just started using logback as a replacement for log4j, and this is the first big difference in behavior that I have found. Please implement this functionality in logback. Thanks! > add option to FileAppender to auto-create parent directories > ------------------------------------------------------------ > > Key: LBCORE-42 > URL: http://jira.qos.ch/browse/LBCORE-42 > Project: logback-core > Issue Type: Bug > Components: Appender > Affects Versions: unspecified > Environment: Operating System: Linux > Platform: PC > Reporter: Bruno Navert > Assignee: Logback dev list > > FileAppender should be able to automatically create parent directories as needed when writing the log file. > For instance, if configured with <File>/var/tmp/project_name/some_file.log</File>, and the new option set to true (suggest <CreateParentDirectories>true</CreateParentDirectories> maybe), then in FileAppender.setFile(), it should do > new File( fileName ).getParentFile().mkdirs() > workaround is to create the directory before initializing logback. Possible but far less than ideal, and rather complicated too -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.qos.ch/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira

1 0

Mkdirs in FileAppender
by Lilianne E. Blaze 22 Jul '08

22 Jul '08

Hello, Could you please please consider adding the ability to create missing parent directories in FileAppender in Logback Core? Suppose I have the following: (tomcat home)/logs/ And I'd like for my webapp to automatically create a subdirectory, so after I run it I have: (tomcat home)/logs/my-webapp/my-webapp.log Right now if I just configure Logback to use the above path, it obviously fails, I have to create /my-webapp/ manually. The code to do it (at least the most basic version is trivial, just modify public synchronized void setFile() throws IOException { so it contains: closeWriter(); // modified code starts here if( mkdirs ) { File f = new File(fileName); File fParent = f.getParentFile(); if( !fParent.exists() ) { fParent.mkdirs(); } } // modified code ends here this.writer = createWriter(new FileOutputStream(fileName, append)); And obviously add mkdirs property and import java.io.File. Greetings, Lilianne E. Blaze

1 0