search

spotbugs / spotbugs-maven-plugin

Maven Mojo Plug-In to generate reports based on the SpotBugs Analyzer
https://spotbugs.github.io/spotbugs-maven-plugin/
Apache License 2.0
69 stars 51 forks source link

Update Continuous Integration to test against Java 11 #78

Closed vimal4david closed 5 years ago

hazendaz commented 5 years ago

Are you referring to travis city? Jdk 11 won't work until groovy updates asm.

Get Outlook for Androidhttps://aka.ms/ghei36

From: vimal4david notifications@github.com Sent: Monday, September 17, 2018 5:09:38 AM To: spotbugs/spotbugs-maven-plugin Cc: Subscribed Subject: [spotbugs/spotbugs-maven-plugin] Update Continuous Integration to test against Java 11 (#78)

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/spotbugs/spotbugs-maven-plugin/issues/78, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AA7ho2Bhipqx-S9CtSLdfI9eN8jgFCTJks5ub2bSgaJpZM4WrhLW.

vimal4david commented 5 years ago

I refer to Travis CI. The link is here

vimal4david commented 5 years ago

Build details: https://travis-ci.org/spotbugs/spotbugs-maven-plugin/jobs/429572630 As mentioned by @hazendaz it's failing with JDK 11

[ERROR] Failed to execute goal org.codehaus.gmavenplus:gmavenplus-plugin:1.6.1:generateStubs (compileGroovy) on project spotbugs-maven-plugin: Error occurred while calling a method on a Groovy class from classpath. InvocationTargetException: startup failed: [ERROR] General error during conversion: java.lang.UnsupportedOperationException [ERROR] [ERROR] java.lang.UnsupportedOperationException [ERROR] at groovyjarjarasm.asm.ClassVisitor.visitNestMemberExperimental(ClassVisitor.java:248) [ERROR] at groovyjarjarasm.asm.ClassReader.accept(ClassReader.java:651) [ERROR] at groovyjarjarasm.asm.ClassReader.accept(ClassReader.java:391)

vimal4david commented 5 years ago

Relates to this Groovy issue

hazendaz commented 5 years ago

Requires groovy 2.5.3 and gmavenplus 1.6.2. Currently both are pending release.

vimal4david commented 5 years ago

The build is failing because Travis CI is using Maven version 3.5.3. I have raised a PR in Travis CI to update there Maven version to 3.5.4

hazendaz commented 5 years ago

We use the maven wrapper so travis ci doesnt need bumped, although the doesnt hurt. For this project look in the .mvn folder for the wrapper properties file and bump to 3.5.4 there.

Get Outlook for Androidhttps://aka.ms/ghei36

From: Vimal David notifications@github.com Sent: Monday, October 15, 2018 11:07:25 AM To: spotbugs/spotbugs-maven-plugin Cc: Jeremy Landis; Mention Subject: Re: [spotbugs/spotbugs-maven-plugin] Update Continuous Integration to test against Java 11 (#78)

The build is failing because Travis CI is using Maven version 3.5.3. I have raised a PRhttps://github.com/travis-ci/travis-cookbooks/pull/997 in Travis CI to update there Maven version to 3.5.4

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/spotbugs/spotbugs-maven-plugin/issues/78#issuecomment-429891840, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AA7ho2O-MqJU8FcWEQZseHG2WdbMxPqOks5ulKStgaJpZM4WrhLW.

hazendaz commented 5 years ago

Closed via #88

I have opened a new issue though against jdk11 on one unit test #90