commons-beanutils is a transitive dependency.  I noticed this by inspection :-)

Based on the PR comments, I am not planning to use the override resolution 
strategy.

Anthony

> On Feb 22, 2017, at 1:22 PM, Kirk Lund <kl...@apache.org> wrote:
> 
> I'm comparing Anthony's PR with my changes for GEODE-2460, and I'm trying
> to understand why we need to add the override for commons-beanutils to
> gradle/dependency-resolution.gradle. Without this, GEODE-2460 passes
> precheckin and uiTest.
> 
> Is there some way to detect that commons-beanutils needs this further
> change?
> 
> Is there a gradle target that shows we end up using 2 different versions?
> 
> GEODE-2460: https://reviews.apache.org/r/56934/
> PR #403: https://github.com/apache/geode/pull/403
> 
> Thanks,
> Kirk

Reply via email to