I have no problem with the concept of the patch. However, I made some changes since that was written and I have to make a few more to resolve LOG4J2-2152.
Ralph > On Apr 8, 2018, at 10:32 AM, Gary Gregory <[email protected]> wrote: > > Any thoughts on this PR? > > Gary > > ---------- Forwarded message ---------- > From: Romain Manni-Bucau <[email protected]> > Date: Tue, Jan 9, 2018 at 4:00 AM > Subject: [apache/logging-log4j2] adding log4j2.forceTCLOnly option (#140) > To: apache/logging-log4j2 <[email protected]> > Cc: Subscribed <[email protected]> > > > ------------------------------ > You can view, comment on, or merge this pull request online at: > > https://github.com/apache/logging-log4j2/pull/140 > Commit Summary > > - adding log4j2.forceTCLOnly option > > File Changes > > - *M* log4j-api/src/main/java/org/apache/logging/log4j/util/ > LoaderUtil.java > <https://github.com/apache/logging-log4j2/pull/140/files#diff-0> (48) > - *A* log4j-api/src/test/java/org/apache/logging/log4j/util/ > LoaderUtilTest.java > <https://github.com/apache/logging-log4j2/pull/140/files#diff-1> (58) > - *M* src/site/xdoc/manual/configuration.xml.vm > <https://github.com/apache/logging-log4j2/pull/140/files#diff-2> (10) > > Patch Links: > > - https://github.com/apache/logging-log4j2/pull/140.patch > - https://github.com/apache/logging-log4j2/pull/140.diff > > — > You are receiving this because you are subscribed to this thread. > Reply to this email directly, view it on GitHub > <https://github.com/apache/logging-log4j2/pull/140>, or mute the thread > <https://github.com/notifications/unsubscribe-auth/ABIfN5qDxD9ZBKTMotzN2RBvxMEAIO87ks5tI0bmgaJpZM4RXp3M> > .
