Uploaded image for project: 'Maven Dependency Plugin'
  1. Maven Dependency Plugin
  2. MDEP-802

analyze-only fails to recognize `aws-java-sdk-core` as a compile-time dependency

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Open
    • Major
    • Resolution: Unresolved
    • 3.3.0
    • None
    • analyze-only
    • None

    Description

      The maven-dependency-plugin fails to recognize `aws-java-sdk-core` as a compile-time dependency, because we only explicitly use a class from `aws-java-sdk-glue`, which in turn extends from a class from `aws-java-sdk-core`.

       

      In order to cope with this problem, we set `aws-java-sdk-core` as an ignored dependency:

       

      
                              <artifactId>maven-dependency-plugin</artifactId>
                              <configuration>
                                  <ignoredNonTestScopedDependencies>
                                      <!-- dependency plugin fails to recognize aws-java-sdk-core as a compile-time dependency, because
                                           we only explicitly use a class from aws-java-sdk-glue, which in turn extends from a class from
                                           aws-java-sdk-core -->
                                      <ignoredDependency>com.amazonaws:aws-java-sdk-core</ignoredDependency>
                                  </ignoredNonTestScopedDependencies>
                              </configuration>
                          </plugin>
      
      

       

      Here is the commit showcasing the issue:

       

      https://github.com/trinodb/trino/pull/11550/commits/22f227bb5721b1ea4783c6a58111ef5007f75316

       

      Ideally we'd like not having to deal with this false positive problem.

      Attachments

        Activity

          People

            Unassigned Unassigned
            findinpath Marius Grama
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

              Created:
              Updated: