Eddiie, Take it easy :-)
Rajith On 7/25/06, Eddie O'Neil <[EMAIL PROTECTED]> 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 <[EMAIL PROTECTED]> 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 <[EMAIL PROTECTED]> 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 <[EMAIL PROTECTED]> wrote: > > > does beehive have a nightly build? are the snapshots published to any > > > maven repo? > > > > > > thx, > > > dims > > > > > > On 7/25/06, Rajith Attapattu <[EMAIL PROTECTED]> 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 <[EMAIL PROTECTED]> 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 <[EMAIL PROTECTED] > wrote: > > > > > > > > > > > > A'right -- mail is in [EMAIL PROTECTED] 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 <[EMAIL PROTECTED]> 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 <[EMAIL PROTECTED]> wrote: > > > > > > > > Eddie, > > > > > > > > > > > > > > > > any update on the issue? > > > > > > > > > > > > > > > > Regards, > > > > > > > > > > > > > > > > Rajith > > > > > > > > > > > > > > > > On 7/20/06, Rajith Attapattu <[EMAIL PROTECTED]> 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 < [EMAIL PROTECTED]> 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 <[EMAIL PROTECTED]> 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 > > > > > > [EMAIL PROTECTED] > > > > > > > > > > > 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) > > > > > > >
