beehive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Rajith Attapattu" <rajit...@gmail.com>
Subject Re: JSR 181 support for axis2
Date Tue, 25 Jul 2006 17:12:19 GMT
Eddiie,

Take it easy :-)

Rajith

On 7/25/06, Eddie O'Neil <ekoneil@gmail.com> 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 <rajith77@gmail.com> 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 <ekoneil@gmail.com> 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 <davanum@gmail.com> wrote:
> > > > does beehive have a nightly build? are the snapshots published to
> any
> > > > maven repo?
> > > >
> > > > thx,
> > > > dims
> > > >
> > > > On 7/25/06, Rajith Attapattu <rajith77@gmail.com> 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 <rajith77@gmail.com> 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 <ekoneil@gmail.com > 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 <ekoneil@gmail.com> 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 <rajith77@gmail.com>
wrote:
> > > > > > > > > Eddie,
> > > > > > > > >
> > > > > > > > > any update on the issue?
> > > > > > > > >
> > > > > > > > > Regards,
> > > > > > > > >
> > > > > > > > > Rajith
> > > > > > > > >
> > > > > > > > > On 7/20/06, Rajith Attapattu <rajith77@gmail.com>
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 <ekoneil@gmail.com>
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)
> > > >
> > >
> >
> >
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message