Return-Path: Delivered-To: apmail-beehive-dev-archive@www.apache.org Received: (qmail 12897 invoked from network); 30 Jul 2006 00:03:42 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (209.237.227.199) by minotaur.apache.org with SMTP; 30 Jul 2006 00:03:42 -0000 Received: (qmail 77067 invoked by uid 500); 30 Jul 2006 00:03:42 -0000 Delivered-To: apmail-beehive-dev-archive@beehive.apache.org Received: (qmail 77043 invoked by uid 500); 30 Jul 2006 00:03:42 -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 77032 invoked by uid 99); 30 Jul 2006 00:03:42 -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:03:42 -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.186 as permitted sender) Received: from [64.233.182.186] (HELO nf-out-0910.google.com) (64.233.182.186) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 29 Jul 2006 17:03:41 -0700 Received: by nf-out-0910.google.com with SMTP id y38so148977nfb for ; Sat, 29 Jul 2006 17:03:17 -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=QBctD1UBrkQ4CwO7DZmGXX2qVxCqIvmUx3X2nwaAYS9TT6qJAKgTNE7PTwLYV5bfq4D8zDkTmDX1FZCSQfM66CasK4WjWESfp+mAkXHBfOKh1X+da3ySt+7LT0gG0eiNqpfZr01kUcDP8Sm8DDVOYrYMoBTgCDx6QaX/sqjSYc0= Received: by 10.49.20.15 with SMTP id x15mr771457nfi; Sat, 29 Jul 2006 17:03:17 -0700 (PDT) Received: by 10.48.254.18 with HTTP; Sat, 29 Jul 2006 17:03:17 -0700 (PDT) Message-ID: Date: Sat, 29 Jul 2006 18:03:17 -0600 From: "Eddie O'Neil" To: "Beehive Developers" , dims@apache.org Subject: Re: JSR 181 support for axis2 In-Reply-To: <19e0530f0607291648o87c8b3cuafffd62f84d08ef7@mail.gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <19e0530f0607250615m330067e7m60596098e0a67e04@mail.gmail.com> <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 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) >