Return-Path: Delivered-To: apmail-beehive-dev-archive@www.apache.org Received: (qmail 84155 invoked from network); 25 Jul 2006 17:12:45 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (209.237.227.199) by minotaur.apache.org with SMTP; 25 Jul 2006 17:12:45 -0000 Received: (qmail 25710 invoked by uid 500); 25 Jul 2006 17:12:45 -0000 Delivered-To: apmail-beehive-dev-archive@beehive.apache.org Received: (qmail 25519 invoked by uid 500); 25 Jul 2006 17:12:43 -0000 Mailing-List: contact dev-help@beehive.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: "Beehive Developers" Delivered-To: mailing list dev@beehive.apache.org Received: (qmail 25502 invoked by uid 99); 25 Jul 2006 17:12:43 -0000 Received: from asf.osuosl.org (HELO asf.osuosl.org) (140.211.166.49) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 25 Jul 2006 10:12:43 -0700 X-ASF-Spam-Status: No, hits=0.8 required=10.0 tests=DNS_FROM_RFC_ABUSE,HTML_MESSAGE,MAILTO_TO_SPAM_ADDR,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (asf.osuosl.org: domain of rajith77@gmail.com designates 66.249.92.171 as permitted sender) Received: from [66.249.92.171] (HELO ug-out-1314.google.com) (66.249.92.171) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 25 Jul 2006 10:12:42 -0700 Received: by ug-out-1314.google.com with SMTP id c2so3021804ugf for ; Tue, 25 Jul 2006 10:12:20 -0700 (PDT) DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=beta; d=gmail.com; h=received:message-id:date:from:to:subject:in-reply-to:mime-version:content-type:references; b=WI2XeHfAFcQqAmTVESXT8suMnfQWCMIowWJlwkT1N6Nd90F9ZdAkFnQ1Vq5qvluds4vMlb603Ycd5sugNxTAISv3lA3uhB8tCKbu6Tqzqbfk0EvVSpOeQXMFolP8L+w1wFeSZEYFIMQXHXaxbsK1VErKnUHdoxZ4tmznRdxVbUg= Received: by 10.78.140.17 with SMTP id n17mr2443328hud; Tue, 25 Jul 2006 10:12:20 -0700 (PDT) Received: by 10.78.193.8 with HTTP; Tue, 25 Jul 2006 10:12:19 -0700 (PDT) Message-ID: Date: Tue, 25 Jul 2006 13:12:19 -0400 From: "Rajith Attapattu" To: "Beehive Developers" Subject: Re: JSR 181 support for axis2 In-Reply-To: MIME-Version: 1.0 Content-Type: multipart/alternative; boundary="----=_Part_11850_4333192.1153847539927" References: <19e0530f0607250615m330067e7m60596098e0a67e04@mail.gmail.com> X-Virus-Checked: Checked by ClamAV on apache.org X-Spam-Rating: minotaur.apache.org 1.6.2 0/1000/N ------=_Part_11850_4333192.1153847539927 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Content-Disposition: inline Eddiie, Take it easy :-) Rajith On 7/25/06, Eddie O'Neil wrote: > > Rajith-- > > Great -- sounds good. I'm tied up today (day job and all...) but > will spin up a thread on open questions on axis-dev@ tonight. > > The interminable patience on this is greatly appreciated. :) > > Eddie > > > On 7/25/06, Rajith Attapattu wrote: > > Eddie, > > > > Thanks and my pleasure to do the patch. > > Once we have the changes and a build in the repo, Dims can work on the > patch > > for axis2. > > > > Thank you once again. > > > > Re: The validation part and the WSDL stuff, lets take it to the > axis-list as > > you suggested. > > We can start by publishing the questions u had on it the last time we > > emailed. > > > > Regards, > > > > Rajith > > > > On 7/25/06, Eddie O'Neil wrote: > > > > > > Rajith-- > > > > > > Yeah -- I started looking at the patch last night and have it up and > > > running with the existing WSM tests passing. I've not done the work > > > to just copy the source files annogen creates, but that's the easy > > > part. :) Thanks for the patch! > > > > > > Dims-- > > > > > > No, we've not done a nightly build for a while. With Rajith's patch > > > (and external eyes that need to be on a WSM distributable), I was > > > going to start publishing a WSM snapshot to a Maven2 repo on > > > people.apache.org/~ekoneil. > > > > > > Sound good? > > > > > > Eddie > > > > > > > > > > > > On 7/25/06, Davanum Srinivas wrote: > > > > does beehive have a nightly build? are the snapshots published to > any > > > > maven repo? > > > > > > > > thx, > > > > dims > > > > > > > > On 7/25/06, Rajith Attapattu wrote: > > > > > Eddie, > > > > > > > > > > Dims also things its fine to include the generated AnnoBeans in > the > > > source > > > > > treee and then remove the meta annotations from the spec files, > thus > > > > > avoiding the legal issue. > > > > > > > > > > Eddie, can u then check the patch and commit with nessacery > changes. > > > > > > > > > > Note the task to generate AnnoBeans is commented out !!! > > > > > Also u need to javac the generated source files from the > new > > > > > location within the source tree instead of the temp directory > where it > > > was > > > > > generated. > > > > > > > > > > I will attach the generated source files as a patch. (pls add it > to > > > the main > > > > > source tree) > > > > > > > > > > Let me know how it goes. > > > > > If we can sort this out on the wsm side then I can sort it out on > the > > > axis2 > > > > > side with Dims help :-) > > > > > > > > > > Eddie, thanks for your support. > > > > > > > > > > Regards, > > > > > > > > > > Rajith > > > > > 1- 416- 482- 2661 x 308 > > > > > > > > > > On 7/24/06, Rajith Attapattu wrote: > > > > > > > > > > > > Hey Eddie, > > > > > > > > > > > > Thats what I thought too. Once we generate the AnnoBeans we can > > > delete the > > > > > > annotations from the spec file. > > > > > > We can include the generated source as part of the permanent > source > > > tree. > > > > > > > > > > > > Since this is a spec file there is no need to generate the > AnnoBeans > > > over > > > > > > and over as the spec files are static, hence the generated > source > > > never > > > > > > change. > > > > > > So lets do that if you are ok with it. > > > > > > > > > > > > Regards, > > > > > > > > > > > > Rajith > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On 7/24/06, Eddie O'Neil wrote: > > > > > > > > > > > > > > A'right -- mail is in legal-discuss@. Will be interesting > to > > > see > > > > > > > how that conversation turns out. :) Interested parties > should > > > follow > > > > > > > the discussion there. > > > > > > > > > > > > > > Eddie > > > > > > > > > > > > > > > > > > > > > On 7/23/06, Eddie O'Neil wrote: > > > > > > > > Rajith-- > > > > > > > > > > > > > > > > Hey -- something came up this weekend and I am just > getting > > > back to > > > > > > > > this now. My gut is that this type of metadata modification > to > > > an API > > > > > > > > class isn't going to fly (mail about this shortly); since > the > > > annogen > > > > > > > > beans are just generated from the annotations themselves, > can we > > > just > > > > > > > > hand code them to match the JSR-181 annotations? > > > > > > > > > > > > > > > > Will take a look at the patch... > > > > > > > > > > > > > > > > Eddie > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On 7/23/06, Rajith Attapattu wrote: > > > > > > > > > Eddie, > > > > > > > > > > > > > > > > > > any update on the issue? > > > > > > > > > > > > > > > > > > Regards, > > > > > > > > > > > > > > > > > > Rajith > > > > > > > > > > > > > > > > > > On 7/20/06, Rajith Attapattu wrote: > > > > > > > > > > > > > > > > > > > > Eddie, > > > > > > > > > > > > > > > > > > > > I totally forgot about sending the patch, but finnaly > did it > > > > > > > today. > > > > > > > > > > Please review it and let me know your comments. > > > > > > > > > > > > > > > > > > > > We still need to figure out the legal issue about adding > an > > > > > > > annotation to > > > > > > > > > > the spec class. > > > > > > > > > > (Does annogen have a way around without annotatiing the > > > classes, > > > > > > > for it to > > > > > > > > > > generate the AnnoBean classes.???) > > > > > > > > > > > > > > > > > > > > > 2.) Generating Annogen beans for meta data > > > > > > > > > > > The annogen task that generates code based on the > > > annotated > > > > > > > classes > > > > > > > > > > > seems to have a bug with inner classes. > > > > > > > > > > > For example WebParam.Mode gives compilation errors > as > > > it > > > > > > > cannot the > > > > > > > > > > > handle the inner class > > > > > > > > > > > So I eidted the generated source file to get it > > > working. (I > > > > > > > maybe > > > > > > > > > > wrong > > > > > > > > > > > here...) > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > This is still an issue :-( > > > > > > > > > > > > > > > > > > > > Regards, > > > > > > > > > > > > > > > > > > > > Rajith > > > > > > > > > > > > > > > > > > > > On 7/11/06, Rajith Attapattu < rajith77@gmail.com> > wrote: > > > > > > > > > > > > > > > > > > > > > > Eddie, > > > > > > > > > > > > > > > > > > > > > > oops, didn't think it was that serious about modifying > the > > > spec > > > > > > > classes. > > > > > > > > > > > But can u please let me know about it. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >[eko] Not sure I follow this -- let me take a look at > it > > > and > > > > > > > I'll get > > > > > > > > > > > >back to you. > > > > > > > > > > > I didn't even submit the patch for this part. I will > do > > > so. > > > > > > > > > > > Please take a look at it then. > > > > > > > > > > > > > > > > > > > > > > I am really gratefull for your assitance. > > > > > > > > > > > > > > > > > > > > > > Regards, > > > > > > > > > > > > > > > > > > > > > > Rajith > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On 7/11/06, Eddie O'Neil wrote: > > > > > > > > > > > > > > > > > > > > > > > > Rajith-- > > > > > > > > > > > > > > > > > > > > > > > > Comments on both of your questions below... > > > > > > > > > > > > > > > > > > > > > > > > Eddie > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > 1.) Annogen requires you to annotate the > annotation > > > classes > > > > > > > with > > > > > > > > > > > > the > > > > > > > > > > > > > following annotation > > > > > > > > > > > > > Now are we allowed to modifty the JSR api classes > to > > > add the > > > > > > > above > > > > > > > > > > > > lines of > > > > > > > > > > > > > code ?? > > > > > > > > > > > > > > > > > > > > > > > > [eko] This is a *great* question :) and probably > the > > > first > > > > > > > time > > > > > > > > > > > > we've run into this at Apache. If you were asking > to > > > add a > > > > > > > *method* > > > > > > > > > > > > to a type described in a specification, the answer > would > > > be no > > > > > > > because > > > > > > > > > > > > > > > > > > > > > > > > we can't change spec classes. Since it's metadata, > I > > > don't > > > > > > > know the > > > > > > > > > > > > answer -- my gut would be that we can't change the > > > metadata on > > > > > > > a > > > > > > > > > > > > specification class, but it's a question worth > asking > > > > > > > legal@apache > > > > > > > > > > > > just to see what folks think. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > 2.) Generating Annogen beans for meta data > > > > > > > > > > > > > The annogen task that generates code based on > the > > > > > > > annotated > > > > > > > > > > > > classes > > > > > > > > > > > > > seems to have a bug with inner classes. > > > > > > > > > > > > > For example WebParam.Mode gives compilation > errors > > > as it > > > > > > > cannot > > > > > > > > > > > > the > > > > > > > > > > > > > handle the inner class > > > > > > > > > > > > > So I eidted the generated source file to get > it > > > working. > > > > > > > (I > > > > > > > > > > > > maybe wrong > > > > > > > > > > > > > here...) > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > [eko] Not sure I follow this -- let me take a look > at it > > > and > > > > > > > I'll get > > > > > > > > > > > > back to you. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > Davanum Srinivas : http://www.wso2.net (Oxygen for Web Service > > > Developers) > > > > > > > > > > > > ------=_Part_11850_4333192.1153847539927--