Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

jandex does not build with Maven 3.9.0 #298

Closed
bdemers opened this issue Feb 27, 2023 · 2 comments · Fixed by #305
Closed

jandex does not build with Maven 3.9.0 #298

bdemers opened this issue Feb 27, 2023 · 2 comments · Fixed by #305
Assignees
Milestone

Comments

@bdemers
Copy link
Contributor

bdemers commented Feb 27, 2023

impsort-maven-plugin isn't compatible with the latest version of Maven
revelc/impsort-maven-plugin#64

Execution sort-imports of goal net.revelc.code:impsort-maven-plugin:1.6.2:sort failed: A required class was missing while executing net.revelc.code:impsort-maven-plugin:1.6.2:sort: org/codehaus/plexus/util/DirectoryScanner

impsort-maven-plugin has already released a fix and this version is already updated in smallrye-build-parent version 39.

Updating to the parent 39

  • Builds successfully (locally) with Maven 3.8.7
  • Reformats a handful of classes (whitespace)
  • Fails with Maven 3.9.0, caused to an issue with the invoker tests (I stopped digging into the issue at this point)
@Ladicek
Copy link
Contributor

Ladicek commented Feb 27, 2023

Unfortunately the latest version of the impsort plugin requires Java 11, and I find it important for Jandex CI builds to run with Java 8 (among others). So this requires to jump through some hoops.

@bdemers
Copy link
Contributor Author

bdemers commented Feb 27, 2023

I can relate, I've been working on a few projects that support older JVM's and it's increasingly becoming more difficult.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants