[jboss-cvs] JBossAS SVN: r100002 - in projects/jboss-cl/branches/Branch_2_0: classloader/src/main/java/org/jboss/classloader/spi and 5 other directories.

jboss-cvs-commits at lists.jboss.org jboss-cvs-commits at lists.jboss.org
Wed Jan 27 11:42:26 EST 2010


Author: thomas.diesler at jboss.com
Date: 2010-01-27 11:42:26 -0500 (Wed, 27 Jan 2010)
New Revision: 100002

Removed:
   projects/jboss-cl/branches/Branch_2_0/classloader/src/main/java/org/jboss/classloader/spi/NativeLibraryProvider.java
Modified:
   projects/jboss-cl/branches/Branch_2_0/classloader/src/main/java/org/jboss/classloader/spi/ClassLoaderPolicy.java
   projects/jboss-cl/branches/Branch_2_0/classloader/src/main/java/org/jboss/classloader/spi/base/BaseClassLoader.java
   projects/jboss-cl/branches/Branch_2_0/classloading-vfs/pom.xml
   projects/jboss-cl/branches/Branch_2_0/classloading-vfs/src/main/java/org/jboss/classloading/spi/vfs/policy/VFSClassLoaderPolicy.java
   projects/jboss-cl/branches/Branch_2_0/classloading/pom.xml
   projects/jboss-cl/branches/Branch_2_0/classloading/src/main/java/org/jboss/classloading/spi/metadata/ClassLoadingMetaData.java
   projects/jboss-cl/branches/Branch_2_0/pom.xml
Log:
Rollback
[JBCL-136] Add a notion of native library mapping
[JBCL-137] Add support for OSGi fragments
http://community.jboss.org/message/522509#522509



Modified: projects/jboss-cl/branches/Branch_2_0/classloader/src/main/java/org/jboss/classloader/spi/ClassLoaderPolicy.java
===================================================================
--- projects/jboss-cl/branches/Branch_2_0/classloader/src/main/java/org/jboss/classloader/spi/ClassLoaderPolicy.java	2010-01-27 16:30:56 UTC (rev 100001)
+++ projects/jboss-cl/branches/Branch_2_0/classloader/src/main/java/org/jboss/classloader/spi/ClassLoaderPolicy.java	2010-01-27 16:42:26 UTC (rev 100002)
@@ -21,7 +21,6 @@
  */
 package org.jboss.classloader.spi;
 
-import java.io.File;
 import java.io.IOException;
 import java.io.InputStream;
 import java.net.URL;
@@ -48,7 +47,6 @@
  * ClassLoader policy.
  * 
  * @author <a href="adrian at jboss.com">Adrian Brock</a>
- * @author thomas.diesler at jboss.com
  * @version $Revision: 1.1 $
  */
 public abstract class ClassLoaderPolicy extends BaseClassLoaderPolicy implements ClassNotFoundHandler, ClassFoundHandler
@@ -61,62 +59,8 @@
 
    /** The class found handlers */
    private List<ClassFoundHandler> classFoundHandlers;
-
-   /** Maps native library to its provider */
-   private volatile List<NativeLibraryProvider> nativeLibraries;
    
    /**
-    * Add a native library provider.
-    * @param provider The library file provider
-    */
-   public void addNativeLibrary(NativeLibraryProvider provider)
-   {
-      if (nativeLibraries == null)
-         nativeLibraries = new CopyOnWriteArrayList<NativeLibraryProvider>();
-      
-      nativeLibraries.add(provider);
-   }
-   
-   /**
-    * Returns the absolute path name of a native library.
-    * 
-    * @param libname The library name 
-    * @return The absolute path of the native library, or null
-    */
-   public String findLibrary(String libname)
-   {
-      List<NativeLibraryProvider> list = nativeLibraries;
-      if (list == null)
-         return null;
-      
-      NativeLibraryProvider libProvider = null;
-      for (NativeLibraryProvider aux : list)
-      {
-         if (libname.equals(aux.getLibraryName()))
-         {
-            libProvider = aux;
-            break;
-         }
-      }
-      
-      if (libProvider == null)
-         return null;
-      
-      File libfile;
-      try
-      {
-         libfile = libProvider.getLibraryLocation();
-      }
-      catch (IOException ex)
-      {
-         log.error("Cannot privide native library location for: " + libname, ex);
-         return null;
-      }
-      
-      return libfile.getAbsolutePath();
-   }
-   
-   /**
     * Get the delegate loader for exported stuff<p>
     *
     * By default this uses {@link #getPackageNames()} to create a {@link FilteredDelegateLoader}

Deleted: projects/jboss-cl/branches/Branch_2_0/classloader/src/main/java/org/jboss/classloader/spi/NativeLibraryProvider.java
===================================================================
--- projects/jboss-cl/branches/Branch_2_0/classloader/src/main/java/org/jboss/classloader/spi/NativeLibraryProvider.java	2010-01-27 16:30:56 UTC (rev 100001)
+++ projects/jboss-cl/branches/Branch_2_0/classloader/src/main/java/org/jboss/classloader/spi/NativeLibraryProvider.java	2010-01-27 16:42:26 UTC (rev 100002)
@@ -1,63 +0,0 @@
-/*
-* JBoss, Home of Professional Open Source
-* Copyright 2010, JBoss Inc., and individual contributors as indicated
-* by the @authors tag. See the copyright.txt 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.classloader.spi;
-
-import java.io.File;
-import java.io.IOException;
-
-/**
- * Provides the local file location for a native library.
- * 
- * @author thomas.diesler at jboss.com
- * @author <a href="adrian at jboss.com">Adrian Brock</a>
- * @version $Revision: 1.1 $
- */
-public interface NativeLibraryProvider
-{
-   /**
-    * Get the library name.
-    * 
-    * As it is used in the call to {@link System#loadLibrary(String)} 
-    * 
-    * @return the library path
-    */
-   String getLibraryName();
-   
-   /**
-    * Get the library path.
-    * 
-    * Relative to the deployment root.
-    * 
-    * @return the library path
-    */
-   String getLibraryPath();
-   
-   /**
-    * Get the local library file location. 
-    * 
-    * This may be proved lazily.
-    * 
-    * @return The native library file
-    * @throws IOException for any error
-    */
-   File getLibraryLocation() throws IOException;
-}

Modified: projects/jboss-cl/branches/Branch_2_0/classloader/src/main/java/org/jboss/classloader/spi/base/BaseClassLoader.java
===================================================================
--- projects/jboss-cl/branches/Branch_2_0/classloader/src/main/java/org/jboss/classloader/spi/base/BaseClassLoader.java	2010-01-27 16:30:56 UTC (rev 100001)
+++ projects/jboss-cl/branches/Branch_2_0/classloader/src/main/java/org/jboss/classloader/spi/base/BaseClassLoader.java	2010-01-27 16:42:26 UTC (rev 100002)
@@ -59,8 +59,6 @@
 
 /**
  * BaseClassLoader.
- * 
- * [TODO] Add meaningful javadoc
  *
  * @author <a href="adrian at jboss.com">Adrian Brock</a>
  * @version $Revision: 1.1 $
@@ -374,20 +372,6 @@
       return domain.checkClassCacheAndBlackList(this, name, null, basePolicy.isImportAll(), false);
    }
 
-   @Override
-   protected String findLibrary(String libname)
-   {
-      String libraryPath = null;
-
-      if (policy != null)
-         libraryPath = policy.findLibrary(libname);
-
-      if (libraryPath == null)
-         libraryPath = super.findLibrary(libname);
-
-      return libraryPath;
-   }
-   
    /**
     * Find the classloader for a class but don't load the class
     *

Modified: projects/jboss-cl/branches/Branch_2_0/classloading/pom.xml
===================================================================
--- projects/jboss-cl/branches/Branch_2_0/classloading/pom.xml	2010-01-27 16:30:56 UTC (rev 100001)
+++ projects/jboss-cl/branches/Branch_2_0/classloading/pom.xml	2010-01-27 16:42:26 UTC (rev 100002)
@@ -31,14 +31,42 @@
     </dependency>
     
     <dependency>
-      <groupId>org.jboss.kernel</groupId>
+      <groupId>org.jboss.microcontainer</groupId>
       <artifactId>jboss-dependency</artifactId>
+      <exclusions>
+        <exclusion>
+          <groupId>org.jboss</groupId>
+          <artifactId>jboss-common-core</artifactId>
+        </exclusion>
+        <exclusion>
+          <groupId>jboss</groupId>
+          <artifactId>jboss-common-logging-spi</artifactId>
+        </exclusion>
+      </exclusions>
     </dependency>
     
     <dependency>
-      <groupId>org.jboss.kernel</groupId>
+      <groupId>org.jboss.microcontainer</groupId>
       <artifactId>jboss-kernel</artifactId>
       <optional>true</optional>
+      <exclusions>
+        <exclusion>
+          <groupId>org.jboss.microcontainer</groupId>
+          <artifactId>jboss-dependency</artifactId>
+        </exclusion>
+        <exclusion>
+          <groupId>org.jboss</groupId>
+          <artifactId>jboss-common-core</artifactId>
+        </exclusion>
+        <exclusion>
+          <groupId>jboss</groupId>
+          <artifactId>jboss-common-logging-spi</artifactId>
+        </exclusion>
+        <exclusion>
+          <groupId>org.jboss</groupId>
+          <artifactId>jbossxb</artifactId>
+        </exclusion>
+      </exclusions>
     </dependency>
     
     <dependency>

Modified: projects/jboss-cl/branches/Branch_2_0/classloading/src/main/java/org/jboss/classloading/spi/metadata/ClassLoadingMetaData.java
===================================================================
--- projects/jboss-cl/branches/Branch_2_0/classloading/src/main/java/org/jboss/classloading/spi/metadata/ClassLoadingMetaData.java	2010-01-27 16:30:56 UTC (rev 100001)
+++ projects/jboss-cl/branches/Branch_2_0/classloading/src/main/java/org/jboss/classloading/spi/metadata/ClassLoadingMetaData.java	2010-01-27 16:42:26 UTC (rev 100002)
@@ -22,7 +22,6 @@
 package org.jboss.classloading.spi.metadata;
 
 import java.util.List;
-
 import javax.xml.bind.annotation.XmlAttribute;
 import javax.xml.bind.annotation.XmlTransient;
 
@@ -37,8 +36,6 @@
 /**
  * ClassLoadingMetaData.
  * 
- * [TODO] Add meaningful javadoc
- * 
  * @author <a href="adrian at jboss.org">Adrian Brock</a>
  * @version $Revision: 1.1 $
  */

Modified: projects/jboss-cl/branches/Branch_2_0/classloading-vfs/pom.xml
===================================================================
--- projects/jboss-cl/branches/Branch_2_0/classloading-vfs/pom.xml	2010-01-27 16:30:56 UTC (rev 100001)
+++ projects/jboss-cl/branches/Branch_2_0/classloading-vfs/pom.xml	2010-01-27 16:42:26 UTC (rev 100002)
@@ -39,12 +39,30 @@
           <groupId>org.jboss.cl</groupId>
           <artifactId>jboss-classloader</artifactId>
         </exclusion>
+        <exclusion>
+          <groupId>org.jboss.microcontainer</groupId>
+          <artifactId>jboss-kernel</artifactId>
+        </exclusion>
       </exclusions>
     </dependency>
 
     <dependency>
-      <groupId>org.jboss.kernel</groupId>
+      <groupId>org.jboss.microcontainer</groupId>
       <artifactId>jboss-kernel</artifactId>
+      <exclusions>
+        <exclusion>
+          <groupId>org.jboss</groupId>
+          <artifactId>jboss-common-core</artifactId>
+        </exclusion>
+        <exclusion>
+          <groupId>jboss</groupId>
+          <artifactId>jboss-common-logging-spi</artifactId>
+        </exclusion>
+        <exclusion>
+          <groupId>org.jboss</groupId>
+          <artifactId>jbossxb</artifactId>
+        </exclusion>
+      </exclusions>
     </dependency>
     
     <dependency>
@@ -86,13 +104,9 @@
         </exclusion>
       </exclusions>
     </dependency>
+    
     <dependency>
       <groupId>org.jboss</groupId>
-      <artifactId>jboss-mdr</artifactId>
-      <scope>provided</scope>
-    </dependency>
-    <dependency>
-      <groupId>org.jboss</groupId>
       <artifactId>jboss-vfs</artifactId>
       <exclusions>
         <exclusion>

Modified: projects/jboss-cl/branches/Branch_2_0/classloading-vfs/src/main/java/org/jboss/classloading/spi/vfs/policy/VFSClassLoaderPolicy.java
===================================================================
--- projects/jboss-cl/branches/Branch_2_0/classloading-vfs/src/main/java/org/jboss/classloading/spi/vfs/policy/VFSClassLoaderPolicy.java	2010-01-27 16:30:56 UTC (rev 100001)
+++ projects/jboss-cl/branches/Branch_2_0/classloading-vfs/src/main/java/org/jboss/classloading/spi/vfs/policy/VFSClassLoaderPolicy.java	2010-01-27 16:42:26 UTC (rev 100002)
@@ -35,7 +35,6 @@
 import java.util.Map;
 import java.util.Set;
 import java.util.concurrent.ConcurrentHashMap;
-import java.util.concurrent.CopyOnWriteArrayList;
 import java.util.jar.Manifest;
 
 import org.jboss.classloader.plugins.ClassLoaderUtils;
@@ -54,12 +53,9 @@
 /**
  * VFSClassLoaderPolicy.
  * 
- * [TODO] add meaningful javadoc
- * 
  * @author <a href="adrian at jboss.org">Adrian Brock</a> 
  * @author <a href="ales.justin at jboss.org">Ales Justin</a>
  * @author <a href="anil.saldhana at jboss.org">Anil Saldhana</a>
- * @author Thomas.Diesler at jboss.com
  * @version $Revision: 1.1 $
  */
 public class VFSClassLoaderPolicy extends ClassLoaderPolicy
@@ -79,9 +75,6 @@
    /** The roots */
    private VirtualFile[] roots;
    
-   /** The fragment roots */
-   private List<VirtualFile> fragments;
-   
    /** The excluded roots */
    private VirtualFile[] excludedRoots;
 
@@ -268,51 +261,6 @@
       return name;
    }
 
-   /**
-    * Attach a new fragment root to the policy.
-    * @param fragRoot The fragment root file
-    */
-   public void attachFragment(VirtualFile fragRoot)
-   {
-      if (fragRoot == null)
-         throw new IllegalArgumentException("Null fragment file");
-      
-      if (fragments == null)
-         fragments = new CopyOnWriteArrayList<VirtualFile>();
-      
-      fragments.add(fragRoot);
-   }
-   
-   /**
-    * Detach a fragment root from the policy.
-    * @param fragRoot The fragment root file
-    * @return true if the fragment could be detached
-    */
-   public boolean detachFragment(VirtualFile fragRoot)
-   {
-      if (fragRoot == null)
-         throw new IllegalArgumentException("Null fragment file");
-      
-      if (fragments == null)
-         return false;
-      
-      return fragments.remove(fragRoot);
-   }
-   
-   /**
-    * Get the array of attached fragment root files.
-    * @return The array of attached fragment root files or null.
-    */
-   public VirtualFile[] getFragmentRoots()
-   {
-      if (fragments == null)
-         return null;
-      
-      VirtualFile[] retarr = new VirtualFile[fragments.size()];
-      fragments.toArray(retarr);
-      return retarr;
-   }
-
    @Override
    public List<? extends DelegateLoader> getDelegates()
    {
@@ -641,27 +589,6 @@
          {
          }
       }
-      
-      if (fragments != null)
-      {
-         for (VirtualFile root : fragments)
-         {
-            try
-            {
-               VirtualFile file = root.getChild(path);
-               if (file != null)
-               {
-                  result = new VirtualFileInfo(file, root);
-                  vfsCache.put(path, result);
-                  return result;
-               }
-            }
-            catch (Exception ignored)
-            {
-            }
-         }
-      }
-      
       return null;
    }
    

Modified: projects/jboss-cl/branches/Branch_2_0/pom.xml
===================================================================
--- projects/jboss-cl/branches/Branch_2_0/pom.xml	2010-01-27 16:30:56 UTC (rev 100001)
+++ projects/jboss-cl/branches/Branch_2_0/pom.xml	2010-01-27 16:42:26 UTC (rev 100002)
@@ -32,8 +32,7 @@
   <properties>
     <version.jboss.vfs>2.1.3.SP1</version.jboss.vfs>
     <version.jboss.man>2.1.1.CR1</version.jboss.man>
-    <version.jboss.mdr>2.2.0.Alpha1</version.jboss.mdr>
-    <version.jboss.kernel>2.2.0.Alpha2</version.jboss.kernel>
+    <version.jboss.microcontainer>2.0.9.GA</version.jboss.microcontainer>
     <version.jboss.common.core>2.2.14.GA</version.jboss.common.core>
     <version.jboss.logging.spi>2.0.5.GA</version.jboss.logging.spi>
     <version.jboss.classloading.spi>5.1.0.SP1</version.jboss.classloading.spi>
@@ -202,21 +201,15 @@
       </dependency>
       
       <dependency>
-        <groupId>org.jboss</groupId>
-        <artifactId>jboss-mdr</artifactId>
-        <version>${version.jboss.mdr}</version>
-      </dependency>
-      
-      <dependency>
-        <groupId>org.jboss.kernel</groupId>
+        <groupId>org.jboss.microcontainer</groupId>
         <artifactId>jboss-dependency</artifactId>
-        <version>${version.jboss.kernel}</version>
+        <version>${version.jboss.microcontainer}</version>
       </dependency>
       
       <dependency>
-        <groupId>org.jboss.kernel</groupId>
+        <groupId>org.jboss.microcontainer</groupId>
         <artifactId>jboss-kernel</artifactId>
-        <version>${version.jboss.kernel}</version>
+        <version>${version.jboss.microcontainer}</version>
       </dependency>
       
       <dependency>




More information about the jboss-cvs-commits mailing list