Return-Path: Delivered-To: apmail-ws-axis-dev-archive@www.apache.org Received: (qmail 30188 invoked from network); 2 Oct 2008 16:30:42 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 2 Oct 2008 16:30:42 -0000 Received: (qmail 97913 invoked by uid 500); 2 Oct 2008 16:30:39 -0000 Delivered-To: apmail-ws-axis-dev-archive@ws.apache.org Received: (qmail 97637 invoked by uid 500); 2 Oct 2008 16:30:38 -0000 Mailing-List: contact axis-dev-help@ws.apache.org; run by ezmlm Precedence: bulk Reply-To: axis-dev@ws.apache.org list-help: list-unsubscribe: List-Post: List-Id: Delivered-To: mailing list axis-dev@ws.apache.org Received: (qmail 97612 invoked by uid 99); 2 Oct 2008 16:30:38 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 02 Oct 2008 09:30:38 -0700 X-ASF-Spam-Status: No, hits=2.0 required=10.0 tests=HTML_MESSAGE,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of keithgchapman@gmail.com designates 64.233.184.227 as permitted sender) Received: from [64.233.184.227] (HELO wr-out-0506.google.com) (64.233.184.227) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 02 Oct 2008 16:29:33 +0000 Received: by wr-out-0506.google.com with SMTP id 58so545500wri.8 for ; Thu, 02 Oct 2008 09:29:50 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:message-id:date:from:to :subject:in-reply-to:mime-version:content-type:references; bh=a3sLoAlI6lkTkdtBlR4OJLXLNI15Jw+zQ7gcXxr/nus=; b=ENDbuYui9hXLQsSDhaTr4EOL0YuEznLosTYedM0nvPvBAXIxWyMOUCq+ynpAIluw7z N/f84RoX1T3zrS3vDC35u5VlOGKsvZ2l22zI7NVZH4qz7GgfxQn7dwCBvfx2BUTuxgTC UXvTE6g6RFcKBKpLnvX6qpCNcK4MmqVI+CaJ4= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=message-id:date:from:to:subject:in-reply-to:mime-version :content-type:references; b=UViSXMSQP4dxq7v9zYIkEXWW+5DGOwZUV51zU+Ktex0gqRCNeooB6Htvpx/1Td1xpF 7lNpvFB1tbFRqUXeZ1tnTbmsK5CMcEdvXHy0lRhbfonM76p2rGOBT00lAzR2HngLmadD K8aR8/HipwbG4s3CERfAsH13V10UTgxWizokM= Received: by 10.151.45.2 with SMTP id x2mr37960ybj.114.1222964990619; Thu, 02 Oct 2008 09:29:50 -0700 (PDT) Received: by 10.151.154.12 with HTTP; Thu, 2 Oct 2008 09:29:50 -0700 (PDT) Message-ID: Date: Thu, 2 Oct 2008 21:59:50 +0530 From: "keith chapman" To: axis-dev@ws.apache.org Subject: Re: [Axis2] Moving all the transports into a common modules In-Reply-To: <672a01200810020924o5deebc0dn5663b48f6e77fb77@mail.gmail.com> MIME-Version: 1.0 Content-Type: multipart/alternative; boundary="----=_Part_4647_24736948.1222964990627" References: <48D59FC2.7020803@gmail.com> <1835B9D1-DB74-427C-B09C-CEC65A4B2F3F@skynet.be> <60708f4b0809212333u7f66430crb1e963d10a557a6c@mail.gmail.com> <60708f4b0809212355s30247b50w9776fdd40535fdc7@mail.gmail.com> <48E4E964.1040008@gmail.com> <672a01200810020924o5deebc0dn5663b48f6e77fb77@mail.gmail.com> X-Virus-Checked: Checked by ClamAV on apache.org ------=_Part_4647_24736948.1222964990627 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline I don't see any issue with backward compatibility. This chage is something in the axis2.xml, this will not break any code that users have with them. If it was a API call then I can understand. Also this change would not effect generated code (AFAIK). Thanks, Keith. On Thu, Oct 2, 2008 at 9:54 PM, Ruwan Linton wrote: > Hi Deepal, > > Synapse also has the same backward compatibility issue that you are talking > about with the axis2.xml that synapse ships with, therefore shall we keep > the transports that synapse community contributed as it is > (org.apache.synapse.transports) ;-) > > Thanks, > Ruwan > > > On Thu, Oct 2, 2008 at 9:01 PM, Deepal jayasinghe wrote: > >> >> > hi, >> > the package name that commons transport module use is org.apache.axis2 >> > >> > I think it should be org.apache.ws.commons >> Saying no one more time :) >> >> I do not think that all the modules in commons should have the package >> name org.apache.ws.commons , as an example have a look at Axiom and Neethi >> >> org.apache.neethi >> org.apache.axiom >> >> So IMO it is ok to have axis2 transport package name as it is :) >> >> -Deepal >> > >> > WDYT? >> > >> > thanks, >> > Amila. >> > >> > On Mon, Sep 22, 2008 at 12:03 PM, Amila Suriarachchi >> > > >> wrote: >> > >> > >> > >> > On Sun, Sep 21, 2008 at 2:59 PM, Andreas Veithen >> > > >> wrote: >> > >> > Deepal, >> > >> > Before we can move anything from Synapse to the new commons >> > module, we need to decide which transports we move (all or >> > only a subset) and based on that, we need to make sure that >> > all the people involved in the maintenance of these transports >> > have commit access to the new module. >> > >> > >> > As a starting point I'll put the synapse SMTP transport to commons >> > transport and try to test with Axis2. >> > >> > thanks, >> > Amila. >> > >> > >> > >> > In the meantime, I also have two comments/questions related to >> > the code that is already in the new module: >> > 1. Wouldn't it be a good idea to take advantage of the move >> > from Axis2 to ws-commons to use a more conventional directory >> > structure, e.g. src/main/java instead of src? >> > 2. I don't see any documentation in the new module. There must >> > have been some docs for the transports in Axis2. When will >> > this be moved? >> > >> > Regards, >> > >> > Andreas >> > >> > >> > On 21 sept. 08, at 03:13, Deepal jayasinghe wrote: >> > >> > Hi all, >> > >> > Few months back we all agreed to move all commons >> > transports (from Axis2 >> > and Synapse) to a common module. As the first step of >> > that I have moved >> > all the Axis2 to transport into a common module in >> > ws-commons [1]. In >> > addition to that we have setup nightly builds from that >> > modules. So now >> > its time for Synapse dev to move their transports into >> > that module :) >> > >> > [1] : >> > >> https://svn.apache.org/repos/asf/webservices/commons/trunk/modules/transport >> > >> > Thank you! >> > Deepal >> > >> > -- >> > Thank you! >> > >> > >> > http://blogs.deepal.org >> > >> > >> > >> --------------------------------------------------------------------- >> > To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org >> > >> > For additional commands, e-mail: >> > axis-dev-help@ws.apache.org >> > >> > >> > >> > >> > >> --------------------------------------------------------------------- >> > To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org >> > >> > For additional commands, e-mail: axis-dev-help@ws.apache.org >> > >> > >> > >> > >> > >> > -- >> > Amila Suriarachchi >> > WSO2 Inc. >> > blog: http://amilachinthaka.blogspot.com/ >> > >> > >> > >> > >> > -- >> > Amila Suriarachchi >> > WSO2 Inc. >> > blog: http://amilachinthaka.blogspot.com/ >> >> >> -- >> Thank you! >> >> >> http://blogs.deepal.org >> >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: dev-unsubscribe@synapse.apache.org >> For additional commands, e-mail: dev-help@synapse.apache.org >> >> > > > -- > Ruwan Linton > http://wso2.org - "Oxygenating the Web Services Platform" > http://ruwansblog.blogspot.com/ > -- Keith Chapman Senior Software Engineer WSO2 Inc. Oxygenating the Web Service Platform. http://wso2.org/ blog: http://www.keith-chapman.org ------=_Part_4647_24736948.1222964990627 Content-Type: text/html; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline
I don't see any issue with backward compatibility.  This chage is something in the axis2.xml, this will not break any code that users have with them. If it was a API call then I can understand.  Also this change would not effect generated code (AFAIK).

Thanks,
Keith.

On Thu, Oct 2, 2008 at 9:54 PM, Ruwan Linton <ruwan.linton@gmail.com> wrote:
Hi Deepal,

Synapse also has the same backward compatibility issue that you are talking about with the axis2.xml that synapse ships with, therefore shall we keep the transports that synapse community contributed as it is (org.apache.synapse.transports) ;-)

Thanks,
Ruwan


On Thu, Oct 2, 2008 at 9:01 PM, Deepal jayasinghe <deepalk@gmail.com> wrote:

> hi,
> the package name that commons transport module use is org.apache.axis2
>
> I think it should be org.apache.ws.commons
Saying no one more time :)

I do not think that all the modules in commons should have the package
name org.apache.ws.commons , as an example have a look at Axiom and Neethi

org.apache.neethi
org.apache.axiom

So IMO it is ok to have axis2 transport package name as it is :)

-Deepal
>
> WDYT?
>
> thanks,
> Amila.
>
> On Mon, Sep 22, 2008 at 12:03 PM, Amila Suriarachchi
> <amilasuriarachchi@gmail.com <mailto:amilasuriarachchi@gmail.com>> wrote:
>
>
>
>     On Sun, Sep 21, 2008 at 2:59 PM, Andreas Veithen
>     <andreas.veithen@skynet.be <mailto:andreas.veithen@skynet.be>> wrote:
>
>         Deepal,
>
>         Before we can move anything from Synapse to the new commons
>         module, we need to decide which transports we move (all or
>         only a subset) and based on that, we need to make sure that
>         all the people involved in the maintenance of these transports
>         have commit access to the new module.
>
>
>     As a starting point I'll put the synapse SMTP transport to commons
>     transport and try to test with Axis2.
>
>     thanks,
>     Amila.
>
>
>
>         In the meantime, I also have two comments/questions related to
>         the code that is already in the new module:
>         1. Wouldn't it be a good idea to take advantage of the move
>         from Axis2 to ws-commons to use a more conventional directory
>         structure, e.g. src/main/java instead of src?
>         2. I don't see any documentation in the new module. There must
>         have been some docs for the transports in Axis2. When will
>         this be moved?
>
>         Regards,
>
>         Andreas
>
>
>         On 21 sept. 08, at 03:13, Deepal jayasinghe wrote:
>
>             Hi all,
>
>             Few months back we all agreed to move all commons
>             transports (from Axis2
>             and Synapse) to a common module. As  the first step of
>             that I have moved
>             all the Axis2 to transport into a common module in
>             ws-commons [1]. In
>             addition to that we have setup nightly builds from that
>             modules. So now
>             its time for Synapse dev to move their transports into
>             that module :)
>
>             [1] :
>             https://svn.apache.org/repos/asf/webservices/commons/trunk/modules/transport
>
>             Thank you!
>             Deepal
>
>             --
>             Thank you!
>
>
>             http://blogs.deepal.org
>
>
>             ---------------------------------------------------------------------
>             To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
>             <mailto:axis-dev-unsubscribe@ws.apache.org>
>             For additional commands, e-mail:
>             axis-dev-help@ws.apache.org
>             <mailto:axis-dev-help@ws.apache.org>
>
>
>
>         ---------------------------------------------------------------------
>         To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
>         <mailto:axis-dev-unsubscribe@ws.apache.org>
>         For additional commands, e-mail: axis-dev-help@ws.apache.org
>         <mailto:axis-dev-help@ws.apache.org>
>
>
>
>
>     --
>     Amila Suriarachchi
>     WSO2 Inc.
>     blog: http://amilachinthaka.blogspot.com/
>
>
>
>
> --
> Amila Suriarachchi
> WSO2 Inc.
> blog: http://amilachinthaka.blogspot.com/


--
Thank you!


http://blogs.deepal.org


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@synapse.apache.org
For additional commands, e-mail: dev-help@synapse.apache.org




--
Ruwan Linton
http://wso2.org - "Oxygenating the Web Services Platform"
http://ruwansblog.blogspot.com/



--
Keith Chapman
Senior Software Engineer
WSO2 Inc.
Oxygenating the Web Service Platform.
http://wso2.org/

blog: http://www.keith-chapman.org
------=_Part_4647_24736948.1222964990627--