[jboss-cvs] JBossAS SVN: r68895 - in trunk: aspects/src/main/org/jboss/aop/asintegration and 3 other directories.

jboss-cvs-commits at lists.jboss.org jboss-cvs-commits at lists.jboss.org
Fri Jan 11 10:08:47 EST 2008


Author: kabir.khan at jboss.com
Date: 2008-01-11 10:08:47 -0500 (Fri, 11 Jan 2008)
New Revision: 68895

Added:
   trunk/aspects/src/main/org/jboss/aop/asintegration/embedded/
   trunk/aspects/src/main/org/jboss/aop/asintegration/embedded/JBossEmbeddedUCLIntegration.java
Modified:
   trunk/aspects/build.xml
   trunk/embedded/build.xml
   trunk/embedded/src/main/resources/javase/deployers/aspect-deployer-beans.xml
Log:
[EMB-5] Accept  launcher classloader as a valid classloader in jboss embedded

Modified: trunk/aspects/build.xml
===================================================================
--- trunk/aspects/build.xml	2008-01-11 14:42:17 UTC (rev 68894)
+++ trunk/aspects/build.xml	2008-01-11 15:08:47 UTC (rev 68895)
@@ -242,6 +242,14 @@
          </fileset>
       </jar>
 
+      <!-- Build jboss-aop-jboss5.jar -->
+      <jar destfile="${build.lib}/jboss-aop-embedded.jar" manifest="${build.etc}/default.mf"
+         update="${jar.update}" index="${jar.index}">
+         <fileset dir="${build.classes}">
+            <include name="org/jboss/aop/asintegration/embedded/**"/>
+         </fileset>
+      </jar>
+
       <!-- Build client jar containing the minimum needed for ejb 3 -->
       <jar destfile="${build.lib}/jboss-aspect-jdk50-client.jar" manifest="${build.etc}/default.mf"
          update="${jar.update}" index="${jar.index}">

Added: trunk/aspects/src/main/org/jboss/aop/asintegration/embedded/JBossEmbeddedUCLIntegration.java
===================================================================
--- trunk/aspects/src/main/org/jboss/aop/asintegration/embedded/JBossEmbeddedUCLIntegration.java	                        (rev 0)
+++ trunk/aspects/src/main/org/jboss/aop/asintegration/embedded/JBossEmbeddedUCLIntegration.java	2008-01-11 15:08:47 UTC (rev 68895)
@@ -0,0 +1,40 @@
+/*
+* JBoss, Home of Professional Open Source.
+* Copyright 2006, Red Hat Middleware LLC, and individual contributors
+* as indicated by the @author tags. See the copyright.txt file in the
+* distribution for a full listing of individual contributors. 
+*
+* This is free software; you can redistribute it and/or modify it
+* under the terms of the GNU Lesser General Public License as
+* published by the Free Software Foundation; either version 2.1 of
+* the License, or (at your option) any later version.
+*
+* This software is distributed in the hope that it will be useful,
+* but WITHOUT ANY WARRANTY; without even the implied warranty of
+* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+* Lesser General Public License for more details.
+*
+* You should have received a copy of the GNU Lesser General Public
+* License along with this software; if not, write to the Free
+* Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
+* 02110-1301 USA, or see the FSF site: http://www.fsf.org.
+*/ 
+package org.jboss.aop.asintegration.embedded;
+
+import org.jboss.aop.asintegration.jboss4.JBoss4Integration;
+import org.jboss.mx.loading.RepositoryClassLoader;
+
+/**
+ * 
+ * @author <a href="kabir.khan at jboss.com">Kabir Khan</a>
+ * @version $Revision: 1.1 $
+ */
+public class JBossEmbeddedUCLIntegration extends JBoss4Integration
+{
+   public boolean isValidClassLoader(ClassLoader loader)
+   {
+      //Accept all loaders as valid, and reject repository classloaders with null repository
+      if (!(loader instanceof RepositoryClassLoader)) return true;
+      return ((RepositoryClassLoader) loader).getLoaderRepository() != null;
+   }
+}

Modified: trunk/embedded/build.xml
===================================================================
--- trunk/embedded/build.xml	2008-01-11 14:42:17 UTC (rev 68894)
+++ trunk/embedded/build.xml	2008-01-11 15:08:47 UTC (rev 68895)
@@ -337,16 +337,16 @@
             <include name="org/jboss/embedded/tomcat/WebinfScanner.class"/>
             <include name="org/jboss/embedded/tomcat/security/*.class"/>
          </fileset>
-      </jar>
-
-
-   	  <jar jarfile="${build.lib}/embedded-jboss/lib/jboss-embedded.jar">
-   	     <fileset dir="${build.classes}">
-   	        <include name="org/jboss/embedded/**"/>
-   	        <exclude name="org/jboss/embedded/tomcat/EmbeddedJBossBootstrapListener.class"/>
-   	        <exclude name="org/jboss/embedded/tomcat/WebinfScanner.class"/>
-   	        <exclude name="org/jboss/embedded/tomcat/security/*.class"/>
-   	     </fileset>
+      </jar>
+
+
+   	  <jar jarfile="${build.lib}/embedded-jboss/lib/jboss-embedded.jar">
+   	     <fileset dir="${build.classes}">
+   	        <include name="org/jboss/embedded/**"/>
+   	        <exclude name="org/jboss/embedded/tomcat/EmbeddedJBossBootstrapListener.class"/>
+   	        <exclude name="org/jboss/embedded/tomcat/WebinfScanner.class"/>
+   	        <exclude name="org/jboss/embedded/tomcat/security/*.class"/>
+   	     </fileset>
    	  </jar>
 
       <jar jarfile="${build.lib}/embedded-jboss/lib/jboss-embedded-all.jar">
@@ -369,13 +369,14 @@
          <zipfileset src="${jboss.ejb3.lib}/jboss-annotations-ejb3.jar"/>
          <zipfileset src="${jboss.jboss.javaee.lib}/jboss-javaee.jar"/>
          <zipfileset src="${jboss.integration.lib}/jboss-integration.jar"/>
-         <zipfileset src="${jboss.aop.lib}/jboss-aop-jdk50.jar"/>
-      	 <zipfileset src="${jboss.aop.lib}/jboss-aop-jboss4-jdk50.jar"/>
+         <zipfileset src="${jboss.aop.lib}/jboss-aop-jdk50.jar"/>
+      	<zipfileset src="${jboss.aop.lib}/jboss-aop-jboss4-jdk50.jar"/>
          <zipfileset src="${jboss.aop.lib}/jboss-aop-deployer-jdk50.jar"/>
          <zipfileset src="${jboss.varia.lib}/hsqldb-plugin.jar"/>
          <zipfileset src="${jboss.mq.lib}/jbossmq.jar"/>
          <zipfileset src="${jboss.security.int.lib}/jbosssx-server.jar"/>
          <zipfileset src="${jboss.aspects.lib}/jboss-aspect-library-jdk50.jar"/>
+         <zipfileset src="${jboss.aspects.lib}/jboss-aop-embedded.jar"/>
          <zipfileset src="${jboss.integration.lib}/jboss-transaction-spi.jar"/>
          <zipfileset src="${jboss.integration.lib}/jboss-classloading-spi.jar"/>
          <zipfileset src="${jboss.naming.lib}/jnpserver.jar"/>
@@ -388,15 +389,15 @@
          <zipfileset src="${jboss.microcontainer.lib}/jboss-kernel.jar"/>
          <zipfileset src="${jboss.microcontainer.lib}/jboss-container.jar"/>
          <zipfileset src="${jboss.microcontainer.lib}/jboss-classloader.jar"/>
-         <zipfileset src="${jboss.microcontainer.lib}/jboss-dependency.jar"/>
-         <zipfileset src="${jboss.microcontainer.lib}/jboss-deployers-client.jar"/>
-         <zipfileset src="${jboss.microcontainer.lib}/jboss-deployers-client-spi.jar"/>
-         <zipfileset src="${jboss.microcontainer.lib}/jboss-deployers-core.jar"/>
-         <zipfileset src="${jboss.microcontainer.lib}/jboss-deployers-core-spi.jar"/>
-         <zipfileset src="${jboss.microcontainer.lib}/jboss-deployers-impl.jar"/>
-         <zipfileset src="${jboss.microcontainer.lib}/jboss-deployers-spi.jar"/>
-         <zipfileset src="${jboss.microcontainer.lib}/jboss-deployers-structure-spi.jar"/>
-         <zipfileset src="${jboss.microcontainer.lib}/jboss-deployers-vfs.jar"/>
+         <zipfileset src="${jboss.microcontainer.lib}/jboss-dependency.jar"/>
+         <zipfileset src="${jboss.microcontainer.lib}/jboss-deployers-client.jar"/>
+         <zipfileset src="${jboss.microcontainer.lib}/jboss-deployers-client-spi.jar"/>
+         <zipfileset src="${jboss.microcontainer.lib}/jboss-deployers-core.jar"/>
+         <zipfileset src="${jboss.microcontainer.lib}/jboss-deployers-core-spi.jar"/>
+         <zipfileset src="${jboss.microcontainer.lib}/jboss-deployers-impl.jar"/>
+         <zipfileset src="${jboss.microcontainer.lib}/jboss-deployers-spi.jar"/>
+         <zipfileset src="${jboss.microcontainer.lib}/jboss-deployers-structure-spi.jar"/>
+         <zipfileset src="${jboss.microcontainer.lib}/jboss-deployers-vfs.jar"/>
          <zipfileset src="${jboss.microcontainer.lib}/jboss-deployers-vfs-spi.jar"/>
          <zipfileset src="${jboss.microcontainer.lib}/jboss-aop-mc-int.jar"/>
          <zipfileset src="${jboss.microcontainer.lib}/jboss-metatype.jar"/>
@@ -429,9 +430,9 @@
          <zipfileset src="${jboss.jboss.jaspi.api.lib}/jboss-jaspi-api.jar"/>
          <zipfileset src="${jboss.metadata.lib}/jboss-metadata.jar"/>
          <zipfileset src="${jboss.jboss.ejb3.cache.lib}/jboss-ejb3-cache.jar"/>
-         <zipfileset src="${jboss.jboss.ejb3.ext.api.lib}/jboss-ejb3-ext-api.jar"/>
-         <zipfileset src="${jboss.jboss.ejb3.impl.lib}/jboss-ejb3-impl.jar"/>
-         <zipfileset src="${jboss.jboss.ha.client.lib}/jboss-ha-client.jar" />
+         <zipfileset src="${jboss.jboss.ejb3.ext.api.lib}/jboss-ejb3-ext-api.jar"/>
+         <zipfileset src="${jboss.jboss.ejb3.impl.lib}/jboss-ejb3-impl.jar"/>
+         <zipfileset src="${jboss.jboss.ha.client.lib}/jboss-ha-client.jar" />
       	 <zipfileset src="${jboss.profileservice.lib}/jboss-profileservice.jar" />
          <zipfileset src="../messaging/output/lib/jboss-messaging-int.jar"/>
       </jar>
@@ -441,8 +442,8 @@
          <zipfileset src="${hibernate.lib}/hibernate3.jar"/>
          <zipfileset src="${hibernate.entitymanager.lib}/ejb3-persistence.jar"/>
          <zipfileset src="${hibernate.entitymanager.lib}/hibernate-entitymanager.jar"/>
-         <zipfileset src="${hibernate.annotations.lib}/hibernate-annotations.jar"/>
-         <zipfileset src="${hibernate.commons.annotations.lib}/hibernate-commons-annotations.jar"/>
+         <zipfileset src="${hibernate.annotations.lib}/hibernate-annotations.jar"/>
+         <zipfileset src="${hibernate.commons.annotations.lib}/hibernate-commons-annotations.jar"/>
          <zipfileset src="${hibernate.validator.lib}/hibernate-validator.jar" />
       </jar>
 
@@ -461,8 +462,8 @@
          <zipfileset src="${apache.xerces.lib}/xml-apis.jar" excludes="**/license"/>
          <zipfileset src="${javassist.lib}/javassist.jar"/>
          <zipfileset src="${trove.lib}/trove.jar"/>
-         <zipfileset src="${quartz.quartz.lib}/quartz-all.jar"/>
-      	 <zipfileset src="${sun.jaxb.lib}/jaxb-api.jar"/>
+         <zipfileset src="${quartz.quartz.lib}/quartz-all.jar"/>
+      	 <zipfileset src="${sun.jaxb.lib}/jaxb-api.jar"/>
       	 <zipfileset src="${sun.jaxb.lib}/jaxb-impl.jar"/>
       </jar>
    </target>

Modified: trunk/embedded/src/main/resources/javase/deployers/aspect-deployer-beans.xml
===================================================================
--- trunk/embedded/src/main/resources/javase/deployers/aspect-deployer-beans.xml	2008-01-11 14:42:17 UTC (rev 68894)
+++ trunk/embedded/src/main/resources/javase/deployers/aspect-deployer-beans.xml	2008-01-11 15:08:47 UTC (rev 68895)
@@ -5,14 +5,8 @@
 -->
 <deployment xmlns="urn:jboss:bean-deployer:2.0">
 
-<!--
-   <bean name="AOPClassLoaderScopingPolicy" class="org.jboss.aop.asintegration.jboss4.RepositoryClassLoaderScopingPolicy"/>
+   <bean name="AOPJBossIntegration" class="org.jboss.aop.asintegration.embedded.JBossEmbeddedUCLIntegration"/>
 
-   <bean name="AOPJBossIntegration" class="org.jboss.aop.asintegration.jboss5.JBoss5Integration">
-      <property name="scopingPolicy"><inject bean="AOPClassLoaderScopingPolicy"/></property>
-   </bean>
--->
-
    <!--
       Valid values for the AspectManager bean for use with enableTransformer=true are:
       * org.jboss.aop.deployers.AspectManagerJDK5 - works with the -javaagent:pluggable-instrumentor.jar switch
@@ -26,7 +20,7 @@
 	      </parameter>
       </constructor>
 
-      <!--<property name="jbossIntegration"><inject bean="AOPJBossIntegration"/></property>-->
+      <property name="jbossIntegration"><inject bean="AOPJBossIntegration"/></property>
 
       <property name="enableLoadtimeWeaving">false</property>
       <!-- only relevant when EnableLoadtimeWeaving is true.




More information about the jboss-cvs-commits mailing list