Re: Review Request 17771: AURORA-193: Fix license plugin
> On Feb. 6, 2014, 5:23 a.m., Jake Farrell wrote: > > The new patch does not modify the gradle-license-plugin sourceSet as > > intended, it ends up using the default of sourceSet.all in the plugin which > > grabs all resources, dist, and 3rdparty dirs. Digging into the code the > > license plugin extends SourceTask, which has the setSource function to > > replace all sourceSets within the SourceTask, leveraging this for any > > License task type we can set the paths we want to use and this fixes the > > issue. > > > > testing done: rm -rf .gradle && ./gradlew clean build > > > > Changes needed to make are: > > > > tasks.withType(nl.javadude.gradle.plugins.license.License).each { > > licenseTask -> > > licenseTask.setSource files("$projectDir/src/main/java", > > "$projectDir/src/test/java") > > } > > > > license { > > header rootProject.file('config/checkstyle/apache.header') > > strictCheck true > > skipExistingHeaders true > > ext.year = Calendar.getInstance().get(Calendar.YEAR) > > } > > new review submitted with these updates: https://reviews.apache.org/r/18316/ - Jake --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17771/#review33762 --- On Feb. 6, 2014, 1:55 a.m., Kevin Sweeney wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/17771/ > --- > > (Updated Feb. 6, 2014, 1:55 a.m.) > > > Review request for Aurora, Jake Farrell and Maxim Khutornenko. > > > Bugs: AURORA-193 > https://issues.apache.org/jira/browse/AURORA-193 > > > Repository: aurora > > > Description > --- > > Copy sourceSets instead of mutating it. > > > Diffs > - > > build.gradle 4ba72c109b92d05581dbfe78194518653f20e12d > > Diff: https://reviews.apache.org/r/17771/diff/ > > > Testing > --- > > ./gradlew clean build > > > Thanks, > > Kevin Sweeney > >
Re: Review Request 17771: AURORA-193: Fix license plugin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17771/#review33762 --- The new patch does not modify the gradle-license-plugin sourceSet as intended, it ends up using the default of sourceSet.all in the plugin which grabs all resources, dist, and 3rdparty dirs. Digging into the code the license plugin extends SourceTask, which has the setSource function to replace all sourceSets within the SourceTask, leveraging this for any License task type we can set the paths we want to use and this fixes the issue. testing done: rm -rf .gradle && ./gradlew clean build Changes needed to make are: tasks.withType(nl.javadude.gradle.plugins.license.License).each { licenseTask -> licenseTask.setSource files("$projectDir/src/main/java", "$projectDir/src/test/java") } license { header rootProject.file('config/checkstyle/apache.header') strictCheck true skipExistingHeaders true ext.year = Calendar.getInstance().get(Calendar.YEAR) } - Jake Farrell On Feb. 6, 2014, 1:55 a.m., Kevin Sweeney wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/17771/ > --- > > (Updated Feb. 6, 2014, 1:55 a.m.) > > > Review request for Aurora, Jake Farrell and Maxim Khutornenko. > > > Bugs: AURORA-193 > https://issues.apache.org/jira/browse/AURORA-193 > > > Repository: aurora > > > Description > --- > > Copy sourceSets instead of mutating it. > > > Diffs > - > > build.gradle 4ba72c109b92d05581dbfe78194518653f20e12d > > Diff: https://reviews.apache.org/r/17771/diff/ > > > Testing > --- > > ./gradlew clean build > > > Thanks, > > Kevin Sweeney > >
Re: Review Request 17771: AURORA-193: Fix license plugin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17771/ --- (Updated Feb. 5, 2014, 5:55 p.m.) Review request for Aurora, Jake Farrell and Maxim Khutornenko. Changes --- Attach diff. Bugs: AURORA-193 https://issues.apache.org/jira/browse/AURORA-193 Repository: aurora Description (updated) --- Copy sourceSets instead of mutating it. Diffs (updated) - build.gradle 4ba72c109b92d05581dbfe78194518653f20e12d Diff: https://reviews.apache.org/r/17771/diff/ Testing --- ./gradlew clean build Thanks, Kevin Sweeney
Re: Review Request 17771: AURORA-193: Fix license plugin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17771/#review33749 --- Ship it! Ship It! - Maxim Khutornenko On Feb. 6, 2014, 1:27 a.m., Kevin Sweeney wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/17771/ > --- > > (Updated Feb. 6, 2014, 1:27 a.m.) > > > Review request for Aurora, Jake Farrell and Maxim Khutornenko. > > > Bugs: AURORA-193 > https://issues.apache.org/jira/browse/AURORA-193 > > > Repository: aurora > > > Description > --- > > Copy sourceSets instead of mutating it. > > Currently available here https://paste.apache.org/2wK3 > > > Diffs > - > > > Diff: https://reviews.apache.org/r/17771/diff/ > > > Testing > --- > > ./gradlew clean build > > > Thanks, > > Kevin Sweeney > >
Review Request 17771: AURORA-193: Fix license plugin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17771/ --- Review request for Aurora, Jake Farrell and Maxim Khutornenko. Summary (updated) - AURORA-193: Fix license plugin Bugs: AURORA-193 https://issues.apache.org/jira/browse/AURORA-193 Repository: aurora Description (updated) --- Copy sourceSets instead of mutating it. Currently available here https://paste.apache.org/2wK3 Diffs - Diff: https://reviews.apache.org/r/17771/diff/ Testing (updated) --- ./gradlew clean build Thanks, Kevin Sweeney