Skip to content
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

[Distribution Build Failure] Detekt fails to resolve com.beust:jcommander:1.74 #249

Closed
3 tasks done
downsrob opened this issue Jan 14, 2022 · 0 comments
Closed
3 tasks done
Assignees
Labels
bug Something isn't working infra Changes to infrastructure, testing, CI/CD, pipelines, etc. v1.2.4

Comments

@downsrob
Copy link
Contributor

downsrob commented Jan 14, 2022

Describe the bug
Detekt 1.0.0-RC15, used in the index management build process for detecting code smells, fails to run due to not being able to resolve the detekt dependency jcommander 1.74. This dependency, jcommander 1.74, does not exist on maven central, and can only be resolved through jcenter. As jcenter is sunset, this dependency may never be resolved, so detekt must be upgraded to a version which doesn't include this dependency, or disabled. In later versions of detekt, the jcommander dependency was moved to version 1.72, so upgrading detekt should resolve this issue.

As upgrading detekt adds new code style checks, additional work will be needed to fix the newly found code smells to enable the detekt check to pass.

To Reproduce
Steps to reproduce the behavior:
./gradlew build or ./gradlew detekt fail due to a missing detekt dependency

> Task :detekt FAILED
FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':detekt'.
> Could not resolve all files for configuration ':detekt'.
   > Could not find com.beust:jcommander:1.74.
     Searched in the following locations:
       - https://repo.maven.apache.org/maven2/com/beust/jcommander/1.74/jcommander-1.74.pom
       - file:/home/runner/.m2/repository/com/beust/jcommander/1.74/jcommander-1.74.pom
       - https://aws.oss.sonatype.org/content/repositories/snapshots/com/beust/jcommander/1.74/jcommander-1.74.pom
     Required by:
         project : > io.gitlab.arturbosch.detekt:detekt-cli:1.0.0-RC15

Remediation
To fix this issue:

Additional context
As jcommander 1.74 is not on maven central, the jcenter workaround described here opensearch-project/opensearch-build#1456 does not help resolve the dependency.

@downsrob downsrob added bug Something isn't working infra Changes to infrastructure, testing, CI/CD, pipelines, etc. v1.2.4 labels Jan 14, 2022
@downsrob downsrob self-assigned this Jan 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working infra Changes to infrastructure, testing, CI/CD, pipelines, etc. v1.2.4
Projects
None yet
Development

No branches or pull requests

1 participant