Unify dependency licenses task configuration (#58116) (#58274)

- Remove duplicate dependency configuration
- Use task avoidance api accross the build
- Remove redundant licensesCheck config

benchmarks/build.gradle

@@ -51,7 +51,7 @@ run.executable = "${BuildParams.runtimeJavaHome}/bin/java"
 disableTasks('forbiddenApisMain')
 
 // No licenses for our benchmark deps (we don't ship benchmarks)
-dependencyLicenses.enabled = false
+tasks.named("dependencyLicenses").configure { it.enabled = false }
 dependenciesInfo.enabled = false
 
 thirdPartyAudit.ignoreViolations(

buildSrc/build.gradle

@@ -176,7 +176,7 @@ if (project != rootProject) {
   groovydoc.enabled = false
 
   // build-tools is not ready for primetime with these...
-  dependencyLicenses.enabled = false
+  tasks.named("dependencyLicenses").configure { it.enabled = false }
   dependenciesInfo.enabled = false
   disableTasks('forbiddenApisMain', 'forbiddenApisMinimumRuntime',
     'forbiddenApisTest', 'forbiddenApisIntegTest', 'forbiddenApisTestFixtures')

client/benchmark/build.gradle

@@ -47,5 +47,5 @@ dependencies {
 }
 
 // No licenses for our benchmark deps (we don't ship benchmarks)
-dependencyLicenses.enabled = false
+tasks.named("dependencyLicenses").configure { it.enabled = false }
 dependenciesInfo.enabled = false

client/client-benchmark-noop-api-plugin/build.gradle

@@ -30,7 +30,7 @@ esplugin {
 // Not published so no need to assemble
 assemble.enabled = false
 
-dependencyLicenses.enabled = false
+tasks.named("dependencyLicenses").configure { it.enabled = false }
 dependenciesInfo.enabled = false
 
 // no unit tests

client/rest-high-level/build.gradle

@@ -62,14 +62,6 @@ processTestResources {
   from(project(':client:rest-high-level').file('src/test/resources'))
 }
 
-dependencyLicenses {
-  // Don't check licenses for dependency that are part of the elasticsearch project
-  // But any other dependency should have its license/notice/sha1
-  dependencies = project.configurations.runtime.fileCollection {
-    it.group.startsWith('org.elasticsearch') == false
-  }
-}
-
 tasks.named('forbiddenApisMain').configure {
   // core does not depend on the httpclient for compile so we add the signatures here. We don't add them for test as they are already
   // specified

client/sniffer/build.gradle

@@ -53,7 +53,7 @@ tasks.named('forbiddenApisTest').configure {
   replaceSignatureFiles 'jdk-signatures'
 }
 
-dependencyLicenses {
+tasks.named("dependencyLicenses").configure {
   mapping from: /http.*/, to: 'httpclient'
   mapping from: /commons-.*/, to: 'commons'
 }
@@ -71,13 +71,6 @@ testingConventions {
   }
 }
 
-
-dependencyLicenses {
-  dependencies = project.configurations.runtime.fileCollection {
-    it.group.startsWith('org.elasticsearch') == false
-  }
-}
-
 thirdPartyAudit.ignoreMissingClasses(
   //commons-logging optional dependencies
   'org.apache.avalon.framework.logger.Logger',

client/test/build.gradle

@@ -48,7 +48,7 @@ tasks.named('forbiddenApisTest').configure {
 jarHell.enabled = false
 
 // TODO: should we have licenses for our test deps?
-dependencyLicenses.enabled = false
+tasks.named("dependencyLicenses").configure { it.enabled = false }
 dependenciesInfo.enabled = false
 
 //we aren't releasing this jar

distribution/tools/plugin-cli/build.gradle

@@ -31,7 +31,7 @@ dependencies {
   testRuntimeOnly 'com.google.guava:guava:18.0'
 }
 
-dependencyLicenses {
+tasks.named("dependencyLicenses").configure {
   mapping from: /bc.*/, to: 'bouncycastle'
 }
 

libs/x-content/build.gradle

@@ -51,7 +51,7 @@ thirdPartyAudit.ignoreMissingClasses(
   'com.fasterxml.jackson.databind.cfg.MapperBuilder'
 )
 
-dependencyLicenses {
+tasks.named("dependencyLicenses").configure {
   mapping from: /jackson-.*/, to: 'jackson'
 }
 

modules/lang-expression/build.gradle

@@ -35,7 +35,7 @@ restResources {
   }
 }
 
-dependencyLicenses {
+tasks.named("dependencyLicenses").configure {
   mapping from: /lucene-.*/, to: 'lucene'
   mapping from: /asm-.*/, to: 'asm'
 }

modules/lang-painless/build.gradle

@@ -42,7 +42,7 @@ dependencies {
   compile project('spi')
 }
 
-dependencyLicenses {
+tasks.named("dependencyLicenses").configure {
   mapping from: /asm-.*/, to: 'asm'
 }
 
@@ -193,4 +193,3 @@ task regen {
     }
   }
 }
-

modules/percolator/build.gradle

@@ -45,10 +45,3 @@ restResources {
     includeCore '_common', 'indices', 'index', 'search', 'msearch'
   }
 }
-
-dependencyLicenses {
-  // Don't check the client's license. We know it.
-  dependencies = project.configurations.runtime.fileCollection {
-    it.group.startsWith('org.elasticsearch') == false
-  } - project.configurations.compileOnly
-}

modules/transport-netty4/build.gradle

@@ -50,7 +50,7 @@ restResources {
   }
 }
 
-dependencyLicenses {
+tasks.named("dependencyLicenses").configure {
   mapping from: /netty-.*/, to: 'netty'
 }
 

plugins/analysis-icu/build.gradle

@@ -42,7 +42,6 @@ restResources {
   }
 }
 
-dependencyLicenses {
+tasks.named("dependencyLicenses").configure {
   mapping from: /lucene-.*/, to: 'lucene'
 }
-

plugins/analysis-kuromoji/build.gradle

@@ -32,7 +32,7 @@ restResources {
   }
 }
 
-dependencyLicenses {
+tasks.named("dependencyLicenses").configure {
   mapping from: /lucene-.*/, to: 'lucene'
 }
 

plugins/analysis-nori/build.gradle

@@ -31,7 +31,6 @@ restResources {
     includeCore '_common', 'indices', 'index', 'search'
   }
 }
-dependencyLicenses {
+tasks.named("dependencyLicenses").configure {
   mapping from: /lucene-.*/, to: 'lucene'
 }
-

plugins/analysis-phonetic/build.gradle

@@ -33,6 +33,6 @@ restResources {
   }
 }
 
-dependencyLicenses {
+tasks.named("dependencyLicenses").configure {
   mapping from: /lucene-.*/, to: 'lucene'
 }

plugins/analysis-smartcn/build.gradle

@@ -32,7 +32,7 @@ restResources {
   }
 }
 
-dependencyLicenses {
+tasks.named("dependencyLicenses").configure {
   mapping from: /lucene-.*/, to: 'lucene'
 }
 

plugins/analysis-stempel/build.gradle

@@ -32,6 +32,6 @@ restResources {
   }
 }
 
-dependencyLicenses {
+tasks.named("dependencyLicenses").configure {
   mapping from: /lucene-.*/, to: 'lucene'
 }

plugins/analysis-ukrainian/build.gradle

@@ -35,7 +35,7 @@ restResources {
   }
 }
 
-dependencyLicenses {
+tasks.named("dependencyLicenses").configure {
   mapping from: /lucene-.*/, to: 'lucene'
   mapping from: /morfologik-.*/, to: 'lucene'
 }

plugins/discovery-azure-classic/build.gradle

@@ -99,7 +99,7 @@ normalization {
   }
 }
 
-dependencyLicenses {
+tasks.named("dependencyLicenses").configure {
   mapping from: /azure-.*/, to: 'azure'
   mapping from: /jackson-.*/, to: 'jackson'
   mapping from: /jersey-.*/, to: 'jersey'

plugins/discovery-ec2/build.gradle

@@ -46,7 +46,7 @@ restResources {
   }
 }
 
-dependencyLicenses {
+tasks.named("dependencyLicenses").configure {
   mapping from: /aws-java-sdk-.*/, to: 'aws-java-sdk'
   mapping from: /jackson-.*/, to: 'jackson'
 }

plugins/discovery-gce/build.gradle

@@ -27,7 +27,7 @@ restResources {
   }
 }
 
-dependencyLicenses {
+tasks.named("dependencyLicenses").configure {
   mapping from: /google-.*/, to: 'google'
 }
 

plugins/ingest-attachment/build.gradle

@@ -82,7 +82,7 @@ restResources {
   }
 }
 
-dependencyLicenses {
+tasks.named("dependencyLicenses").configure {
   mapping from: /apache-mime4j-.*/, to: 'apache-mime4j'
 }
 

plugins/repository-azure/build.gradle

@@ -42,7 +42,7 @@ restResources {
   }
 }
 
-dependencyLicenses {
+tasks.named("dependencyLicenses").configure {
   mapping from: /azure-.*/, to: 'azure'
   mapping from: /jackson-.*/, to: 'jackson'
   mapping from: /jersey-.*/, to: 'jersey'

plugins/repository-gcs/build.gradle

@@ -72,7 +72,7 @@ restResources {
   }
 }
 
-dependencyLicenses {
+tasks.named("dependencyLicenses").configure {
   mapping from: /google-cloud-.*/, to: 'google-cloud'
   mapping from: /google-auth-.*/, to: 'google-auth'
   mapping from: /google-http-.*/, to: 'google-http'

plugins/repository-hdfs/build.gradle

@@ -89,7 +89,7 @@ normalization {
   }
 }
 
-dependencyLicenses {
+tasks.named("dependencyLicenses").configure {
   mapping from: /hadoop-.*/, to: 'hadoop'
 }
 

plugins/repository-s3/build.gradle

@@ -53,7 +53,7 @@ dependencies {
   testImplementation project(':test:fixtures:s3-fixture')
 }
 
-dependencyLicenses {
+tasks.named("dependencyLicenses").configure {
   mapping from: /aws-java-sdk-.*/, to: 'aws-java-sdk'
   mapping from: /jmespath-java.*/, to: 'aws-java-sdk'
   mapping from: /jackson-.*/, to: 'jackson'

plugins/transport-nio/build.gradle

@@ -39,7 +39,7 @@ dependencies {
   compile "io.netty:netty-transport:${versions.netty}"
 }
 
-dependencyLicenses {
+tasks.named("dependencyLicenses").configure {
   mapping from: /netty-.*/, to: 'netty'
 }
 

qa/os/build.gradle

@@ -52,7 +52,7 @@ tasks.test.enabled = false
 testingConventions.enabled = false
 
 // this project doesn't get published
-tasks.dependencyLicenses.enabled = false
+tasks.named("dependencyLicenses").configure { it.enabled = false }
 tasks.dependenciesInfo.enabled = false
 
 tasks.thirdPartyAudit.ignoreMissingClasses()

qa/wildfly/build.gradle

@@ -85,7 +85,7 @@ check.dependsOn integTest
 
 test.enabled = false
 
-dependencyLicenses.enabled = false
+tasks.named("dependencyLicenses").configure { it.enabled = false }
 dependenciesInfo.enabled = false
 
 thirdPartyAudit.enabled = false

server/build.gradle

@@ -322,7 +322,7 @@ if (BuildParams.runtimeJavaVersion > JavaVersion.VERSION_1_8) {
   thirdPartyAudit.ignoreMissingClasses 'javax.xml.bind.DatatypeConverter'
 }
 
-dependencyLicenses {
+tasks.named("dependencyLicenses").configure {
   mapping from: /lucene-.*/, to: 'lucene'
   dependencies = project.configurations.runtime.fileCollection {
     it.group.startsWith('org.elasticsearch') == false ||

test/build.gradle

@@ -26,7 +26,7 @@ subprojects {
   apply plugin: 'elasticsearch.publish'
 
   // TODO: should we have licenses for our test deps?
-  dependencyLicenses.enabled = false
+  tasks.named("dependencyLicenses").configure { it.enabled = false }
   dependenciesInfo.enabled = false
 
   // TODO: why is the test framework pulled in...

test/framework/build.gradle

@@ -44,7 +44,7 @@ tasks.named('forbiddenApisMain').configure {
 }
 
 // TODO: should we have licenses for our test deps?
-dependencyLicenses.enabled = false
+tasks.named("dependencyLicenses").configure { it.enabled = false }
 dependenciesInfo.enabled = false
 
 thirdPartyAudit.ignoreMissingClasses(

x-pack/license-tools/build.gradle

@@ -10,7 +10,7 @@ project.forbiddenPatterns {
   exclude '**/*.key'
 }
 
-dependencyLicenses.enabled = false
+tasks.named("dependencyLicenses").configure { it.enabled = false }
 
 task buildZip(type: Zip, dependsOn: jar) {
   String parentDir = "license-tools-${version}"

x-pack/plugin/async-search/build.gradle

@@ -30,9 +30,5 @@ dependencies {
   testImplementation project(path: xpackModule('ilm'))
 }
 
-dependencyLicenses {
-  ignoreSha 'x-pack-core'
-}
-
 integTest.enabled = false
 

x-pack/plugin/ccr/build.gradle

@@ -46,10 +46,6 @@ dependencies {
   testImplementation project(path: xpackModule('monitoring'), configuration: 'testArtifacts')
 }
 
-dependencyLicenses {
-  ignoreSha 'x-pack-core'
-}
-
 testingConventions.naming {
   IT {
     baseClass "org.elasticsearch.xpack.CcrIntegTestCase"

x-pack/plugin/core/build.gradle

@@ -18,7 +18,7 @@ esplugin {
   requiresKeystore false
 }
 
-dependencyLicenses {
+tasks.named("dependencyLicenses").configure {
   mapping from: /http.*/, to: 'httpclient' // pulled in by rest client
   mapping from: /commons-.*/, to: 'commons' // pulled in by rest client
 }

x-pack/plugin/identity-provider/build.gradle

@@ -57,7 +57,7 @@ dependencies {
 compileJava.options.compilerArgs << "-Xlint:-rawtypes,-unchecked"
 compileTestJava.options.compilerArgs << "-Xlint:-rawtypes,-unchecked"
 
-dependencyLicenses {
+tasks.named("dependencyLicenses").configure {
   mapping from: /java-support|opensaml-.*/, to: 'shibboleth'
   mapping from: /http.*/, to: 'httpclient'
 }

x-pack/plugin/monitoring/build.gradle

@@ -38,7 +38,7 @@ artifacts {
   testArtifacts testJar
 }
 
-dependencyLicenses {
+tasks.named("dependencyLicenses").configure {
   mapping from: /http.*/, to: 'httpclient' // pulled in by rest client
   mapping from: /commons-.*/, to: 'commons' // pulled in by rest client
 }

x-pack/plugin/search-business-rules/build.gradle

@@ -11,7 +11,6 @@ esplugin {
 }
 archivesBaseName = 'x-pack-searchbusinessrules'
 
-
 integTest.enabled = false
 
 dependencies {
@@ -20,8 +19,3 @@ dependencies {
   testImplementation project(":test:framework")
 }
 
-// copied from CCR
-dependencyLicenses {
-  ignoreSha 'x-pack-core'
-}
-

x-pack/plugin/security/build.gradle

@@ -155,7 +155,7 @@ artifacts {
   testArtifacts testJar
 }
 
-dependencyLicenses {
+tasks.named("dependencyLicenses").configure {
   mapping from: /java-support|opensaml-.*/, to: 'shibboleth'
   mapping from: /http.*/, to: 'httpclient'
 }

x-pack/plugin/security/cli/build.gradle

@@ -19,7 +19,7 @@ dependencies {
   testImplementation project(path: xpackModule('core'), configuration: 'testArtifacts')
 }
 
-dependencyLicenses {
+tasks.named("dependencyLicenses").configure {
   mapping from: /bc.*/, to: 'bouncycastle'
 }
 

x-pack/plugin/sql/jdbc/build.gradle

@@ -30,14 +30,8 @@ dependencies {
   testImplementation project(path: xpackModule('core'), configuration: 'testArtifacts')
 }
 
-dependencyLicenses {
-  mapping from: /sql-proto.*/, to: 'elasticsearch'
-  mapping from: /sql-client.*/, to: 'elasticsearch'
+tasks.named("dependencyLicenses").configure {
   mapping from: /jackson-.*/, to: 'jackson'
-  mapping from: /elasticsearch-core.*/, to: 'elasticsearch'
-  ignoreSha 'sql-proto'
-  ignoreSha 'sql-client'
-  ignoreSha 'elasticsearch'
 }
 
 shadowJar {

x-pack/plugin/sql/qa/jdbc/build.gradle

@@ -15,7 +15,7 @@ dependencies {
  * other qa projects. */
 test.enabled = false
 
-dependencyLicenses.enabled = false
+tasks.named("dependencyLicenses").configure { it.enabled = false }
 dependenciesInfo.enabled = false
 
 // the main files are actually test files, so use the appropriate forbidden api sigs

x-pack/plugin/sql/qa/server/build.gradle

@@ -37,7 +37,7 @@ dependencies {
  * other qa projects. */
 test.enabled = false
 
-dependencyLicenses.enabled = false
+tasks.named("dependencyLicenses").configure { it.enabled = false }
 dependenciesInfo.enabled = false
 
 // just a test fixture: we aren't using this jars in releases and H2GIS requires disabling a lot of checks

x-pack/plugin/sql/sql-action/build.gradle

@@ -33,12 +33,9 @@ tasks.named('forbiddenApisMain').configure {
   replaceSignatureFiles 'jdk-signatures'
 }
 
-dependencyLicenses {
-  mapping from: /elasticsearch-core.*/, to: 'elasticsearch'
+tasks.named("dependencyLicenses").configure {
   mapping from: /jackson-.*/, to: 'jackson'
   mapping from: /lucene-.*/, to: 'lucene'
-  ignoreSha 'elasticsearch'
-  ignoreSha 'elasticsearch-core'
 }
 
 thirdPartyAudit.ignoreMissingClasses(

x-pack/plugin/sql/sql-cli/build.gradle

@@ -34,19 +34,12 @@ dependencies {
   testImplementation project(":test:framework")
 }
 
-dependencyLicenses {
-  mapping from: /elasticsearch-cli.*/, to: 'elasticsearch'
-  mapping from: /elasticsearch-core.*/, to: 'elasticsearch'
+tasks.named("dependencyLicenses").configure {
   mapping from: /jackson-.*/, to: 'jackson'
   mapping from: /lucene-.*/, to: 'lucene'
   mapping from: /sql-action.*/, to: 'elasticsearch'
   mapping from: /sql-client.*/, to: 'elasticsearch'
   mapping from: /jline-.*/, to: 'jline'
-  ignoreSha 'elasticsearch-cli'
-  ignoreSha 'elasticsearch-core'
-  ignoreSha 'elasticsearch'
-  ignoreSha 'sql-action'
-  ignoreSha 'sql-client'
 }
 
 shadowJar {

x-pack/plugin/sql/sql-client/build.gradle

@@ -13,12 +13,8 @@ dependencies {
   testImplementation project(path: xpackModule('core'), configuration: 'testArtifacts')
 }
 
-dependencyLicenses {
+tasks.named("dependencyLicenses").configure {
   mapping from: /jackson-.*/, to: 'jackson'
-  mapping from: /sql-proto.*/, to: 'elasticsearch'
-  mapping from: /elasticsearch-cli.*/, to: 'elasticsearch'
-  mapping from: /elasticsearch-core.*/, to: 'elasticsearch'
-  ignoreSha 'elasticsearch-core'
 }
 
 tasks.named('forbiddenApisMain').configure {

x-pack/plugin/sql/sql-proto/build.gradle

@@ -23,8 +23,6 @@ tasks.named('forbiddenApisMain').configure {
   replaceSignatureFiles 'jdk-signatures'
 }
 
-dependencyLicenses {
-  mapping from: /elasticsearch-core.*/, to: 'elasticsearch'
+tasks.named("dependencyLicenses").configure {
   mapping from: /jackson-.*/, to: 'jackson'
-  ignoreSha 'elasticsearch-core'
 }

x-pack/plugin/watcher/build.gradle

@@ -19,9 +19,8 @@ ext.compactProfile = 'full'
 compileJava.options.compilerArgs << "-Xlint:-rawtypes,-unchecked"
 compileTestJava.options.compilerArgs << "-Xlint:-rawtypes,-unchecked"
 
-dependencyLicenses {
+tasks.named("dependencyLicenses").configure {
   mapping from: /owasp-java-html-sanitizer.*/, to: 'owasp-java-html-sanitizer'
-  ignoreSha 'x-pack-core'
 }
 
 dependencies {