GEODE-6323 Fix dependency tracking for manifest jars (#3126)
GEODE-6323 Fix dependency tracking for manifest jars (#3126)

Because `doFirst` registers an action to be executed, it is not valid to be used as part of a task's configuration, causing aberrant behavior when combined with Gradle's output caching.

If "configuration" must occur at execution time, then that configuration needs to be the output of a task on which the "configured" task depends.  We have done that here to correctly generate the classpath in our dependency jars.  For more information, see
https://guides.gradle.org/using-build-cache/#suggestions_for_authoring_your_build

Co-authored-by: Robert Houghton <8ee5723e9b22f5849055c1010807733cae5c117f@pivotal.io>
Co-authored-by: Patrick Rhomberg <94aa6ac7d2dec5e282188d5611632f3ad54cf605@pivotal.io>

jdeppe-pivotal/geode,masaki-yamakawa/geode,smgoller/geode,smgoller/geode,davebarnes97/geode,masaki-yamakawa/geode,jdeppe-pivotal/geode,davinash/geode,jdeppe-pivotal/geode,masaki-yamakawa/geode,smgoller/geode,smgoller/geode,davebarnes97/geode,davebarnes97/geode,PurelyApplied/geode,jdeppe-pivotal/geode,masaki-yamakawa/geode,davinash/geode,PurelyApplied/geode,PurelyApplied/geode,PurelyApplied/geode,davinash/geode,PurelyApplied/geode,davinash/geode,smgoller/geode,masaki-yamakawa/geode,davinash/geode,jdeppe-pivotal/geode,smgoller/geode,smgoller/geode,masaki-yamakawa/geode,davinash/geode,davebarnes97/geode,davinash/geode,PurelyApplied/geode,PurelyApplied/geode,jdeppe-pivotal/geode,masaki-yamakawa/geode,jdeppe-pivotal/geode,davebarnes97/geode,davebarnes97/geode,davebarnes97/geode