Re: [log4j] getCallerClass()

2017-12-23 Thread Matt Sicker
And we can add another page to the javadocs for the history of this method. :) On 22 December 2017 at 10:21, Ralph Goers wrote: > Reflection.getCallerClass() has been removed in Java 10 [1]. The pre-Java > 9 version is already sensitive to this so it shouldn’t cause

Re: [log4j] providing sourcewith Message

2017-12-23 Thread Mikael Ståldal
Yes, StackTraceElement does not allow null methodName. But what about using the empty string, or a string like "" if it is not possible to get? On 2017-12-22 20:26, Jeffrey Shaw wrote: I didn't know about clirr, but I can try running that. I tried to not break any existing public methods.

Re: [log4j] release 2.10.1

2017-12-23 Thread Mikael Ståldal
Yes, it does. On 2017-12-23 17:39, Ralph Goers wrote: I am using the same version of Maven and Java. Does your reactor order match mine?

Re: [log4j] release 2.10.1

2017-12-23 Thread Ralph Goers
I am using the same version of Maven and Java. Does your reactor order match mine? [INFO] Reactor Summary: [INFO] [INFO] Apache Log4j 2 . SUCCESS [ 1.532 s] [INFO] Apache Log4j API Java 9 support SUCCESS [ 4.568 s] [INFO] Apache Log4j

Re: [log4j] release 2.10.1

2017-12-23 Thread Mikael Ståldal
I just tried with latest master. No errors in log4j-api, but in log4j-osgi: $ mvn -v Apache Maven 3.3.9 (bb52d8502b132ec0a5a3f4c09453c07478323dc5; 2015-11-10T17:41:47+01:00) Maven home: /opt/apache-maven-3.3.9 Java version: 1.8.0_144, vendor: Oracle Corporation Java home: