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 Wed, 02 Aug 2006 16:24:55 GMT
Cool, Looking forward to your comments on axis-dev.

Regards,

Rajith

On 8/2/06, Eddie O'Neil <ekoneil@gmail.com> wrote:
>
>   Sure -- just saw the thread.  He asks some good questions.
>
>   Will comment in axis-dev@
>
> Eddie
>
>
> On 8/2/06, Rajith Attapattu <rajith77@gmail.com> wrote:
> > Eddie,
> >
> > There are some comments by Nicholas on WSM in the axis2 thread on JSR
> 181.
> > Could u take peek when ever time permits you and answer them from the
> WSM
> > POV.
> >
> > Thanks,
> >
> > Rajith
> >
> > On 7/30/06, Eddie O'Neil <ekoneil@gmail.com> wrote:
> > >
> > > Dims--
> > >
> > >   Thanks for the great feedback -- these are fixed.
> > >
> > > Eddie
> > >
> > >
> > >
> > > On 7/30/06, Davanum Srinivas <davanum@gmail.com> wrote:
> > > > Hi Eddie,
> > > >
> > > > Needed a couple of things to get "ant clean build install test"
> working.
> > > > http://issues.apache.org/jira/browse/BEEHIVE-1136
> > > > http://issues.apache.org/jira/browse/BEEHIVE-1137
> > > >
> > > > thanks,
> > > > dims
> > > >
> > > > On 7/29/06, Eddie O'Neil <ekoneil@gmail.com> wrote:
> > > > > 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 <ekoneil@gmail.com> 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 <davanum@gmail.com> 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 <ekoneil@gmail.com> 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
> > > > > > > >         <dependency>
> > > > > > > >             <groupId>org.apache.beehive</groupId>
> > > > > > > >             <artifactId>beehive-jsr181</artifactId>
> > > > > > > >             <version>1.0-SNAPSNOT</version>
> > > > > > > >         </dependency>
> > > > > > > >         <dependency>
> > > > > > > >             <groupId>org.apache.beehive</groupId>
> > > > > > > >             <artifactId>beehive-wsm</artifactId>
> > > > > > > >             <version>1.0-SNAPSNOT</version>
> > > > > > > >         </dependency>
> > > > > > > >         <dependency>
> > > > > > > >             <groupId>org.apache.beehive</groupId>
> > > > > > > >             <artifactId>beehive-wsdltypes</artifactId>
> > > > > > > >             <version>1.0-SNAPSNOT</version>
> > > > > > > >         </dependency>
> > > > > > > > :::::
> > > > > > > >
> > > > > > > > ::::: Add this to your M2 settings.xml
> > > > > > > >                 <repository>
> > > > > > > >                     <id>wsm-snapshot</id>
> > > > > > > >                     <name>wsm-snapshot</name>
> > > > > > > >
> > > > > > > > <url>http://people.apache.org/~ekoneil/repository/maven2
> </url>
> > > > > > > >                 </repository>
> > > > > > > > :::::
> > > > > > > >
> > > > > > > >
> > > > > > > > On 7/27/06, Rajith Attapattu <rajith77@gmail.com>
wrote:
> > > > > > > > > Thanks Eddie.
> > > > > > > > >
> > > > > > > > > On 7/27/06, Eddie O'Neil <ekoneil@gmail.com>
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 <rajith77@gmail.com>
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 <ekoneil@gmail.com>
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 <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.Modegives
> > > 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.Modegives
> > > > > > > > > > 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)
> > > > > > >
> > > > > >
> > > > >
> > > >
> > > >
> > > > --
> > > > Davanum Srinivas : http://www.wso2.net (Oxygen for Web Service
> > > Developers)
> > > >
> > >
> >
> >
>

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