Mailing-List: contact dev-help@commons.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Commons Developers List" <dev@commons.apache.org>
Received-SPF: error (nike.apache.org: local policy)
DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=s2048; d=yahoo.de;
	b=arY+RbyIYTHa/k9FYg/uQ6jpAL1b/KadfQ2DbuiQlKA0qmj9sSGNhluisEdf0aSBNNAXAlUMUGm6N3m2axGx0KYuRJEf0Y/vVqegs9+egbHvCiVEOoXD9fKjhQGqDPJo+CsRw+WkZw3syunR9lyFQjiWoN3W0UA+EFCzDpJqZb2Bd8Pki/tEmRsxrwIeXqS57WA7qXWnuyV2XcH2oS0vO99EMaskulpaT524dUC42Dhs3E0vBjDtiNcYKuy9CfFKWf8uBNi9iJwBj5o4CmOGp6pLlNbAySE0Gbi7r60QwT3B3y9ktRmfvEqeQ7PLOabYhvaNfKrOiFpgJ2efwPepQA==;
Date: Sun, 4 Jan 2015 16:57:40 +0000 (UTC)
From: Mark Struberg <struberg@yahoo.de>
Reply-To: Mark Struberg <struberg@yahoo.de>
To: Commons Developers List <dev@commons.apache.org>
Message-ID: 
 <128562648.4263551.1420390660721.JavaMail.yahoo@jws11172.mail.ir2.yahoo.com>
In-Reply-To: 
 <CAF8HOZLBSjG0=u5OqSGv6ozsBkSTuTOoe=mqWFKJtiOya4GdCA@mail.gmail.com>
References: 
 <CAF8HOZLBSjG0=u5OqSGv6ozsBkSTuTOoe=mqWFKJtiOya4GdCA@mail.gmail.com>
Subject: Re: Announce: Commons Inject
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

Hi Jochen!


The code is now indeed self-contained. I did not really look at the code bu=
t like to start with just a few small observations:

1.) the repo contains the whole eclipse project files. I'd rather remove th=
ose from the repos and add them (+ idea files) to svn:ignore.

2.) javax-inject.api. I'd probably replace this via our own impl:
http://repo1.maven.org/maven2/org/apache/geronimo/specs/geronimo-atinject_1=
.0_spec/1.0/

3.) the atinject (JSR-330) TCK. I compiled the project and it only said it =
passes 2 tests. I double checked with OpenWebBeans and over there we run (a=
nd pass of course) 50 tests. Maybe there is something wrong with the integr=
ation?

4.) the Scopes.
You currently have a Enum for this. I guess it would be pretty easy to swit=
ch this to using scope annotations which are meta-annotated with @Scope ins=
tead? And also implement the @Singleton scope based on that approach instea=
d of rolling your own?


Just a few ideas.=20


LieGrue,
strub


> On Wednesday, 17 December 2014, 13:56, Jochen Wiedmann <jochen.wiedmann@g=
mail.com> wrote:
> > Well spotted. I had added Guice as a Maven dependency so as to
> validate certain things while implementing. It's now removed. This
> should eliminate your concerns. Also, please note that the remaining
> dependencies are all provided, with the exception of
> javax.inject-1.jar and javax.inject-tck-1.jar, which are required for
> obvious reasons. (After all, this is the implemented standard.)
>=20
>=20
>=20
> On Wed, Dec 17, 2014 at 11:52 AM, Benedikt Ritter <britter@apache.org>=20
> wrote:
>>  2014-11-19 8:44 GMT+01:00 Mark Struberg <struberg@yahoo.de>:
>>>=20
>>>  Jochen, I might have done something wrong so please help me.
>>>=20
>>>  I've checked out your svn link and built it.
>>>=20
>>>  Then I did a
>>>=20
>>>  $> mvn clean -DincludeScope=3Druntime dependency:copy-dependencies
>>>  -rw-r--r--+ 1 struberg staff    4467 19. Nov 08:41 aopalliance-1.0.jar
>>>  -rw-r--r--+ 1 struberg staff 2228009 19. Nov 08:41 guava-16.0.1.jar
>>>  -rw-r--r--+ 1 struberg staff  642741 19. Nov 08:41 guice-4.0-beta5.jar
>>>  -rw-r--r--+ 1 struberg staff    2497 19. Nov 08:41 javax.inject-1.jar
>>>=20
>>>=20
>>>  $> du -hs target/dependencies
>>>=20
>>>  show me 2.8 MB
>>>=20
>>>=20
>>>  Plus your own jar which is 76k.
>>>  Is there something wrong? What are you using guice and guava for?
>>>  Also there is an own ASF package for atinject [1].
>>>=20
>>=20
>>  Jochen, can you please comment?
>>=20
>>  Benedikt
>>=20
>>=20
>>>=20
>>>  LieGrue,
>>>  strub
>>>=20
>>>=20
>>>=20
>>>  [1]
>>>=20
> http://repo1.maven.org/maven2/org/apache/geronimo/specs/geronimo-atinject=
_1.0_spec/1.0/
>>>=20
>>>=20
>>>=20
>>>=20
>>>  On Wednesday, 19 November 2014, 8:34, Mark Struberg=20
> <struberg@yahoo.de>
>>>  wrote:
>>>  >> Sorry, did not mean to step on somebody's toes.
>>>  >
>>>  >No worries you didn't. It's most probably our fault as our=20
> (OpenWebBeans)
>>>  documentation sucks and we did not properly document all this stuff ;)
>>>  >
>>>  >If one of you guys is at ApacheCon in Budapest right now, then=20
> I'd love
>>>  to give you a quick rush through our code an see if there is something=
=20
> you
>>>  could use and also if we could improve something in OWB.
>>>  >
>>>  >LieGrue,
>>>  >strub
>>>  >
>>>  >
>>>  >
>>>  >
>>>  >----- Original Message -----
>>>  >> From: Oliver Heger <oliver.heger@oliver-heger.de>
>>>  >> To: Commons Developers List <dev@commons.apache.org>
>>>  >> Cc:
>>>  >> Sent: Sunday, 9 November 2014, 11:45
>>>  >> Subject: Re: Announce: Commons Inject
>>>  >>
>>>  >>
>>>  >>
>>>  >> On 08.11.2014 21:51, Romain Manni-Bucau wrote:
>>>  >>>  Le 8 nov. 2014 19:51, "Oliver Heger"
>>>  >> <oliver.heger@oliver-heger.de> a =C3=A9crit
>>>  >>>  :
>>>  >>>>
>>>  >>>>  Hi Jochen,
>>>  >>>>
>>>  >>>>  do you intend to position this framework for specific=20
> use cases? In
>>>  >>>>  which way is it different or special from other=20
> implementations?
>>>  >>>>
>>>  >>>>  Just as one example: In the company I am working for,=20
> we are using
>>>  CDI
>>>  >>>>  in a pretty large JSE application. Due to the huge=20
> class path the
>>>  setup
>>>  >>>>  of the CDI container takes a long time and consumes a=20
> lot of memory
>>>  >>>>  (tested with both Weld SE and OpenWebBeans). So a=20
> fast and
>>>  lightweight
>>>  >>>>  implementation for this special purpose would be=20
> interesting.
>>>  >>>>
>>>  >>>
>>>  >>>  Surely a bad example since you can achieve it with owb,=20
> just
>>>  configure the
>>>  >>>  scanner service
>>>  >>
>>>  >> Sorry, did not mean to step on somebody's toes.
>>>  >>
>>>  >> Oliver
>>>  >>
>>>  >>>
>>>  >>>>  Oliver
>>>  >>>>
>>>  >>>>  Am 04.11.2014 um 15:55 schrieb Jochen Wiedmann:
>>>  >>>>>  Hi,
>>>  >>>>>
>>>  >>>>>  As some of you (hopefulyl not all) may have=20
> noticed, have added a
>>>  >>>>>  project called Commons Inject to the Sandbox [1]=20
> today. Commons
>>>  >> Inject
>>>  >>>>>  is a JSR 330 compliant dependency injection=20
> framework. It is
>>>  >> something
>>>  >>>>>  I had in the works for quite some time, but now=20
> it has reached a
>>>  >>>>>  decent state with my preliminary milestones=20
> reached:
>>>  >>>>>
>>>  >>>>>  - Passes the JSR 330 TCK.
>>>  >>>>>  - Integrated lifecycle handling via=20
> @PostConstruct and @PreDestroy.
>>>  >>>>>  Works with all scopes,
>>>  >>>>>     including lazy singletons. (I never got this=20
> to work with Guice,
>>>  >>>>>  which has been my major
>>>  >>>>>     driver for doing this.)
>>>  >>>>>  - Integrated logger injection framework for=20
> Log4J, SLF4J, and
>>>  >> Commons
>>>  >>>>>  Logging. Others can
>>>  >>>>>     easily be added.
>>>  >>>>>
>>>  >>>>>  I intend to use this for serious works from now=20
> on and consequently
>>>  >>>>>  hope to create a release real soon, at which=20
> point I'll ask to
>>>  >> move
>>>  >>>>>  this to proper.
>>>  >>>>>
>>>  >>>>>  Any feedback welcome.
>>>  >>>>>
>>>  >>>>>  Jochen
>>>  >>>>>
>>>  >>>>>  [1]
>>>  >>=20
> https://svn.apache.org/viewvc/commons/sandbox/commons-inject/trunk/
>>>  >>>>>
>>>  >>>>>
>>>  >>>>
>>>  >>>>
>>>  >>>> =20
> ---------------------------------------------------------------------
>>>  >>>>  To unsubscribe, e-mail:=20
> dev-unsubscribe@commons.apache.org
>>>  >>>>  For additional commands, e-mail:=20
> dev-help@commons.apache.org
>>>  >
>>>  >>
>>>  >>>>
>>>  >>>
>>>  >>
>>>  >>=20
> ---------------------------------------------------------------------
>>>  >> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
>>>  >> For additional commands, e-mail: dev-help@commons.apache.org
>>>  >>
>>>  >
>>>=20
>> ---------------------------------------------------------------------
>>>  >To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
>>>  >For additional commands, e-mail: dev-help@commons.apache.org
>>>  >
>>>  >
>>>  >
>>>=20
>>>  ---------------------------------------------------------------------
>>>  To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
>>>  For additional commands, e-mail: dev-help@commons.apache.org
>>>=20
>>>=20
>>=20
>>  --
>>  http://people.apache.org/~britter/
>>  http://www.systemoutprintln.de/
>>  http://twitter.com/BenediktRitter
>>  http://github.com/britter
>=20
>=20
>=20
> --=20
> Our time is just a point along a line that runs forever with no end.
> (Al Stewart, Lord Grenville)
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org