Return-Path: Delivered-To: apmail-beehive-dev-archive@www.apache.org Received: (qmail 15410 invoked from network); 30 Jul 2006 00:33:45 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (209.237.227.199) by minotaur.apache.org with SMTP; 30 Jul 2006 00:33:45 -0000 Received: (qmail 85764 invoked by uid 500); 30 Jul 2006 00:33:44 -0000 Delivered-To: apmail-beehive-dev-archive@beehive.apache.org Received: (qmail 85733 invoked by uid 500); 30 Jul 2006 00:33:44 -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 85722 invoked by uid 99); 30 Jul 2006 00:33:44 -0000 Received: from asf.osuosl.org (HELO asf.osuosl.org) (140.211.166.49) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 29 Jul 2006 17:33:44 -0700 X-ASF-Spam-Status: No, hits=0.8 required=10.0 tests=DNS_FROM_RFC_ABUSE,MAILTO_TO_SPAM_ADDR,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (asf.osuosl.org: domain of ekoneil@gmail.com designates 64.233.182.188 as permitted sender) Received: from [64.233.182.188] (HELO nf-out-0910.google.com) (64.233.182.188) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 29 Jul 2006 17:33:43 -0700 Received: by nf-out-0910.google.com with SMTP id y38so151735nfb for ; Sat, 29 Jul 2006 17:33:19 -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:content-transfer-encoding:content-disposition:references; b=kWfF91vMBxsYpZdu7Ml5oUiYvT/Y2jV3yjeo6cmll/3+MEV+VaCUobCVjEYp5vi929BPRHqDWMZbziYA8qDJUy7wzbBTkNbVT7uDjuhAGS01opqdBBsHWNisqO2/ZDbgGjTBNupsBDOA4TRDIZ9sE2qCB5RMvfgr2Ox3m2f44HY= Received: by 10.49.90.4 with SMTP id s4mr778925nfl; Sat, 29 Jul 2006 17:33:19 -0700 (PDT) Received: by 10.48.254.18 with HTTP; Sat, 29 Jul 2006 17:33:19 -0700 (PDT) Message-ID: Date: Sat, 29 Jul 2006 18:33:19 -0600 From: "Eddie O'Neil" To: "Beehive Developers" , dims@apache.org Subject: Re: JSR 181 support for axis2 In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <19e0530f0607291648o87c8b3cuafffd62f84d08ef7@mail.gmail.com> X-Virus-Checked: Checked by ClamAV on apache.org X-Spam-Rating: minotaur.apache.org 1.6.2 0/1000/N Dims-- One more tip (needed to make sure it worked), I'm able to do this: cd /foobar/wsm svn co https://svn.apache.org/repos/asf/beehive/wsm/trunk . ant clean build install test to checkout and build a clean source tree, so if using a local M2 repo is easier than dealing with the one online, that's fine for now until nightlies builds work. Eddie On 7/29/06, Eddie O'Neil wrote: > Heh...it's neither hourly nor nightly. Today, Beehive doesn't have > builds integrated into ASF infrastructure (sadly...). > > If you need to make any changes, I can just move the build to: > > http://people.apache.org/dist/snapshots/maven2/ > > and send you the script I used to install the JARs via Maven2 and scp. > > For any local changes, though, the WSM build will actually install > built JARs directly into your local Maven2 repository if that's > easier. > > Eddie > > > > On 7/29/06, Davanum Srinivas wrote: > > Eddie, > > > > Great!! Is this a hourly build or a nightly build? :) Just In case i > > need to do any changes to beehive i can plan accordingly. > > > > thanks, > > dims > > > > On 7/29/06, Eddie O'Neil wrote: > > > Dims / Rajith-- > > > > > > I've uploaded a WSM snapshot to a Maven2 repository here: > > > > > > http://people.apache.org/~ekoneil/repository/maven2/ > > > > > > it contains these artifacts: > > > > > > beehive-wsm > > > beehive-jsr181 > > > beehive-wsdltypes > > > beehive-wsm-axis > > > > > > The first three would be needed to support the 181 work on Axis2; the > > > fourth JAR is only needed for Axis1.x. > > > > > > These should help the work to get Rajith's code integrated to the > > > Axis2 build. > > > > > > I'm starting to work on two more things on the WSM side: > > > > > > 1) an annotation processor extension for code generating a > > > services.xml file given an annotated Java source file > > > 2) a Maven2 archetype for building Axis2 services with WSM that would > > > use #1 and produce a .jar file that can be renamed to .aar and > > > deployed > > > > > > Let me know how the repository works out. > > > > > > Eddie > > > > > > ::::: pom.xml snip for WSM support > > > > > > org.apache.beehive > > > beehive-jsr181 > > > 1.0-SNAPSNOT > > > > > > > > > org.apache.beehive > > > beehive-wsm > > > 1.0-SNAPSNOT > > > > > > > > > org.apache.beehive > > > beehive-wsdltypes > > > 1.0-SNAPSNOT > > > > > > ::::: > > > > > > ::::: Add this to your M2 settings.xml > > > > > > wsm-snapshot > > > wsm-snapshot > > > > > > http://people.apache.org/~ekoneil/repository/maven2 > > > > > > ::::: > > > > > > > > > On 7/27/06, Rajith Attapattu wrote: > > > > Thanks Eddie. > > > > > > > > On 7/27/06, Eddie O'Neil wrote: > > > > > > > > > > FYI -- will have JARs in a Maven2 repo later today. They won't be > > > > > mirrored, but there will be public access to them. > > > > > > > > > > Eddie > > > > > > > > > > > > > > > > > > > > On 7/27/06, Rajith Attapattu wrote: > > > > > > Eddie, > > > > > > > > > > > > Awesome, thanks a lot. > > > > > > Yes, please reply to the thread on axis2 dev about my proposed ASM based > > > > > > class scanner for the drop in support. > > > > > > If you can or Dims can have a the jars in your maven repos, then Dims > > > > > can > > > > > > work on the patch and see how to proceed further. > > > > > > > > > > > > Thanks guys for the support. > > > > > > > > > > > > Regards, > > > > > > > > > > > > Rajith > > > > > > > > > > > > On 7/27/06, Eddie O'Neil wrote: > > > > > > > > > > > > > > Rajith / Dims -- > > > > > > > > > > > > > > I committed Rajith's patch that adds the reflection support for > > > > > > > creating a WsmService using annogen. The one change to the patch was > > > > > > > that I moved the generated annogen beans from "javax.jws" to > > > > > > > "o.a.b.w.axis.annogen" and removed the annogen annotations from the > > > > > > > "javax.jws" classes as per the legal-discuss thread. > > > > > > > > > > > > > > This should free Dims up to start the work of applying the Axis > > > > > patch. > > > > > > > > > > > > > > Next steps are to upload this to my Maven repo and to talk about > > > > > > > whether / how to support drop-in web service wiring in axis-dev@. > > > > > > > > > > > > > > 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) > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > Davanum Srinivas : http://www.wso2.net (Oxygen for Web Service Developers) > > >