1 files changed, 27 insertions, 33 deletions

diff --git a/src/main/java/org/apache/maven/plugin/nar/NarAssemblyMojo.java b/src/main/java/org/apache/maven/plugin/nar/NarAssemblyMojo.java
index 0efe342..ed3fffd 100644
--- a/src/main/java/org/apache/maven/plugin/nar/NarAssemblyMojo.java
+++ b/src/main/java/org/apache/maven/plugin/nar/NarAssemblyMojo.java
@@ -45,9 +45,6 @@ public class NarAssemblyMojo
     /**
      * List of classifiers which you want to assemble. Example ppc-MacOSX-g++-static, x86-Windows-msvc-shared,
      * i386-Linux-g++-executable, ....
-     * 
-     * @parameter expression=""
-     * @required
      */
     private List classifiers;
 
@@ -57,46 +54,43 @@ public class NarAssemblyMojo
     public final void narExecute()
         throws MojoExecutionException, MojoFailureException
     {
-        for ( Iterator j = classifiers.iterator(); j.hasNext(); )
-        {
-            String classifier = (String) j.next();
+        List narArtifacts = getNarManager().getNarDependencies( "compile" );
+        
+        List dependencies = getNarManager().getAttachedNarDependencies( narArtifacts, classifiers );
 
-            List narArtifacts = getNarManager().getNarDependencies( "compile" );
-            List dependencies = getNarManager().getAttachedNarDependencies( narArtifacts, classifier );
-            // this may make some extra copies...
-            for ( Iterator d = dependencies.iterator(); d.hasNext(); )
-            {
-                Artifact dependency = (Artifact) d.next();
-                getLog().debug( "Assemble from " + dependency );
+        // this may make some extra copies...
+        for ( Iterator d = dependencies.iterator(); d.hasNext(); )
+        {
+            Artifact dependency = (Artifact) d.next();
+            getLog().debug( "Assemble from " + dependency );
 
-                // FIXME reported to maven developer list, isSnapshot
-                // changes behaviour
-                // of getBaseVersion, called in pathOf.
-                dependency.isSnapshot();
+            // FIXME reported to maven developer list, isSnapshot
+            // changes behaviour
+            // of getBaseVersion, called in pathOf.
+            dependency.isSnapshot();
 
-                File srcDir = getLayout().getNarUnpackDirectory(
-                        getUnpackDirectory(), 
-                        getNarManager().getNarFile( dependency ));
+            File srcDir = getLayout().getNarUnpackDirectory(
+                    getUnpackDirectory(), 
+                    getNarManager().getNarFile( dependency ));
 //                File srcDir = new File( getLocalRepository().pathOf( dependency ) );
 //                srcDir = new File( getLocalRepository().getBasedir(), srcDir.getParent() );
 //                srcDir = new File( srcDir, "nar/" );
 
-                File dstDir = getTargetDirectory();
-                try
-                {
-                    FileUtils.mkdir( dstDir.getPath() );
-                    getLog().debug( "SrcDir: " + srcDir );
-                    if ( srcDir.exists() )
-                    {
-                        FileUtils.copyDirectoryStructure( srcDir, dstDir );
-                    }
-                }
-                catch ( IOException ioe )
+            File dstDir = getTargetDirectory();
+            try
+            {
+                FileUtils.mkdir( dstDir.getPath() );
+                getLog().debug( "SrcDir: " + srcDir );
+                if ( srcDir.exists() )
                 {
-                    throw new MojoExecutionException( "Failed to copy directory for dependency " + dependency
-                        + " from " + srcDir + " to " + dstDir, ioe );
+                    FileUtils.copyDirectoryStructure( srcDir, dstDir );
                 }
             }
+            catch ( IOException ioe )
+            {
+                throw new MojoExecutionException( "Failed to copy directory for dependency " + dependency
+                    + " from " + srcDir + " to " + dstDir, ioe );
+            }
         }
     }
 }