Skip to content

Conversation

Eng-Fouad
Copy link
Contributor

@Eng-Fouad Eng-Fouad commented Jun 22, 2025

project.exe() is deprecated since Gradle 8.11 and will be removed in the next release Gradle 9.0.

* @deprecated Since 8.11. This method will be removed in Gradle 9.0. Use {@link org.gradle.process.ExecOperations#exec(Action)} or {@link ProviderFactory#exec(Action)} instead.
     */
    @Deprecated
    ExecResult exec(Action<? super ExecSpec> action);

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

quarkus-bot bot commented Jun 22, 2025

Status for workflow Quarkus CI

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

✅ 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.

@Eng-Fouad Eng-Fouad mentioned this pull request Jun 22, 2025
10 tasks
@gastaldi gastaldi merged commit 47eac67 into quarkusio:main Jun 22, 2025
23 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.25 - main milestone Jun 22, 2025
@Eng-Fouad Eng-Fouad deleted the gradle-deprecated-1 branch June 22, 2025 12:33
reaver585 added a commit to reaver585/quarkus that referenced this pull request Jun 23, 2025
…ecated-1"

This reverts commit 47eac67, reversing
changes made to 7353d12.
reaver585 added a commit to reaver585/quarkus that referenced this pull request Jun 23, 2025
…ecated-1"

This reverts commit 47eac67, reversing
changes made to 7353d12.
@aloubyansky
Copy link
Member

We are running in Gradle CI issues that seem to be caused by this change (although the CI for this PR looked green).
I'm opening a PR to revert this change until we figure out why these changes are not enough.
#48547

@aloubyansky
Copy link
Member

@reaver585 was faster #48546

@Eng-Fouad
Copy link
Contributor Author

We are running in Gradle CI issues that seem to be caused by this change (although the CI for this PR looked green). I'm opening a PR to revert this change until we figure out why these changes are not enough. #48547

Sorry about that. Actually, I was surprised that the CI completed successfully in 10 minutes.

@Eng-Fouad
Copy link
Contributor Author

Eng-Fouad commented Jun 24, 2025

I think I found the issue with this PR. getProviders().exec() is a lazy method and it won't be triggered unless we call getProviders().exec().get(). Anyway, I opened another PR with alternative implementation.

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.

3 participants