Code Monkey home page Code Monkey logo

Comments (15)

aalmiray avatar aalmiray commented on July 22, 2024 2

Turns out the bug is in setting up the classpath that invokes the JavaExec type that runs JandexMain. I know how to fix this. Expect a release in the coming days.

from jandex-gradle-plugin.

re-thc avatar re-thc commented on July 22, 2024 1

7.4.2 is fine. Failure is with 7.5-rc-1. Could be a gradle bug itself or a change of APIs / functionality that has resulted in this. Don't have a specific project yet. Just went to the Quarkus website, checked out a sample and added this plugin to it and triggered the issue.

from jandex-gradle-plugin.

aalmiray avatar aalmiray commented on July 22, 2024

This is strange. The source does not make reference to internal APIs, it's written in Java so there should not be a dependence on an specific Gradle version as it happens when Groovy code is compiled AFAICT https://github.com/kordamp/jandex-gradle-plugin/blob/master/src/main/java/org/kordamp/gradle/plugin/jandex/tasks/JandexMain.java

from jandex-gradle-plugin.

re-thc avatar re-thc commented on July 22, 2024

I just tried this on an empty/blank Quarkus project and it builds. Once the jandex-gradle-plugin is added it fails. Either jandex or this plugin but if it's jandex it should fail without this plugin since Quarkus uses it under the hood?

from jandex-gradle-plugin.

re-thc avatar re-thc commented on July 22, 2024

gradle/gradle#14727 could be this bug? Is there a way we can work around it?

from jandex-gradle-plugin.

aalmiray avatar aalmiray commented on July 22, 2024

Do you have a reproducible project?

from jandex-gradle-plugin.

aalmiray avatar aalmiray commented on July 22, 2024

Do you get the failure with an earlier Gradle version? say 7.4?

from jandex-gradle-plugin.

aalmiray avatar aalmiray commented on July 22, 2024

The plugin reads compiled classes from the output of the main resourceSet, as is. It may be the case those classes have the missing annotation applied to them. The classpath passed to Jandex may not contain the annotation class anymore in 7.5

I don't have access to a computer right now but perhaps running a simple project in debug mode with 7.4.2 vs 7.5 should result in logging the execution classpath used by the JandexMain class. With any luck this should show if a JAR is missing. If not then the annotation used to be located in an existing JAR.

In any case I don't think there's a workaround if Gradle 7.5 is required. If my suspicion is correct then reading classes to be processed must be done in a different way.

from jandex-gradle-plugin.

re-thc avatar re-thc commented on July 22, 2024

Any luck on fixing this? Thanks!

from jandex-gradle-plugin.

gbloggs avatar gbloggs commented on July 22, 2024

I'm just building against the released version of gradle, 7.5 and getting this error.

    id "org.kordamp.gradle.jandex" version "0.12.0"
    id 'io.quarkus' version '2.11.0.Final'


Gradle 7.5

Build time: 2022-07-14 12:48:15 UTC
Revision: c7db7b958189ad2b0c1472b6fe663e6d654a5103

Kotlin: 1.6.21
Groovy: 3.0.10
Ant: Apache Ant(TM) version 1.10.11 compiled on July 10 2021
JVM: 17.0.1 (GraalVM Community 17.0.1+12-jvmci-21.3-b05)
OS: Windows 10 10.0 amd64



from jandex-gradle-plugin.

gsmet avatar gsmet commented on July 22, 2024

@aalmiray hey. Any chance this issue could be fixed? It's blocking the Quarkus upgrade to Gradle 7.5.

Thanks!

from jandex-gradle-plugin.

aalmiray avatar aalmiray commented on July 22, 2024

@gsmet currently looking at it. It appears I'll have to switch to Gradle's Worker API to make it work. Thing is, I'm still learning how to use said API. Hope to get a release ready this week.

from jandex-gradle-plugin.

aalmiray avatar aalmiray commented on July 22, 2024

@gsmet version 0.13.0 has just been released with the fix. Please let me know if there any further changes are required.

from jandex-gradle-plugin.

gsmet avatar gsmet commented on July 22, 2024

Thanks! @glefloch can you have a look?

from jandex-gradle-plugin.

glefloch avatar glefloch commented on July 22, 2024

Thanks @aalmiray, I will update quarkus sample projects

from jandex-gradle-plugin.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.