Mailing-List: contact deltaspike-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: deltaspike-dev@incubator.apache.org
Received-SPF: pass (athena.apache.org: domain of gerhard.petracek@gmail.com
 designates 209.85.213.175 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <1325971496.2209.YahooMailNeo@web27805.mail.ukl.yahoo.com>
References: 
 <CAF9TksMruA5oGj-owuLJT9Z72=BWEG4Qtm77WBYoqGBdXGvu+w@mail.gmail.com>
 <CAGJtJfHdJ9=5VxDZnpSBuWHo31i+Uy2W6eFoLqm-38jt-=w1MA@mail.gmail.com>
 <1325971496.2209.YahooMailNeo@web27805.mail.ukl.yahoo.com>
From: Gerhard Petracek <gerhard.petracek@gmail.com>
Date: Mon, 9 Jan 2012 16:17:19 +0100
Message-ID: 
 <CAGJtJfEVXoFB9v-=ALAN0_dhHqijnKKysaNmQrgeWBTRY2WDTQ@mail.gmail.com>
Subject: Re: Please review DELTASPIKE-45
To: deltaspike-dev@incubator.apache.org
Content-Type: multipart/alternative; boundary=20cf3010eb6945d80104b619e620

--20cf3010eb6945d80104b619e620
Content-Type: text/plain; charset=ISO-8859-1

since stu signed it, we can continue with it.

regards,
gerhard


2012/1/7 Mark Struberg <struberg@yahoo.de>

> Imo the impl looks quite fine. The classes are small enough and straight
> forward to keep it in the api module.
>
> One thing we would need is the iCLA from Stu. I've seen he wrote most of
> the stuff originally, and he is also on the initial committer list. But we
> didn't have an iCLA from him on file yet it seems [1].
>
> Stu, once we have your iCLA on file we can go forward importing this stuff.
>
> LieGrue,
> strub
>
>
>
> [1] http://people.apache.org/committer-index.html
>
>
> ----- Original Message -----
> > From: Gerhard Petracek <gerhard.petracek@gmail.com>
> > To: deltaspike-dev@incubator.apache.org
> > Cc:
> > Sent: Saturday, January 7, 2012 9:55 PM
> > Subject: Re: Please review DELTASPIKE-45
> >
> > hi jason,
> >
> > the only alternative which comes to my mind right now is to move the impl
> > classes to the impl module and to introduce e.g.:
> > org.apache.deltaspike.core.spi.metadata.AnnotatedTypeBuilderData and
> > AnnotatedTypeBuilder
> > just loads the impl via the low-level config mechanism (which will be
> > discussed soon).
> >
> > regards,
> > gerhard
> >
> >
> >
> > 2012/1/7 Jason Porter <lightguard.jp@gmail.com>
> >
> >>  I have the classes all checked into my branch [1]. Please review. I
> know
> >>  many of them need Javadoc, so you can forget that part. Mainly the
> >>  AnnotatedTypeBuilder needed many classes that were in Solder Impl. As I
> >>  believe AnnotatedTypeBuilder is pretty helpful for everyone doing CDI
> >>  Extension development I put them all in api, so we'll have some *Impl
> >>  classes in api. If everyone is okay with that, great. Otherwise we may
> need
> >>  to find a new place to put them as we can't put them in impl and keep
> >>  AnnotatedTypeBuilder in api.
> >>
> >>  [1]
> https://github.com/LightGuard/incubator-deltaspike/tree/DELTASPIKE-45
> >>
> >>  --
> >>  Jason Porter
> >>  http://lightguard-jp.blogspot.com
> >>  http://twitter.com/lightguardjp
> >>
> >>  Software Engineer
> >>  Open Source Advocate
> >>  Author of Seam Catch - Next Generation Java Exception Handling
> >>
> >>  PGP key id: 926CCFF5
> >>  PGP key available at: keyserver.net, pgp.mit.edu
> >>
> >
>

--20cf3010eb6945d80104b619e620--