You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@beehive.apache.org by Rajith Attapattu <ra...@gmail.com> on 2006/08/02 15:13:14 UTC
Re: JSR 181 support for axis2
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 <ek...@gmail.com> wrote:
>
> Dims--
>
> Thanks for the great feedback -- these are fixed.
>
> Eddie
>
>
>
> On 7/30/06, Davanum Srinivas <da...@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 <ek...@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 <ek...@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 <da...@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 <ek...@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 <ra...@gmail.com> wrote:
> > > > > > > Thanks Eddie.
> > > > > > >
> > > > > > > On 7/27/06, Eddie O'Neil <ek...@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 <ra...@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 <ek...@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 <ra...@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 <ek...@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 <da...@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 <ra...@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 <ra...@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 <ek...@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.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)
> >
>
Re: JSR 181 support for axis2
Posted by Rajith Attapattu <ra...@gmail.com>.
Cool, Looking forward to your comments on axis-dev.
Regards,
Rajith
On 8/2/06, Eddie O'Neil <ek...@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 <ra...@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 <ek...@gmail.com> wrote:
> > >
> > > Dims--
> > >
> > > Thanks for the great feedback -- these are fixed.
> > >
> > > Eddie
> > >
> > >
> > >
> > > On 7/30/06, Davanum Srinivas <da...@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 <ek...@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 <ek...@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 <da...@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 <ek...@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 <ra...@gmail.com> wrote:
> > > > > > > > > Thanks Eddie.
> > > > > > > > >
> > > > > > > > > On 7/27/06, Eddie O'Neil <ek...@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 <ra...@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 <ek...@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 <ra...@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 <ek...@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 <da...@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)
> > > >
> > >
> >
> >
>
Re: JSR 181 support for axis2
Posted by Eddie O'Neil <ek...@gmail.com>.
Sure -- just saw the thread. He asks some good questions.
Will comment in axis-dev@
Eddie
On 8/2/06, Rajith Attapattu <ra...@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 <ek...@gmail.com> wrote:
> >
> > Dims--
> >
> > Thanks for the great feedback -- these are fixed.
> >
> > Eddie
> >
> >
> >
> > On 7/30/06, Davanum Srinivas <da...@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 <ek...@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 <ek...@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 <da...@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 <ek...@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 <ra...@gmail.com> wrote:
> > > > > > > > Thanks Eddie.
> > > > > > > >
> > > > > > > > On 7/27/06, Eddie O'Neil <ek...@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 <ra...@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 <ek...@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 <ra...@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 <ek...@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 <da...@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 <ra...@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 <ra...@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 <ek...@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.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)
> > >
> >
>
>