axis-java-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Davanum Srinivas <dava...@gmail.com>
Subject Re: [axis2] move axiom to ws-commons
Date Mon, 30 Jan 2006 20:56:08 GMT
Sanjiva,

I have no problem doing it. I did not see a *strong* need from a code
dependency point of view. If we just want do it, because we want to do
it, then it's fine :)

I was just pointing out that "circular dependency" is not an issue :) :)

-- dims

On 1/30/06, Sanjiva Weerawarana <sanjiva@opensource.lk> wrote:
> On Mon, 2006-01-30 at 13:50 -0500, Davanum Srinivas wrote:
> > I've reviewed the code in question, there are exactly 2 files in
> > policy project that uses axiom namely
> >
> > - OMPolicyReader : OMPolicyReader is loaded on demand (using a class
> > loader) from PolicyFactory.java. So moving it to axis2 core should not
> > be a problem.
> > - PolicyAttachmentUtil : PolicyAttachmentUtil is not even referenced
> > from any other files in policy project. So there should not be a
> > problem moving it to axis2 core.
> >
> > So, we don't have to do this right now. IMHO, We can defer this
> > surgery for later.
>
> But this is the wrong thing to do from the point of view of the policy
> model project.
>
> What's the reason for deferring the surgery? Its a relatively minor
> surgery; easily outpatient care! ;-)
>
> Sanjiva.
>
>
>


--
Davanum Srinivas : http://wso2.com/blogs/

Mime
View raw message