Author: brett
Date: Thu Jun 16 20:55:50 2005
New Revision: 191079

URL: http://svn.apache.org/viewcvs?rev=191079&view=rev
Log:
make the artifact type handlers configurable

Added:
    
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/handler/DefaultArtifactHandler.java
Removed:
    
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/handler/AbstractArtifactHandler.java
    
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/handler/EjbHandler.java
    
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/handler/GenericHandler.java
    
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/handler/JarHandler.java
    
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/handler/JavadocHandler.java
    
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/handler/PlexusApplicationHandler.java
    
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/handler/PlexusServiceHandler.java
    
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/handler/PluginHandler.java
    
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/handler/PomHandler.java
    
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/handler/SourceHandler.java
    
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/handler/WarHandler.java
    
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/handler/manager/ArtifactHandlerManager.java
    
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/handler/manager/DefaultArtifactHandlerManager.java
Modified:
    
maven/components/trunk/maven-artifact-ant/src/main/resources/META-INF/plexus/components.xml
    
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/deployer/DefaultArtifactDeployer.java
    
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/handler/ArtifactHandler.java
    
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/installer/DefaultArtifactInstaller.java
    
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/repository/layout/DefaultRepositoryLayout.java
    
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/repository/layout/LegacyRepositoryLayout.java
    
maven/components/trunk/maven-artifact/src/main/resources/META-INF/plexus/components.xml
    
maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin/version/DefaultPluginVersionManager.java

Modified: 
maven/components/trunk/maven-artifact-ant/src/main/resources/META-INF/plexus/components.xml
URL: 
http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact-ant/src/main/resources/META-INF/plexus/components.xml?rev=191079&r1=191078&r2=191079&view=diff
==============================================================================
--- 
maven/components/trunk/maven-artifact-ant/src/main/resources/META-INF/plexus/components.xml
 (original)
+++ 
maven/components/trunk/maven-artifact-ant/src/main/resources/META-INF/plexus/components.xml
 Thu Jun 16 20:55:50 2005
@@ -139,43 +139,58 @@
     <component>
       <role>org.apache.maven.artifact.handler.ArtifactHandler</role>
       <role-hint>ejb</role-hint>
-      
<implementation>org.apache.maven.artifact.handler.EjbHandler</implementation>
-    </component>
-
-    <component>
-      <role>org.apache.maven.artifact.handler.ArtifactHandler</role>
-      <role-hint>plexus-service</role-hint>
-      
<implementation>org.apache.maven.artifact.handler.PlexusServiceHandler</implementation>
+      
<implementation>org.apache.maven.artifact.handler.DefaultArtifactHandler</implementation>
+      <configuration>
+        <type>ejb</type>
+        <extension>jar</extension>
+      </configuration>
     </component>
 
     <component>
       <role>org.apache.maven.artifact.handler.ArtifactHandler</role>
       <role-hint>jar</role-hint>
-      
<implementation>org.apache.maven.artifact.handler.JarHandler</implementation>
+      
<implementation>org.apache.maven.artifact.handler.DefaultArtifactHandler</implementation>
+      <configuration>
+        <type>jar</type>
+      </configuration>
     </component>
 
     <component>
       <role>org.apache.maven.artifact.handler.ArtifactHandler</role>
       <role-hint>maven-plugin</role-hint>
-      
<implementation>org.apache.maven.artifact.handler.PluginHandler</implementation>
+      
<implementation>org.apache.maven.artifact.handler.DefaultArtifactHandler</implementation>
+      <configuration>
+        <type>maven-plugin</type>
+        <extension>jar</extension>
+      </configuration>
     </component>
 
     <component>
       <role>org.apache.maven.artifact.handler.ArtifactHandler</role>
       <role-hint>pom</role-hint>
-      
<implementation>org.apache.maven.artifact.handler.PomHandler</implementation>
+      
<implementation>org.apache.maven.artifact.handler.DefaultArtifactHandler</implementation>
+      <configuration>
+        <type>pom</type>
+      </configuration>
     </component>
 
     <component>
       <role>org.apache.maven.artifact.handler.ArtifactHandler</role>
-      <role-hint>source</role-hint>
-      
<implementation>org.apache.maven.artifact.handler.SourceHandler</implementation>
+      <role-hint>java-source</role-hint>
+      
<implementation>org.apache.maven.artifact.handler.DefaultArtifactHandler</implementation>
+      <configuration>
+        <type>java-source</type>
+        <extension>jar</extension>
+      </configuration>
     </component>
 
     <component>
       <role>org.apache.maven.artifact.handler.ArtifactHandler</role>
       <role-hint>war</role-hint>
-      
<implementation>org.apache.maven.artifact.handler.WarHandler</implementation>
+      
<implementation>org.apache.maven.artifact.handler.DefaultArtifactHandler</implementation>
+      <configuration>
+        <type>war</type>
+      </configuration>
     </component>
 
     <!--

Modified: 
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/deployer/DefaultArtifactDeployer.java
URL: 
http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/deployer/DefaultArtifactDeployer.java?rev=191079&r1=191078&r2=191079&view=diff
==============================================================================
--- 
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/deployer/DefaultArtifactDeployer.java
 (original)
+++ 
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/deployer/DefaultArtifactDeployer.java
 Thu Jun 16 20:55:50 2005
@@ -52,7 +52,7 @@
 
         try
         {
-            String extension = artifactHandlerManager.getArtifactHandler( 
artifact.getType() ).extension();
+            String extension = artifactHandlerManager.getArtifactHandler( 
artifact.getType() ).getExtension();
             source = new File( basedir, finalName + "." + extension );
         }
         catch ( ArtifactHandlerNotFoundException e )

Modified: 
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/handler/ArtifactHandler.java
URL: 
http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/handler/ArtifactHandler.java?rev=191079&r1=191078&r2=191079&view=diff
==============================================================================
--- 
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/handler/ArtifactHandler.java
 (original)
+++ 
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/handler/ArtifactHandler.java
 Thu Jun 16 20:55:50 2005
@@ -24,7 +24,11 @@
 {
     static String ROLE = ArtifactHandler.class.getName();
 
-    String extension();
+    String getExtension();
 
-    String directory();
+    String getDirectory();
+
+    String getClassifier();
+
+    String getPackaging();
 }

Added: 
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/handler/DefaultArtifactHandler.java
URL: 
http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/handler/DefaultArtifactHandler.java?rev=191079&view=auto
==============================================================================
--- 
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/handler/DefaultArtifactHandler.java
 (added)
+++ 
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/handler/DefaultArtifactHandler.java
 Thu Jun 16 20:55:50 2005
@@ -0,0 +1,81 @@
+package org.apache.maven.artifact.handler;
+
+/*
+ * Copyright 2001-2005 The Apache Software Foundation.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+/**
+ * @author <a href="mailto:[EMAIL PROTECTED]">Brett Porter</a>
+ * @version $Id: AbstractArtifactHandler.java 189871 2005-06-10 00:57:19Z 
brett $
+ */
+public class DefaultArtifactHandler
+    implements ArtifactHandler
+{
+    private String extension;
+
+    private String type;
+
+    private String classifier;
+
+    private String directory;
+
+    private String packaging;
+
+    public DefaultArtifactHandler()
+    {
+    }
+
+    public DefaultArtifactHandler( String type )
+    {
+        this.type = type;
+    }
+
+    public String getExtension()
+    {
+        if ( extension == null )
+        {
+            extension = type;
+        }
+        return extension;
+    }
+
+    public String getType()
+    {
+        return type;
+    }
+
+    public String getClassifier()
+    {
+        return classifier;
+    }
+
+    public String getDirectory()
+    {
+        if ( directory == null )
+        {
+            directory = type + "s";
+        }
+        return directory;
+    }
+
+    public String getPackaging()
+    {
+        if ( packaging == null )
+        {
+            packaging = type;
+        }
+        return packaging;
+    }
+}

Modified: 
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/installer/DefaultArtifactInstaller.java
URL: 
http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/installer/DefaultArtifactInstaller.java?rev=191079&r1=191078&r2=191079&view=diff
==============================================================================
--- 
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/installer/DefaultArtifactInstaller.java
 (original)
+++ 
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/installer/DefaultArtifactInstaller.java
 Thu Jun 16 20:55:50 2005
@@ -47,7 +47,7 @@
 
         try
         {
-            String extension = artifactHandlerManager.getArtifactHandler( 
artifact.getType() ).extension();
+            String extension = artifactHandlerManager.getArtifactHandler( 
artifact.getType() ).getExtension();
             source = new File( basedir, finalName + "." + extension );
         }
         catch ( ArtifactHandlerNotFoundException e )

Modified: 
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/repository/layout/DefaultRepositoryLayout.java
URL: 
http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/repository/layout/DefaultRepositoryLayout.java?rev=191079&r1=191078&r2=191079&view=diff
==============================================================================
--- 
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/repository/layout/DefaultRepositoryLayout.java
 (original)
+++ 
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/repository/layout/DefaultRepositoryLayout.java
 Thu Jun 16 20:55:50 2005
@@ -62,9 +62,9 @@
             path.append( '-' ).append( artifact.getClassifier() );
         }
 
-        if ( artifactHandler.extension() != null && 
artifactHandler.extension().length() > 0 )
+        if ( artifactHandler.getExtension() != null && 
artifactHandler.getExtension().length() > 0 )
         {
-            path.append( '.' ).append( artifactHandler.extension() );
+            path.append( '.' ).append( artifactHandler.getExtension() );
         }
 
         return path.toString();

Modified: 
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/repository/layout/LegacyRepositoryLayout.java
URL: 
http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/repository/layout/LegacyRepositoryLayout.java?rev=191079&r1=191078&r2=191079&view=diff
==============================================================================
--- 
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/repository/layout/LegacyRepositoryLayout.java
 (original)
+++ 
maven/components/trunk/maven-artifact/src/main/java/org/apache/maven/artifact/repository/layout/LegacyRepositoryLayout.java
 Thu Jun 16 20:55:50 2005
@@ -49,7 +49,7 @@
         StringBuffer path = new StringBuffer();
 
         path.append( artifact.getGroupId() ).append( '/' );
-        path.append( artifactHandler.directory() ).append( '/' );
+        path.append( artifactHandler.getDirectory() ).append( '/' );
         path.append( artifact.getArtifactId() ).append( '-' ).append( 
artifact.getVersion() );
 
         if ( artifact.hasClassifier() )
@@ -57,9 +57,9 @@
             path.append( '-' ).append( artifact.getClassifier() );
         }
 
-        if ( artifactHandler.extension() != null && 
artifactHandler.extension().length() > 0 )
+        if ( artifactHandler.getExtension() != null && 
artifactHandler.getExtension().length() > 0 )
         {
-            path.append( '.' ).append( artifactHandler.extension() );
+            path.append( '.' ).append( artifactHandler.getExtension() );
         }
 
         return path.toString();

Modified: 
maven/components/trunk/maven-artifact/src/main/resources/META-INF/plexus/components.xml
URL: 
http://svn.apache.org/viewcvs/maven/components/trunk/maven-artifact/src/main/resources/META-INF/plexus/components.xml?rev=191079&r1=191078&r2=191079&view=diff
==============================================================================
--- 
maven/components/trunk/maven-artifact/src/main/resources/META-INF/plexus/components.xml
 (original)
+++ 
maven/components/trunk/maven-artifact/src/main/resources/META-INF/plexus/components.xml
 Thu Jun 16 20:55:50 2005
@@ -148,49 +148,58 @@
     <component>
       <role>org.apache.maven.artifact.handler.ArtifactHandler</role>
       <role-hint>ejb</role-hint>
-      
<implementation>org.apache.maven.artifact.handler.EjbHandler</implementation>
-    </component>
-
-    <component>
-      <role>org.apache.maven.artifact.handler.ArtifactHandler</role>
-      <role-hint>plexus-application</role-hint>
-      
<implementation>org.apache.maven.artifact.handler.PlexusApplicationHandler</implementation>
-    </component>
-
-    <component>
-      <role>org.apache.maven.artifact.handler.ArtifactHandler</role>
-      <role-hint>plexus-service</role-hint>
-      
<implementation>org.apache.maven.artifact.handler.PlexusServiceHandler</implementation>
+      
<implementation>org.apache.maven.artifact.handler.DefaultArtifactHandler</implementation>
+      <configuration>
+        <type>ejb</type>
+        <extension>jar</extension>
+      </configuration>
     </component>
 
     <component>
       <role>org.apache.maven.artifact.handler.ArtifactHandler</role>
       <role-hint>jar</role-hint>
-      
<implementation>org.apache.maven.artifact.handler.JarHandler</implementation>
+      
<implementation>org.apache.maven.artifact.handler.DefaultArtifactHandler</implementation>
+      <configuration>
+        <type>jar</type>
+      </configuration>
     </component>
 
     <component>
       <role>org.apache.maven.artifact.handler.ArtifactHandler</role>
       <role-hint>maven-plugin</role-hint>
-      
<implementation>org.apache.maven.artifact.handler.PluginHandler</implementation>
+      
<implementation>org.apache.maven.artifact.handler.DefaultArtifactHandler</implementation>
+      <configuration>
+        <type>maven-plugin</type>
+        <extension>jar</extension>
+      </configuration>
     </component>
 
     <component>
       <role>org.apache.maven.artifact.handler.ArtifactHandler</role>
       <role-hint>pom</role-hint>
-      
<implementation>org.apache.maven.artifact.handler.PomHandler</implementation>
+      
<implementation>org.apache.maven.artifact.handler.DefaultArtifactHandler</implementation>
+      <configuration>
+        <type>pom</type>
+      </configuration>
     </component>
 
     <component>
       <role>org.apache.maven.artifact.handler.ArtifactHandler</role>
-      <role-hint>source</role-hint>
-      
<implementation>org.apache.maven.artifact.handler.SourceHandler</implementation>
+      <role-hint>java-source</role-hint>
+      
<implementation>org.apache.maven.artifact.handler.DefaultArtifactHandler</implementation>
+      <configuration>
+        <type>java-source</type>
+        <extension>jar</extension>
+      </configuration>
     </component>
 
     <component>
       <role>org.apache.maven.artifact.handler.ArtifactHandler</role>
       <role-hint>war</role-hint>
-      
<implementation>org.apache.maven.artifact.handler.WarHandler</implementation>
+      
<implementation>org.apache.maven.artifact.handler.DefaultArtifactHandler</implementation>
+      <configuration>
+        <type>war</type>
+      </configuration>
     </component>
 
     <component>

Modified: 
maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin/version/DefaultPluginVersionManager.java
URL: 
http://svn.apache.org/viewcvs/maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin/version/DefaultPluginVersionManager.java?rev=191079&r1=191078&r2=191079&view=diff
==============================================================================
--- 
maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin/version/DefaultPluginVersionManager.java
 (original)
+++ 
maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin/version/DefaultPluginVersionManager.java
 Thu Jun 16 20:55:50 2005
@@ -2,7 +2,6 @@
 
 import org.apache.maven.artifact.Artifact;
 import org.apache.maven.artifact.factory.ArtifactFactory;
-import org.apache.maven.artifact.handler.PluginHandler;
 import org.apache.maven.artifact.repository.ArtifactRepository;
 import org.apache.maven.artifact.resolver.ArtifactResolutionException;
 import org.apache.maven.artifact.resolver.ArtifactResolver;
@@ -531,7 +530,7 @@
         throws PluginVersionResolutionException
     {
         Artifact releaseArtifact = artifactFactory.createArtifact( groupId, 
artifactId, "RELEASE",
-                                                                   
Artifact.SCOPE_RUNTIME, PluginHandler.PLUGIN_TYPE );
+                                                                   
Artifact.SCOPE_RUNTIME, "maven-plugin" );
 
         try
         {



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to