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/12/02 23:17:03 UTC

[logging-log4cxx] branch master updated: LOG4CXX-513 Reordered some rolling code to create intermediate directories (#45)

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

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


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

commit beb771eae0d7e8ee40067a969d8199ab9639982e
Author: Robert Middleton <rm...@users.noreply.github.com>
AuthorDate: Wed Dec 2 18:16:57 2020 -0500

    LOG4CXX-513 Reordered some rolling code to create intermediate directories (#45)
---
 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;