On 21/11/2019 19:29, ggreg...@apache.org wrote: > This is an automated email from the ASF dual-hosted git repository. > > ggregory pushed a commit to branch master > in repository https://gitbox.apache.org/repos/asf/commons-daemon.git > > > The following commit(s) were added to refs/heads/master by this push: > new 8a46539 [DAEMON-414] prunsrv uses its log is before it is > initialized. > 8a46539 is described below > > commit 8a46539bd0a7fe42fab667a249a1c59906faf9ab > Author: Gary Gregory <gardgreg...@gmail.com> > AuthorDate: Thu Nov 21 14:29:25 2019 -0500
Sorry, this needs to be reverted. You've moved the logging initialisation above the code that reads the logging configuration. This breaks the logging. A better fix would probably be to change the two lines that write to the log file before it is ready to write to stderr instead. Mark > > [DAEMON-414] prunsrv uses its log is before it is initialized. > --- > src/changes/changes.xml | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/src/changes/changes.xml b/src/changes/changes.xml > index f1163e8..03bb314 100644 > --- a/src/changes/changes.xml > +++ b/src/changes/changes.xml > @@ -39,7 +39,10 @@ > </properties> > <body> > <release version="1.2.3" date="TBD" description="Bug fix release"> > - </release< > + <action issue="DAEMON-414" type="fix" dev="ggregory" due-to="Gary > Gregory"> > + prunsrv uses its log is before it is initialized. > + </action> > + </release> > <release version="1.2.2" date="2019-10-04" description="Bug fix release"> > <action issue="DAEMON-408" type="fix" dev="markt"> > Procrun. Correct a regression in the fix for DAEMON-401 that > prevented >
