Re: [log4j] Log4j Java 9 support has been added

2017-04-27 Thread Mikael Ståldal
OK, I can change JDK per module. Now I cannot build log4j-jmx-gui since the class com.sun.tools.jconsole.JConsolePlugin (used in our ClientGuiJConsolePlugin) doesn't seem to work with JDK 9. I tried to set that module to JDK 7, but it doesn't help. It is not good that we break a popular IDE like

Re: [log4j] Log4j Java 9 support has been added

2017-04-26 Thread Ralph Goers
Sure you can. In the project settings select the module. You can change it there. Ralph > On Apr 26, 2017, at 7:35 AM, Mikael Ståldal wrote: > > I don't think you can specify different JDK:s for different modules with > IntelliJ. You can specify different source and target levels, but that > d

Re: [log4j] Log4j Java 9 support has been added

2017-04-26 Thread Mikael Ståldal
I don't think you can specify different JDK:s for different modules with IntelliJ. You can specify different source and target levels, but that doesn't help here. Even though I set log4j-perf to 1.7, I get that error. On Wed, Apr 26, 2017 at 4:13 PM, Ralph Goers wrote: > You could compile the ap

Re: [log4j] Log4j Java 9 support has been added

2017-04-26 Thread Ralph Goers
You could compile the api module with Java 9 and the rest with Java 7. Until the module-info.java files are added the api module is the only thing that has Java 9 code in it. Ralph > On Apr 26, 2017, at 6:55 AM, Mikael Ståldal wrote: > > I was able to get log4j-api to build by manually instal

Re: [log4j] Log4j Java 9 support has been added

2017-04-26 Thread Mikael Ståldal
I was able to get log4j-api to build by manually installing JDK 9 and setting JDK 9 for that module. But now it fails in log4j-perf with the error below. Maybe related to https://bugs.openjdk.java.net/browse/JDK-8152842 Error:java: Annotation generator had thrown the exception. java.lang.NoClassD

Re: [log4j] Log4j Java 9 support has been added

2017-04-26 Thread Apache
You could try using java 9 Sent from my iPad > On Apr 26, 2017, at 2:20 AM, Mikael Ståldal wrote: > > Now I cannot build the project in IntelliJ IDEA anymore. It complains that > lambda expressions are not supported in Java 7 in StackWalkerStackLocator. > > Any suggestions? > > On Fri, Apr 21

Re: [log4j] Log4j Java 9 support has been added

2017-04-26 Thread Mikael Ståldal
Now I cannot build the project in IntelliJ IDEA anymore. It complains that lambda expressions are not supported in Java 7 in StackWalkerStackLocator. Any suggestions? On Fri, Apr 21, 2017 at 10:59 AM, Mikael Ståldal wrote: > Should the vendor of the Java 7, 8 and 9 toolchains be "sun"? Shouldn'

Re: [log4j] Log4j Java 9 support has been added

2017-04-21 Thread Mikael Ståldal
Should the vendor of the Java 7, 8 and 9 toolchains be "sun"? Shouldn't it be "oracle"? On Fri, Apr 21, 2017 at 9:26 AM, Matt Sicker wrote: > 🎉🎉🎉 > > On 21 April 2017 at 01:41, Ralph Goers wrote: > > > I’ve pushed the support for Java 9 and Stackwalker. Java 9 is now > required > > to build Log

Re: [log4j] Log4j Java 9 support has been added

2017-04-21 Thread Matt Sicker
🎉🎉🎉 On 21 April 2017 at 01:41, Ralph Goers wrote: > I’ve pushed the support for Java 9 and Stackwalker. Java 9 is now required > to build Log4j in addition to Java 7. > > Ralph > -- Matt Sicker

[log4j] Log4j Java 9 support has been added

2017-04-20 Thread Ralph Goers
I’ve pushed the support for Java 9 and Stackwalker. Java 9 is now required to build Log4j in addition to Java 7. Ralph