[ 
https://issues.apache.org/jira/browse/GEODE-6611?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16814760#comment-16814760
 ] 

ASF subversion and git services commented on GEODE-6611:
--------------------------------------------------------

Commit 20b4fbbc457397d252c91b54f756d5dd0d65486d in geode's branch 
refs/heads/develop from Robert Houghton
[ https://gitbox.apache.org/repos/asf?p=geode.git;h=20b4fbb ]

GEODE-6611: Audit geode-all-bom plugin inclusions

Only include plugins used in the project. In this case, our `java`
plugin, not the rest

Authored-by: Robert Houghton <[email protected]>


> Subprojects should be audited with respect to which plugins aught be applied
> ----------------------------------------------------------------------------
>
>                 Key: GEODE-6611
>                 URL: https://issues.apache.org/jira/browse/GEODE-6611
>             Project: Geode
>          Issue Type: Improvement
>            Reporter: Patrick Rhomberg
>            Priority: Major
>          Time Spent: 20m
>  Remaining Estimate: 0h
>
> Previously, every Gradle subproject was aggressively configured via blocks 
> like
> {noformat}
> subprojets {
>   apply plugin: 'java'
>   ...
> }
> {noformat}
> As the code base expanded, we have introduced many subprojects who do not 
> warrant a "standard" configuration as a Java project.  For instance, in 
> GEODE-6569, a subproject responsible for producing Geode's BOM was also 
> producing a trivial jar.  This is the direct result of a configuration like 
> the above.
> Modularity was restored by GEODE-6383 and such invasive configuration no 
> longer exists.  We will soon be positioned to remove from such subprojects 
> those plugins that do not belong.
> Each subproject should be audited and only pull in the plugins necessary.  
> This may resolve many tangental issues, particularly with publication, e.g., 
> the war subproject publishing jars not fit for (direct) consumption.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to