add dependency handling, and make bugfixes to the assemble plugin

git-svn-id: https://svn.apache.org/repos/asf/maven/components/trunk@163798 13f79535-47bb-0310-9956-ffa450edef68

maven-plugins/maven-assemble-plugin/pom.xml

@@ -10,6 +10,11 @@
   <name>Maven Assemble Plugin</name>
   <version>1.0-SNAPSHOT</version>
   <dependencies>
+    <dependency>
+      <groupId>org.apache.maven</groupId>
+      <artifactId>maven-artifact</artifactId>
+      <version>2.0-SNAPSHOT</version>
+    </dependency>
     <dependency>
       <groupId>plexus</groupId>
       <artifactId>plexus-archiver</artifactId>

maven-plugins/maven-assemble-plugin/src/main/java/org/apache/maven/plugin/assemble/AssembleMojo.java

@@ -16,12 +16,17 @@
  * limitations under the License.
  */
 
+import org.apache.maven.artifact.Artifact;
+import org.apache.maven.artifact.resolver.filter.ArtifactFilter;
+import org.apache.maven.artifact.resolver.filter.ScopeArtifactFilter;
 import org.apache.maven.plugin.AbstractPlugin;
 import org.apache.maven.plugin.PluginExecutionException;
 import org.apache.maven.plugins.assemble.model.Assembly;
+import org.apache.maven.plugins.assemble.model.DependencySet;
 import org.apache.maven.plugins.assemble.model.FileSet;
 import org.apache.maven.plugins.assemble.model.io.xpp3.AssemblyXpp3Reader;
 import org.codehaus.plexus.archiver.Archiver;
+import org.codehaus.plexus.archiver.ArchiverException;
 import org.codehaus.plexus.archiver.jar.JarArchiver;
 import org.codehaus.plexus.archiver.tar.TarArchiver;
 import org.codehaus.plexus.archiver.zip.ZipArchiver;
@@ -32,18 +37,23 @@
 import java.io.InputStream;
 import java.io.InputStreamReader;
 import java.io.Reader;
+import java.util.ArrayList;
 import java.util.Iterator;
+import java.util.List;
+import java.util.Set;
 
 /**
  * @author <a href="mailto:brett@apache.org">Brett Porter</a>
  * @version $Id$
  * @goal assemble
+ * @requiresDependencyResolution test
  * @description assemble an application bundle or distribution
  * @parameter name="basedir" type="String" required="true" validator="" expression="#basedir" description=""
  * @parameter name="outputDirectory" type="java.io.File" required="true" validator="" expression="#project.build.directory" description=""
  * @parameter name="descriptor" type="java.io.File" required="false" validator="" expression="#maven.assemble.descriptor" description=""
  * @parameter name="finalName" type="String" required="true" validator="" expression="#project.build.finalName" description=""
  * @parameter name="descriptorId" type="String" required="false" validator="" expression="#maven.assemble.descriptorId" description=""
+ * @parameter name="dependencies" type="java.util.Set" required="false" validator="" expression="#project.artifacts" description=""
  */
 public class AssembleMojo
     extends AbstractPlugin
@@ -63,6 +73,8 @@ public class AssembleMojo
 
     private String finalName;
 
+    private Set dependencies;
+
     public void execute()
         throws PluginExecutionException
     {
@@ -108,7 +120,7 @@ else if ( descriptorId != null )
             Assembly assembly = reader.read( r );
 
             // TODO: include dependencies marked for distribution under certain formats
-            // TODO: how, might we plugin this into an installer, such as NSIS?
+            // TODO: how, might we plug this into an installer, such as NSIS?
             // TODO: allow file mode specifications?
 
             String fullName = finalName + "-" + assembly.getId();
@@ -120,6 +132,80 @@ else if ( descriptorId != null )
                 String filename = fullName + "." + format;
 
                 // TODO: use component roles? Can we do that in a mojo?
+                Archiver archiver = createArchiver( format );
+
+                processFileSets( archiver, assembly.getFileSets() );
+                processDependencySets( archiver, assembly.getDependencySets() );
+
+                archiver.setDestFile( new File( outputDirectory, filename ) );
+                archiver.createArchive();
+            }
+        }
+        finally
+        {
+            IOUtil.close( r );
+        }
+    }
+
+    private void processDependencySets( Archiver archiver, List dependencySets )
+        throws ArchiverException
+    {
+        for ( Iterator i = dependencySets.iterator(); i.hasNext(); )
+        {
+            DependencySet depedencySet = (DependencySet) i.next();
+            String output = depedencySet.getOutputDirectory();
+            output = getOutputDirectory( output );
+
+            AndArtifactFilter filter = new AndArtifactFilter();
+            filter.add( new ScopeArtifactFilter( depedencySet.getScope() ) );
+            if ( !depedencySet.getIncludes().isEmpty() )
+            {
+                filter.add( new IncludesArtifactFilter( depedencySet.getIncludes() ) );
+            }
+            if ( !depedencySet.getExcludes().isEmpty() )
+            {
+                filter.add( new ExcludesArtifactFilter( depedencySet.getExcludes() ) );
+            }
+
+            // TODO: includes and excludes
+            for ( Iterator j = dependencies.iterator(); j.hasNext(); )
+            {
+                Artifact artifact = (Artifact) j.next();
+
+                if ( filter.include( artifact ) )
+                {
+                    archiver.addFile( artifact.getFile(), output + artifact.getFile().getName() );
+                }
+            }
+        }
+    }
+
+    private String getOutputDirectory( String output )
+    {
+        if ( output == null )
+        {
+            output = "";
+        }
+        if ( !output.endsWith( "/" ) && !output.endsWith( "\\" ) )
+        {
+            // TODO: shouldn't archiver do this?
+            output += '/';
+        }
+
+        if ( output.startsWith( "/" ) )
+        {
+            output = finalName + output;
+        }
+        else
+        {
+            output = finalName + "/" + output;
+        }
+        return output;
+    }
+
+    private Archiver createArchiver( String format )
+        throws ArchiverException
+    {
         Archiver archiver;
         if ( format.startsWith( "tar" ) )
         {
@@ -162,18 +248,23 @@ else if ( format.startsWith( "jar" ) )
             // TODO: better handling
             throw new IllegalArgumentException( "Unknown format: " + format );
         }
+        return archiver;
+    }
 
-                for ( Iterator j = assembly.getFilesets().iterator(); j.hasNext(); )
+    private void processFileSets( Archiver archiver, java.util.List fileSets )
+        throws ArchiverException
     {
-                    FileSet fileset = (FileSet) j.next();
+        for ( Iterator i = fileSets.iterator(); i.hasNext(); )
-                    String directory = fileset.getDirectory();
+        {
-                    String output = fileset.getOutputDirectory();
+            FileSet fileSet = (FileSet) i.next();
+            String directory = fileSet.getDirectory();
+            String output = fileSet.getOutputDirectory();
             if ( directory == null )
             {
                 directory = basedir;
                 if ( output == null )
                 {
-                            output = "/";
+                    output = "";
                 }
             }
             else
@@ -183,33 +274,118 @@ else if ( format.startsWith( "jar" ) )
                     output = directory;
                 }
             }
-                    if ( !output.endsWith( "/" ) && !output.endsWith( "\\" ) )
+            output = getOutputDirectory( output );
-                    {
-                        // TODO: shouldn't archiver do this?
-                        output += '/';
-                    }
 
-                    String[] includes = (String[]) fileset.getIncludes().toArray( EMPTY_STRING_ARRAY );
+            String[] includes = (String[]) fileSet.getIncludes().toArray( EMPTY_STRING_ARRAY );
             if ( includes.length == 0 )
             {
                 includes = null;
             }
-                    String[] excludes = null;
+
-                    if ( !fileset.getExcludes().isEmpty() )
+            List excludesList = fileSet.getExcludes();
-                    {
+            excludesList.addAll( getDefaultExcludes() );
-                        excludes = (String[]) fileset.getExcludes().toArray( EMPTY_STRING_ARRAY );
+            String[] excludes = (String[]) excludesList.toArray( EMPTY_STRING_ARRAY );
-                    }
+
-                    // TODO: default excludes?
+            // TODO: default excludes should be in the archiver?
             archiver.addDirectory( new File( directory ), output, includes, excludes );
         }
+    }
 
-                archiver.setDestFile( new File( outputDirectory, filename ) );
+    public List getDefaultExcludes()
-                archiver.createArchive();
-            }
-        }
-        finally
     {
-            IOUtil.close( r );
+        List defaultExcludes = new ArrayList();
+        defaultExcludes.add( "**/*~" );
+        defaultExcludes.add( "**/#*#" );
+        defaultExcludes.add( "**/.#*" );
+        defaultExcludes.add( "**/%*%" );
+        defaultExcludes.add( "**/._*" );
+
+        // CVS
+        defaultExcludes.add( "**/CVS" );
+        defaultExcludes.add( "**/CVS/**" );
+        defaultExcludes.add( "**/.cvsignore" );
+
+        // SCCS
+        defaultExcludes.add( "**/SCCS" );
+        defaultExcludes.add( "**/SCCS/**" );
+
+        // Visual SourceSafe
+        defaultExcludes.add( "**/vssver.scc" );
+
+        // Subversion
+        defaultExcludes.add( "**/.svn" );
+        defaultExcludes.add( "**/.svn/**" );
+
+        // Mac
+        defaultExcludes.add( "**/.DS_Store" );
+
+        return defaultExcludes;
+    }
+
+    // TODO: move to maven-artifact - generally useful
+    private static class AndArtifactFilter
+        implements ArtifactFilter
+    {
+        private final List filters = new ArrayList();
+
+        public boolean include( Artifact artifact )
+        {
+            boolean include = true;
+            for ( Iterator i = filters.iterator(); i.hasNext() && include; )
+            {
+                ArtifactFilter filter = (ArtifactFilter) i.next();
+                if ( !filter.include( artifact ) )
+                {
+                    include = false;
+                }
+            }
+            return include;
+        }
+
+        public void add( ArtifactFilter artifactFilter )
+        {
+            filters.add( artifactFilter );
+        }
+    }
+
+    private static class IncludesArtifactFilter
+        implements ArtifactFilter
+    {
+        private final List patterns;
+
+        public IncludesArtifactFilter( List patterns )
+        {
+            this.patterns = patterns;
+        }
+
+        public boolean include( Artifact artifact )
+        {
+            String id = artifact.getGroupId() + ":" + artifact.getArtifactId();
+
+            boolean matched = false;
+            for ( Iterator i = patterns.iterator(); i.hasNext() & !matched; )
+            {
+                // TODO: what about wildcards? Just specifying groups? versions?
+                if ( id.equals( i.next() ) )
+                {
+                    matched = true;
+                }
+            }
+            return matched;
+        }
+    }
+
+    private static class ExcludesArtifactFilter
+        extends IncludesArtifactFilter
+    {
+        public ExcludesArtifactFilter( List patterns )
+        {
+            super( patterns );
+        }
+
+        public boolean include( Artifact artifact )
+        {
+            return !super.include( artifact );
         }
     }
 }

maven-plugins/maven-assemble-plugin/src/main/mdo/descriptor.mdo

@@ -30,14 +30,21 @@
           </association>
         </field>
         <field>
-          <name>filesets</name>
+          <name>fileSets</name>
           <version>1.0.0</version>
-          <required>true</required>
           <association>
             <type>FileSet</type>
             <multiplicity>*</multiplicity>
           </association>
         </field>
+        <field>
+          <name>dependencySets</name>
+          <version>1.0.0</version>
+          <association>
+            <type>DependencySet</type>
+            <multiplicity>*</multiplicity>
+          </association>
+        </field>
       </fields>
     </class>
     <class>
@@ -73,6 +80,40 @@
         </field>
       </fields>
     </class>
+    <class>
+      <name>DependencySet</name>
+      <version>1.0.0</version>
+      <fields>
+        <field>
+          <name>outputDirectory</name>
+          <version>1.0.0</version>
+          <type>String</type>
+        </field>
+        <field>
+          <name>scope</name>
+          <version>1.0.0</version>
+          <type>String</type>
+          <defaultValue>runtime</defaultValue>
+          <required>true</required>
+        </field>
+        <field>
+          <name>includes</name>
+          <version>1.0.0</version>
+          <association>
+            <type>String</type>
+            <multiplicity>*</multiplicity>
+          </association>
+        </field>
+        <field>
+          <name>excludes</name>
+          <version>1.0.0</version>
+          <association>
+            <type>String</type>
+            <multiplicity>*</multiplicity>
+          </association>
+        </field>
+      </fields>
+    </class>
   </classes>
 </model>
 

maven-plugins/maven-assemble-plugin/src/main/resources/assemblies/bin.xml

@@ -5,22 +5,22 @@
     <format>tar.bz2</format>
     <format>zip</format>
   </formats>
-  <filesets>
+  <fileSets>
-    <fileset>
+    <fileSet>
       <includes>
         <include>README*</include>
         <include>LICENSE*</include>
         <include>NOTICE*</include>
       </includes>
-    </fileset>
+    </fileSet>
     <!-- TODO: docs? -->
-    <fileset>
+    <fileSet>
       <!-- TODO: use expresssions instead: ${project.build.directory}, ${project.build.finalName} -->
       <directory>target</directory>
-      <outputDirectory>/</outputDirectory>
+      <outputDirectory></outputDirectory>
       <includes>
         <include>*.jar</include>
       </includes>
-    </fileset>
+    </fileSet>
-  </filesets>
+  </fileSets>
 </assembly>

maven-plugins/maven-assemble-plugin/src/main/resources/assemblies/src.xml

@@ -5,18 +5,18 @@
     <format>tar.bz2</format>
     <format>zip</format>
   </formats>
-  <filesets>
+  <fileSets>
-    <fileset>
+    <fileSet>
       <includes>
         <include>README*</include>
         <include>LICENSE*</include>
         <include>NOTICE*</include>
         <include>pom.xml</include>
       </includes>
-    </fileset>
+    </fileSet>
-    <fileset>
+    <fileSet>
       <!-- TODO: use expresssions instead: ${project.build.sourceDirectory}, etc -->
       <directory>src</directory>
-    </fileset>
+    </fileSet>
-  </filesets>
+  </fileSets>
 </assembly>