Skip to content

Conversation

@nastra
Copy link
Contributor

@nastra nastra commented Jul 3, 2023

Reverts #7955

With Gradle 8.2 and when running ./gradlew clean build -x test -x integrationTest, this would fail with the below error:

FAILURE: Build failed with an exception.

* What went wrong:
A problem was found with the configuration of task ':iceberg-data:revapiAnalyze' (type 'RevapiAnalyzeTask').
  - Gradle detected a problem with the following location: '/home/nastra/Development/workspace/iceberg/common/build/libs/iceberg-common-1.4.0-SNAPSHOT.jar'.
    
    Reason: Task ':iceberg-data:revapiAnalyze' uses this output of task ':iceberg-common:jar' without declaring an explicit or implicit dependency. This can lead to incorrect results being produced, depending on what order the tasks are executed.
    
    Possible solutions:
      1. Declare task ':iceberg-common:jar' as an input of ':iceberg-data:revapiAnalyze'.
      2. Declare an explicit dependency on ':iceberg-common:jar' from ':iceberg-data:revapiAnalyze' using Task#dependsOn.
      3. Declare an explicit dependency on ':iceberg-common:jar' from ':iceberg-data:revapiAnalyze' using Task#mustRunAfter.
    
    For more information, please refer to https://docs.gradle.org/8.2/userguide/validation_problems.html#implicit_dependency in the Gradle documentation.

@github-actions github-actions bot added the build label Jul 3, 2023
@nastra nastra requested a review from Fokko July 3, 2023 07:59
@snazy
Copy link
Member

snazy commented Jul 3, 2023

Wait - let's fix the offending task dependencies. Gradle just complains that those are wrong - aka inconsistent builds.

@snazy
Copy link
Member

snazy commented Jul 3, 2023

Fix here: #7970

@nastra nastra closed this Jul 3, 2023
@nastra nastra deleted the revert-7955-gradle-8.2 branch July 3, 2023 09:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants