Skip to content

Gradle quarkusDev not working on multiprojects when jandex is present #48670

New issue

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

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

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Malandril
Copy link
Contributor

Fixes #48637 by avoiding adding a mustRunAfter accross build projects

@quarkus-bot quarkus-bot bot added area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/gradle Gradle labels Jun 28, 2025
@aloubyansky
Copy link
Member

@Malandril thanks a lot for the fix! It'd be great to include your reproducer as a test though to make sure we don't break it in the future. Would you like to do that?

Our current framework for these kinds of tests assumes there is one (possibly multi) project, which isn't exactly your case. But it should be doable.
Just in case, here is an example test https://github.com/quarkusio/quarkus/blob/main/integration-tests/gradle/src/test/java/io/quarkus/gradle/devmode/BasicJavaApplicationModuleDevModeTest.java
and a test project https://github.com/quarkusio/quarkus/tree/main/integration-tests/gradle/src/main/resources/basic-java-application-project

Let me know in case you want to give it a try. Otherwise, I'll look into it. Thanks!

@Malandril
Copy link
Contributor Author

@aloubyansky i'll look into i'll try to strip down the reproducer to the minimum

@aloubyansky
Copy link
Member

That'd be awesome @Malandril.
Test projects are copied and "augmented" with the relevant Quarkus properties here https://github.com/quarkusio/quarkus/blob/main/integration-tests/gradle/src/test/java/io/quarkus/gradle/QuarkusGradleTestBase.java#L21
This is where the assumption there is only one project is.
In your case, I was thinking we could simply call it directly for each project and then build one with https://github.com/quarkusio/quarkus/blob/main/integration-tests/gradle/src/test/java/io/quarkus/gradle/QuarkusGradleWrapperTestBase.java#L51

And then test dev mode in the main one.

Copy link

quarkus-bot bot commented Jun 30, 2025

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit 345841b.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

You can consult the Develocity build scans.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/gradle Gradle
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Gradle multiproject build with jandex fails with when running quarkusDev
2 participants