David, regarding javassist:
We use javassist for 2 concerns: 1.) scanning the classpath for managed beans. This part is completely encapsulated via the MetaDataDiscoveryService SPI. You could easily use your own classpath scanning library and simply set it via openwebbeans.properties. The current default is: #use the web metadata as default org.apache.webbeans.spi.deployer.MetaDataDiscoveryService=org.apache.webbeans.spi.ee.deployer.WarMetaDataDiscoveryImpl 2.) we also use javassist for creating Proxies (see our JavassistProxyFactory.java). If there is a need, then we could easily move this to a SPI too. I don't think we have other places where we do use javassist, do we Gurkan? LieGrue, strub --- On Thu, 1/14/10, Matthias Wessendorf <[email protected]> wrote: > From: Matthias Wessendorf <[email protected]> > Subject: Re: Comments on dependencies > To: [email protected] > Date: Thursday, January 14, 2010, 9:49 AM > On Wed, Jan 13, 2010 at 9:37 PM, > Gurkan Erdogdu <[email protected]> > wrote: > > Hi David; > > > > Thanks for comments. > > > >>>>M2_REPO/xml-apis/xml-apis/1.0.b2/xml-apis-1.0.b2.jar > -- not needed for compilation > > webbeans-impl's pom.xml does not contain this. Where > is the location of this dependency, which pom? > > inherited from here > http://repo2.maven.org/maven2/dom4j/dom4j/1.6.1/dom4j-1.6.1.pom > > > > >>>>Also, > M2_REPO/javassist/javassist/3.8.0.GA/javassist-3.8.0.GA.jar > -- > > used in one class that appears to be a test helper. > If it is not used > > elsewhere, you might consider removing it. > > Look at > org.apache.webbeans.proxy.JavassistProxyFactory class in > webbeans-impl module, it uses javassist.jar that is defined > in "trunk/pom.xml". This jar is required. > > > > I think he asked for a replacement :-) David, feel free to > contribute > something like that ;-) > > >>>>For webbeans-resource: > > webbeans-resource depends on webbeans-impl module, so > it transitively get your listed modules except xml-apis. I > do not see xml-api jar. > > again inherited from here: > http://repo2.maven.org/maven2/dom4j/dom4j/1.6.1/dom4j-1.6.1.pom > (as of the dependency, mentioned by Gurkan) > > > > > Thanks; > > > > --Gurkan > > > > > > ________________________________ > > From: David Ezzio <[email protected]> > > To: [email protected] > > Sent: Wed, January 13, 2010 5:47:49 PM > > Subject: Comments on dependencies > > > > All, > > > > At rev 898415, I set up Eclipse projects using the > eclipse plugin. > > > > Systematically removing M2_REPO jars from the > project's dependency list, I discovered the following: > > > > For webbeans-impl: > > > > M2_REPO/xml-apis/xml-apis/1.0.b2/xml-apis-1.0.b2.jar > -- not needed for compilation > > > > Also, > M2_REPO/javassist/javassist/3.8.0.GA/javassist-3.8.0.GA.jar > -- used in one class that appears to be a test helper. If > it is not used elsewhere, you might consider removing it. > > > > For webbeans-resource: > > > > > M2_REPO/javassist/javassist/3.8.0.GA/javassist-3.8.0.GA.jar > -- not needed for compilation > > > > M2_REPO/dom4j/dom4j/1.6.1/dom4j-1.6.1.jar -- not > needed for compilation > > > > > M2_REPO/net/sf/scannotation/scannotation/1.0.2/scannotation-1.0.2.jar > -- not needed for compilation > > > > M2_REPO/xml-apis/xml-apis/1.0.b2/xml-apis-1.0.b2.jar > -- not needed for compilation > > > > Any corrections welcomed. > > > > David > > > > > > > > > ___________________________________________________________________ > > Yahoo! Türkiye açıldı! http://yahoo.com.tr > > İnternet üzerindeki en iyi içeriği Yahoo! Türkiye > sizlere sunuyor! > > > > -- > Matthias Wessendorf > > blog: http://matthiaswessendorf.wordpress.com/ > sessions: http://www.slideshare.net/mwessendorf > twitter: http://twitter.com/mwessendorf >
