This certainly should fix it. I removed the version previously and it was using the same version as everything else, though. Either way works.
On 4 August 2014 10:19, Gary Gregory <[email protected]> wrote: > The dependency report shows the perf and core modules using different > versions of the Disruptor. I thought this would fix it. > > Gary > > > On Mon, Aug 4, 2014 at 10:53 AM, Matt Sicker <[email protected]> wrote: > >> But this uses the parent pom version otherwise. I didn't think it was >> necessary to have! But it's certainly a better idea to use the property >> rather than hardcoded. >> >> >> On 4 August 2014 08:35, Remko Popma <[email protected]> wrote: >> >>> Thanks! >>> >>> >>> On Mon, Aug 4, 2014 at 10:29 PM, <[email protected]> wrote: >>> >>>> Author: ggregory >>>> Date: Mon Aug 4 13:29:26 2014 >>>> New Revision: 1615552 >>>> >>>> URL: http://svn.apache.org/r1615552 >>>> Log: >>>> Add missing Disruptor version. >>>> >>>> Modified: >>>> logging/log4j/log4j2/trunk/log4j-perf/pom.xml >>>> >>>> Modified: logging/log4j/log4j2/trunk/log4j-perf/pom.xml >>>> URL: >>>> http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/log4j-perf/pom.xml?rev=1615552&r1=1615551&r2=1615552&view=diff >>>> >>>> ============================================================================== >>>> --- logging/log4j/log4j2/trunk/log4j-perf/pom.xml (original) >>>> +++ logging/log4j/log4j2/trunk/log4j-perf/pom.xml Mon Aug 4 13:29:26 >>>> 2014 >>>> @@ -80,6 +80,7 @@ >>>> <dependency> >>>> <groupId>com.lmax</groupId> >>>> <artifactId>disruptor</artifactId> >>>> + <version>${disruptor.version}</version> >>>> </dependency> >>>> </dependencies> >>>> >>>> >>>> >>>> >>> >> >> >> -- >> Matt Sicker <[email protected]> >> > > > > -- > E-Mail: [email protected] | [email protected] > Java Persistence with Hibernate, Second Edition > <http://www.manning.com/bauer3/> > JUnit in Action, Second Edition <http://www.manning.com/tahchiev/> > Spring Batch in Action <http://www.manning.com/templier/> > Blog: http://garygregory.wordpress.com > Home: http://garygregory.com/ > Tweet! http://twitter.com/GaryGregory > -- Matt Sicker <[email protected]>
