cxf-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From James Mao <james....@iona.com>
Subject Re: commons-lang dependency?
Date Sun, 15 Jul 2007 01:44:38 GMT
After a quick search

./common/common/src/main/java/org/apache/cxf/jaxb/JAXBToStringStyle.java:import 
org.apache.commons.lang.builder.ToStringBuilder;
./common/common/src/main/java/org/apache/cxf/jaxb/JAXBToStringBuilder.java:import 
org.apache.commons.lang.builder.ToStringStyle;

./common/common/src/main/java/org/apache/cxf/jaxb/JAXBToStringStyle.java:importorg.apache.commons.lang.SystemUtils;
./common/common/src/main/java/org/apache/cxf/jaxb/JAXBToStringStyle.java:importorg.apache.commons.lang.builder.ToStringBuilder;
./common/common/src/main/java/org/apache/cxf/jaxb/JAXBToStringStyle.java:importorg.apache.commons.lang.builder.ToStringStyle;
./common/common/src/main/java/org/apache/cxf/jaxb/JAXBToStringStyle.java:     
*@see org.apache.commons.lang.builder.ToStringStyle

./common/common/src/test/java/org/apache/cxf/jaxb/JAXBElementToStringStyleTest.java:import

org.apache.commons.lang.builder.ToStringBuilder;

./systests/src/test/java/org/apache/cxf/systest/jaxb/model/ExtendedWidget.java:import 
org.apache.commons.lang.builder.EqualsBuilder;
./systests/src/test/java/org/apache/cxf/systest/jaxb/model/ExtendedWidget.java:import 
org.apache.commons.lang.builder.ToStringBuilder;
./systests/src/test/java/org/apache/cxf/systest/jaxb/model/Widget.java:import 
org.apache.commons.lang.builder.EqualsBuilder;
./systests/src/test/java/org/apache/cxf/systest/jaxb/model/Widget.java:import 
org.apache.commons.lang.builder.ToStringBuilder;


Looks like JAXBToStringStyle and JAXBToStringStyle heavily used in the 
xjc plugin

I think commons-lang is ok, will that be a problem?

James

> I see that the common module has a declared dependency on 
> commons-lang. Is
> this an optional dependency? Which modules require it?
>
> I'd be against a hard dependency on commons-lang because we've already 
> got a
> load of dependencies, but if its optional (i.e. only required for JBI or
> WS-*), we can just mark it as <optional> in the pom.
>
> Thanks,
> - Dan
>
>

Mime
View raw message