cxf-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Freeman Fang <freeman.f...@gmail.com>
Subject Re: svn commit: r1302802 - in /cxf/branches/2.5.x-fixes
Date Tue, 20 Mar 2012 11:19:49 GMT

On 2012-3-20, at 下午6:19, Sergey Beryozkin wrote:

> Hi Freeman
> On 20/03/12 10:12, ffang@apache.org wrote:
>> Author: ffang
>> Date: Tue Mar 20 09:08:19 2012
>> New Revision: 1302802
>>
>> URL: http://svn.apache.org/viewvc?rev=1302802&view=rev
>> Log:
>> rt-transports-http module should optionally depend on spring
>>
>> Modified:
>>     cxf/branches/2.5.x-fixes/rt/bindings/http/pom.xml
>>     cxf/branches/2.5.x-fixes/rt/databinding/aegis/pom.xml
>>     cxf/branches/2.5.x-fixes/rt/databinding/xmlbeans/pom.xml
>>     cxf/branches/2.5.x-fixes/rt/frontend/jaxrs/pom.xml
>>     cxf/branches/2.5.x-fixes/rt/javascript/javascript-tests/pom.xml
>>     cxf/branches/2.5.x-fixes/rt/transports/http/pom.xml
>>     cxf/branches/2.5.x-fixes/systests/databinding/pom.xml
>>     cxf/branches/2.5.x-fixes/systests/jaxrs/pom.xml
>>     cxf/branches/2.5.x-fixes/systests/rs-security/pom.xml
>>     cxf/branches/2.5.x-fixes/systests/ws-security-examples/pom.xml
>>     cxf/branches/2.5.x-fixes/systests/ws-security/pom.xml
>>     cxf/branches/2.5.x-fixes/tools/javato/ws/pom.xml
>>
>> Modified: cxf/branches/2.5.x-fixes/rt/frontend/jaxrs/pom.xml
>> URL: http://svn.apache.org/viewvc/cxf/branches/2.5.x-fixes/rt/frontend/jaxrs/pom.xml?rev=1302802&r1=1302801&r2=1302802&view=diff
>> =
>> =
>> =
>> =
>> =
>> =
>> =
>> =
>> =
>> =====================================================================
>> --- cxf/branches/2.5.x-fixes/rt/frontend/jaxrs/pom.xml (original)
>> +++ cxf/branches/2.5.x-fixes/rt/frontend/jaxrs/pom.xml Tue Mar 20  
>> 09:08:19 2012
>> @@ -182,6 +182,10 @@
>>              <artifactId>easymock</artifactId>
>>              <scope>test</scope>
>>          </dependency>
>> +<dependency>
>> +<groupId>org.springframework</groupId>
>> +<artifactId>spring-context</artifactId>
>> +</dependency>
>>      </dependencies>
>
> Did you mean to add a 'test' scope here like in all the other  
> modules ?
> Cheers, Sergey
>
Hi Sergey,

No, because the main code in jaxrs also depend on spring, there's  
already a spring-core dependency. But I think the spring dependency  
should be optional there also, just like other modules,  wdyt?

Best Regards
Freeman

---------------------------------------------
Freeman Fang

FuseSource
Email:ffang@fusesource.com
Web: fusesource.com
Twitter: freemanfang
Blog: http://freemanfang.blogspot.com










Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message