From a7e470dd929f124d0b9a2f9fcbcbc66209ca83ce Mon Sep 17 00:00:00 2001 From: yuri1969 <1969yuri1969@gmail.com> Date: Wed, 30 Mar 2022 21:17:02 +0200 Subject: [PATCH] Reduce exported transient depencencies Commit 1192aa1 migrated Gradle dependency configurations from `provided` to `implementation`. This altered the build output. Particularly, the Grails pipeline exports many unnecessary depencencies. AFAIK Gradle's `compileOnly` is the replacement for the deprecated `provided` configuration, is it not? --- asset-pipeline-classpath-test/build.gradle | 4 ++-- asset-pipeline-core/build.gradle | 6 +++--- asset-pipeline-grails/build.gradle | 17 ++++++++--------- asset-pipeline-spring-boot/build.gradle | 2 +- coffee-asset-pipeline/build.gradle | 2 +- compass-asset-pipeline/build.gradle | 2 +- groocss-asset-pipeline/build.gradle | 2 +- handlebars-asset-pipeline/build.gradle | 2 +- jsx-asset-pipeline/build.gradle | 2 +- less-asset-pipeline/build.gradle | 2 +- sass-asset-pipeline/build.gradle | 2 +- typescript-asset-pipeline/build.gradle | 2 +- 12 files changed, 22 insertions(+), 23 deletions(-) diff --git a/asset-pipeline-classpath-test/build.gradle b/asset-pipeline-classpath-test/build.gradle index 1726a208..1f700f2c 100644 --- a/asset-pipeline-classpath-test/build.gradle +++ b/asset-pipeline-classpath-test/build.gradle @@ -36,5 +36,5 @@ repositories { dependencies { - implementation localGroovy() -} \ No newline at end of file + compileOnly localGroovy() +} diff --git a/asset-pipeline-core/build.gradle b/asset-pipeline-core/build.gradle index d55f87a9..9b89b16c 100644 --- a/asset-pipeline-core/build.gradle +++ b/asset-pipeline-core/build.gradle @@ -55,9 +55,9 @@ groovydoc { } dependencies { - implementation 'org.codehaus.groovy:groovy:2.4.19' - implementation 'org.codehaus.groovy:groovy-json:2.4.19' - implementation 'org.codehaus.groovy:groovy-templates:2.4.19' + compileOnly 'org.codehaus.groovy:groovy:2.4.19' + compileOnly 'org.codehaus.groovy:groovy-json:2.4.19' + compileOnly 'org.codehaus.groovy:groovy-templates:2.4.19' doc 'org.codehaus.groovy:groovy-all:2.4.19' doc 'org.fusesource.jansi:jansi:1.11' api 'org.mozilla:rhino:1.7R4' diff --git a/asset-pipeline-grails/build.gradle b/asset-pipeline-grails/build.gradle index 5a8238ec..9d58faf2 100644 --- a/asset-pipeline-grails/build.gradle +++ b/asset-pipeline-grails/build.gradle @@ -111,15 +111,14 @@ publishing { dependencies { - implementation 'org.springframework.boot:spring-boot-starter-logging' - implementation "org.springframework.boot:spring-boot-starter-actuator" - implementation "org.springframework.boot:spring-boot-autoconfigure" - implementation "org.springframework.boot:spring-boot-starter-tomcat" - - implementation "org.grails:grails-web-boot" - implementation "org.grails:grails-dependencies" - implementation 'javax.servlet:javax.servlet-api:3.1.0' - implementation "org.grails:grails-dependencies" + compileOnly 'org.springframework.boot:spring-boot-starter-logging' + compileOnly "org.springframework.boot:spring-boot-starter-actuator" + compileOnly "org.springframework.boot:spring-boot-autoconfigure" + compileOnly "org.springframework.boot:spring-boot-starter-tomcat" + + compileOnly "org.grails:grails-web-boot" + compileOnly "org.grails:grails-dependencies" + compileOnly 'javax.servlet:javax.servlet-api:3.1.0' implementation project(':asset-pipeline-core'), { exclude group:'org.mozilla', module:'rhino' exclude group:'com.google.javascript', module:'closure-compiler-unshaded' diff --git a/asset-pipeline-spring-boot/build.gradle b/asset-pipeline-spring-boot/build.gradle index 9fadccf5..d95faedb 100644 --- a/asset-pipeline-spring-boot/build.gradle +++ b/asset-pipeline-spring-boot/build.gradle @@ -43,7 +43,7 @@ repositories { } dependencies { - implementation 'org.codehaus.groovy:groovy:2.4.19' + compileOnly 'org.codehaus.groovy:groovy:2.4.19' api project(':asset-pipeline-core') api project(':asset-pipeline-servlet') api 'org.springframework.boot:spring-boot-starter-web:1.5.1.RELEASE' diff --git a/coffee-asset-pipeline/build.gradle b/coffee-asset-pipeline/build.gradle index 534f81a1..3ffb1f80 100644 --- a/coffee-asset-pipeline/build.gradle +++ b/coffee-asset-pipeline/build.gradle @@ -40,7 +40,7 @@ sourceSets { dependencies { api project(':asset-pipeline-core') - implementation 'org.codehaus.groovy:groovy:2.4.19' + compileOnly 'org.codehaus.groovy:groovy:2.4.19' api 'org.mozilla:rhino:1.7R4' testImplementation "org.spockframework:spock-core:1.3-groovy-2.4" diff --git a/compass-asset-pipeline/build.gradle b/compass-asset-pipeline/build.gradle index e903ca7b..b90655c0 100644 --- a/compass-asset-pipeline/build.gradle +++ b/compass-asset-pipeline/build.gradle @@ -43,7 +43,7 @@ sourceSets { } dependencies { - implementation 'org.codehaus.groovy:groovy-all:2.4.19' + compileOnly 'org.codehaus.groovy:groovy-all:2.4.19' api project(':asset-pipeline-core') // api 'log4j:log4j:1.2.17' // api 'org.yaml:snakeyaml:1.26' diff --git a/groocss-asset-pipeline/build.gradle b/groocss-asset-pipeline/build.gradle index b1ef5118..bccb8330 100644 --- a/groocss-asset-pipeline/build.gradle +++ b/groocss-asset-pipeline/build.gradle @@ -40,7 +40,7 @@ sourceSets { } dependencies { - implementation 'org.codehaus.groovy:groovy-all:2.4.19' + compileOnly 'org.codehaus.groovy:groovy-all:2.4.19' api project(':asset-pipeline-core') api "org.groocss:groocss:1.0-M1" diff --git a/handlebars-asset-pipeline/build.gradle b/handlebars-asset-pipeline/build.gradle index c1cbdee7..8f57c268 100644 --- a/handlebars-asset-pipeline/build.gradle +++ b/handlebars-asset-pipeline/build.gradle @@ -41,7 +41,7 @@ sourceSets { dependencies { api project(':asset-pipeline-core') - implementation 'org.codehaus.groovy:groovy-all:2.4.19' + compileOnly 'org.codehaus.groovy:groovy-all:2.4.19' api 'org.mozilla:rhino:1.7R4' // api 'log4j:log4j:1.2.17' diff --git a/jsx-asset-pipeline/build.gradle b/jsx-asset-pipeline/build.gradle index 5306c2df..1f01f34b 100644 --- a/jsx-asset-pipeline/build.gradle +++ b/jsx-asset-pipeline/build.gradle @@ -45,7 +45,7 @@ java { dependencies { api project(':asset-pipeline-core') - implementation 'org.codehaus.groovy:groovy:2.4.19' + compileOnly 'org.codehaus.groovy:groovy:2.4.19' api 'org.mozilla:rhino:1.7R4' testImplementation "org.spockframework:spock-core:1.3-groovy-2.4" diff --git a/less-asset-pipeline/build.gradle b/less-asset-pipeline/build.gradle index 90bf356a..9b2b87e3 100644 --- a/less-asset-pipeline/build.gradle +++ b/less-asset-pipeline/build.gradle @@ -43,7 +43,7 @@ sourceSets { } dependencies { - implementation 'org.codehaus.groovy:groovy-all:2.4.19' + compileOnly 'org.codehaus.groovy:groovy-all:2.4.19' api project(':asset-pipeline-core') api 'org.mozilla:rhino:1.7R4' api 'org.slf4j:slf4j-api:1.7.28' diff --git a/sass-asset-pipeline/build.gradle b/sass-asset-pipeline/build.gradle index 4ec05fa0..95ccc084 100644 --- a/sass-asset-pipeline/build.gradle +++ b/sass-asset-pipeline/build.gradle @@ -42,7 +42,7 @@ java { } dependencies { - implementation 'org.codehaus.groovy:groovy-all:2.4.19' + compileOnly 'org.codehaus.groovy:groovy-all:2.4.19' api project(':asset-pipeline-core') api 'org.slf4j:slf4j-api:1.7.28' api 'io.bit3:jsass:5.10.4' diff --git a/typescript-asset-pipeline/build.gradle b/typescript-asset-pipeline/build.gradle index 61c9f443..1236a261 100644 --- a/typescript-asset-pipeline/build.gradle +++ b/typescript-asset-pipeline/build.gradle @@ -41,7 +41,7 @@ java { dependencies { api project(':asset-pipeline-core') api project(':jsx-asset-pipeline') - implementation 'org.codehaus.groovy:groovy:2.4.19' + compileOnly 'org.codehaus.groovy:groovy:2.4.19' api 'org.mozilla:rhino:1.7R4' // api 'log4j:log4j:1.2.17'