You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@logging.apache.org by rm...@apache.org on 2020/11/30 23:41:43 UTC

[logging-log4cxx] branch log4cxx-513 updated: LOG4CXX-513 Reordered some rolling code to create intermediate directories

This is an automated email from the ASF dual-hosted git repository.

rmiddleton pushed a commit to branch log4cxx-513
in repository https://gitbox.apache.org/repos/asf/logging-log4cxx.git


The following commit(s) were added to refs/heads/log4cxx-513 by this push:
     new a3deefc  LOG4CXX-513 Reordered some rolling code to create intermediate directories
a3deefc is described below

commit a3deefccc24af9c32a176cf234a4d88d0ec99eb2
Author: Robert Middleton <ro...@rm5248.com>
AuthorDate: Mon Nov 30 19:11:06 2020 -0500

    LOG4CXX-513 Reordered some rolling code to create intermediate directories
---
 src/main/cpp/rollingfileappender.cpp | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/main/cpp/rollingfileappender.cpp b/src/main/cpp/rollingfileappender.cpp
index dfabb3b..90b5b48 100644
--- a/src/main/cpp/rollingfileappender.cpp
+++ b/src/main/cpp/rollingfileappender.cpp
@@ -340,11 +340,13 @@ bool RollingFileAppenderSkeleton::rollover(Pool& p)
 						}
 						else
 						{
+							closeWriter();
+							setFile(rollover1->getActiveFileName());
+							// Call activateOptions to create any intermediate directories(if required)
+							FileAppender::activateOptions(p);
 							OutputStreamPtr os(new FileOutputStream(
 									rollover1->getActiveFileName(), rollover1->getAppend()));
 							WriterPtr newWriter(createWriter(os));
-							closeWriter();
-							setFile(rollover1->getActiveFileName());
 							setWriter(newWriter);
 
 							bool success = true;