You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cxf.apache.org by Sergey Beryozkin <sb...@gmail.com> on 2012/03/20 11:19:06 UTC
Re: svn commit: r1302802 - in /cxf/branches/2.5.x-fixes
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
Re: svn commit: r1302802 - in /cxf/branches/2.5.x-fixes
Posted by Freeman Fang <fr...@gmail.com>.
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