Regarding the unstable integration test, Surefire1295AttributeJvmCrashesToTestsIT, I opened a discussion: https://github.com/michaeltandy/crashjvm/issues/1 Not related to Java 9/10. It was unstable test before as well.
On Sun, Feb 18, 2018 at 4:39 PM, Tibor Digana <tibordig...@apache.org> wrote: > Can you tell me how and where you reproduced this stacktrace? > > Regarding the hack in *surefire-booter* I understand what happened and > why today the hack is not needed and why it does not cause any problem if I > remove it from the POM. > This happened because two related issues were fixed in November and > January. > The hack was one of the solutions but the really good solution was done > later in src/test/java and the hack in POM was not removed. > https://issues.apache.org/jira/browse/SUREFIRE-1439 > https://issues.apache.org/jira/browse/SUREFIRE-1463 > > Our entire problem was with this method: > *org.apache.commons.lang3.SystemUtils.isJavaVersionAtLeast()* > It threw NPE. Now we do not use it and we use JAVA_RECENT.atLeast( JAVA_9 > ) instead. > > After I have removed the obsolete hack in the POM, the tests in > *surefire-booter* still passed. > > By looking at your stacktrace I think this is class loader issue but I do > not know the causes. > Can you test it again and print longer stacktrace with every cause? > > Offtopic: master failed on different problem and not the java version > issue. The integration test, Surefire1295AttributeJvmCrashesToTestsIT, > which failed is using native library and it is not very stable. After it > has finished I will download the logs from surefire-its/target. Usually the > logs do not contain expected message. It is not related to JDK 9/10 because > this test was not stable before with Java 7/8. > > > > On Sun, Feb 18, 2018 at 4:58 AM, Olivier Lamy <ol...@apache.org> wrote: > >> locally >> >> mvn clean install >> -Djdk.home=/Library/Java/JavaVirtualMachines/jdk-10.jdk/Contents/Home >> -Djacoco.skip=true >> >> Caused by: java.lang.ClassNotFoundException: >> org.apache.commons.lang3.SystemUtils >> >> at java.base/java.net.URLClassLoader.findClass(URLClassLoader.java:466) >> >> at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:566) >> >> at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:499) >> >> at >> org.apache.maven.surefire.shadefire.booter.IsolatedClassLoad >> er.loadClass(IsolatedClassLoader.java:97) >> >> ... 44 more >> >> Caused by: java.lang.ClassNotFoundException: >> org.apache.commons.lang3.SystemUtils >> >> at java.base/java.net.URLClassLoader.findClass(URLClassLoader.java:466) >> >> at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:566) >> >> at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:499) >> >> at >> org.apache.maven.surefire.shadefire.booter.IsolatedClassLoad >> er.loadClass(IsolatedClassLoader.java:97) >> >> >> >> >> On 18 February 2018 at 13:44, Tibor Digana <tibordig...@apache.org> >> wrote: >> >> > The Jenkins CI was shutdown for several hours. So the HEAD was not >> taken. >> > I can trigger the build manually. >> > >> > Cheers >> > Tibor >> > >> > On Sun, Feb 18, 2018 at 4:39 AM, Tibor Digana <tibordig...@apache.org> >> > wrote: >> > >> > > I will explain again everything. >> > > Do not worry, please. >> > > >> > > The build won't fail if you use -Djdk.home=/path/to/jdk. We don't >> have to >> > > force the users to change JDK if they are fine with 1.6. >> > > The library commons-lang3:3.7 was especially needed during our testing >> > but >> > > not in the main code, therefore Java 1.6 is still fine for this plugin >> > > because the "os.version" is not read from this library. >> > > It is also written in README.md on how to build the project. It is >> also >> > > safe for release plugin. >> > > The Jenkins CI passed: https://builds.apache.org/job/ >> > > maven-wip/job/maven-surefire/job/SUREFIRE-1463/ >> > > The information in master is old one - not related to current HEAD on >> > > master. >> > > The reason is that the libraries can still use javac compiler with >> > > "-source 1.6" but the integration tests are important, not the >> compiler. >> > > Did you see this job? >> > > https://builds.apache.org/job/maven-wip/job/maven-surefire/ >> > > job/SUREFIRE-1463/ >> > > >> > > Cheers >> > > Tibor >> > > >> > > >> > > On Sun, Feb 18, 2018 at 4:24 AM, Olivier Lamy <ol...@apache.org> >> wrote: >> > > >> > >> and now master fail again with jdk10... >> > >> Why are you reverting my changes? >> > >> As said in an email I wanted to have something working with jdk10. >> > >> So please first discuss on the email thread if you are not happy >> with my >> > >> changes... >> > >> >> > >> >> > >> On 18 February 2018 at 10:36, <tibordig...@apache.org> wrote: >> > >> >> > >> > Revert "[SUREFIRE-1473] upgrade commons-lang3 to avoid NPE on >> jdk10" >> > >> > >> > >> > This reverts commit 8bcf1f14a8697ef3359d7598fbd8062a687f2d5a. >> > >> > >> > >> > >> > >> > Project: http://git-wip-us.apache.org/r >> epos/asf/maven-surefire/repo >> > >> > Commit: http://git-wip-us.apache.org/repos/asf/maven-surefire/ >> > >> > commit/5e732c6b >> > >> > Tree: http://git-wip-us.apache.org/repos/asf/maven-surefire/tree/5 >> > >> e732c6b >> > >> > Diff: http://git-wip-us.apache.org/repos/asf/maven-surefire/diff/5 >> > >> e732c6b >> > >> > >> > >> > Branch: refs/heads/master >> > >> > Commit: 5e732c6b2a87026ac56032cc8b2b7e8e92e6aa7e >> > >> > Parents: 3c41c55 >> > >> > Author: Tibor17 <tibordig...@apache.org> >> > >> > Authored: Sun Feb 18 00:13:42 2018 +0100 >> > >> > Committer: Tibor17 <tibordig...@apache.org> >> > >> > Committed: Sun Feb 18 00:13:42 2018 +0100 >> > >> > >> > >> > ------------------------------------------------------------ >> > ---------- >> > >> > pom.xml | 2 +- >> > >> > 1 file changed, 1 insertion(+), 1 deletion(-) >> > >> > ------------------------------------------------------------ >> > ---------- >> > >> > >> > >> > >> > >> > http://git-wip-us.apache.org/repos/asf/maven-surefire/blob/ >> > >> > 5e732c6b/pom.xml >> > >> > ------------------------------------------------------------ >> > ---------- >> > >> > diff --git a/pom.xml b/pom.xml >> > >> > index 2bcb0d7..3592b31 100644 >> > >> > --- a/pom.xml >> > >> > +++ b/pom.xml >> > >> > @@ -89,7 +89,7 @@ >> > >> > <mavenVersion>2.2.1</mavenVersion> >> > >> > <!-- <shadedVersion>2.12.4</shadedVersion> commented out due >> to >> > >> > https://issues.apache.org/jira/browse/MRELEASE-799 --> >> > >> > <mavenPluginPluginVersion>3.5</mavenPluginPluginVersion> >> > >> > - <commonsLang3Version>3.7</commonsLang3Version> >> > >> > + <commonsLang3Version>3.5</commonsLang3Version> >> > >> > <commonsIoVersion>2.5</commonsIoVersion> >> > >> > <mavenSharedUtilsVersion>0.9</mavenSharedUtilsVersion> >> > >> > <powermockVersion>2.0.0-beta.5</powermockVersion> >> > >> > >> > >> > >> > >> >> > >> >> > >> -- >> > >> Olivier Lamy >> > >> http://twitter.com/olamy | http://linkedin.com/in/olamy >> > >> >> > > >> > > >> > >> >> >> >> -- >> Olivier Lamy >> http://twitter.com/olamy | http://linkedin.com/in/olamy >> > >