Author: jdcasey
Date: Thu Sep 25 12:35:49 2008
New Revision: 699061

URL: http://svn.apache.org/viewvc?rev=699061&view=rev
Log:
[MASSEMBLY-291] If the project type is 'pom', and the assembly doesn't have the 
assemblyId as part of its coordinate, then attach the assembly file with a null 
classifier to ensure it gets installed/deployed normally.

Added:
    
maven/plugins/trunk/maven-assembly-plugin/src/it/projects/mojo-configuration/massembly-291/
    
maven/plugins/trunk/maven-assembly-plugin/src/it/projects/mojo-configuration/massembly-291/goals.txt
   (with props)
    
maven/plugins/trunk/maven-assembly-plugin/src/it/projects/mojo-configuration/massembly-291/pom.xml
   (with props)
    
maven/plugins/trunk/maven-assembly-plugin/src/it/projects/mojo-configuration/massembly-291/src/
    
maven/plugins/trunk/maven-assembly-plugin/src/it/projects/mojo-configuration/massembly-291/src/main/
    
maven/plugins/trunk/maven-assembly-plugin/src/it/projects/mojo-configuration/massembly-291/src/main/assembly/
    
maven/plugins/trunk/maven-assembly-plugin/src/it/projects/mojo-configuration/massembly-291/src/main/assembly/src.xml
   (with props)
    
maven/plugins/trunk/maven-assembly-plugin/src/it/projects/mojo-configuration/massembly-291/test.properties
   (with props)
    
maven/plugins/trunk/maven-assembly-plugin/src/it/projects/mojo-configuration/massembly-291/verify.bsh
   (with props)
Modified:
    
maven/plugins/trunk/maven-assembly-plugin/src/main/java/org/apache/maven/plugin/assembly/mojos/AbstractAssemblyMojo.java

Added: 
maven/plugins/trunk/maven-assembly-plugin/src/it/projects/mojo-configuration/massembly-291/goals.txt
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-assembly-plugin/src/it/projects/mojo-configuration/massembly-291/goals.txt?rev=699061&view=auto
==============================================================================
--- 
maven/plugins/trunk/maven-assembly-plugin/src/it/projects/mojo-configuration/massembly-291/goals.txt
 (added)
+++ 
maven/plugins/trunk/maven-assembly-plugin/src/it/projects/mojo-configuration/massembly-291/goals.txt
 Thu Sep 25 12:35:49 2008
@@ -0,0 +1,2 @@
+clean
+deploy
\ No newline at end of file

Propchange: 
maven/plugins/trunk/maven-assembly-plugin/src/it/projects/mojo-configuration/massembly-291/goals.txt
------------------------------------------------------------------------------
    svn:eol-style = native

Added: 
maven/plugins/trunk/maven-assembly-plugin/src/it/projects/mojo-configuration/massembly-291/pom.xml
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-assembly-plugin/src/it/projects/mojo-configuration/massembly-291/pom.xml?rev=699061&view=auto
==============================================================================
--- 
maven/plugins/trunk/maven-assembly-plugin/src/it/projects/mojo-configuration/massembly-291/pom.xml
 (added)
+++ 
maven/plugins/trunk/maven-assembly-plugin/src/it/projects/mojo-configuration/massembly-291/pom.xml
 Thu Sep 25 12:35:49 2008
@@ -0,0 +1,37 @@
+
+<project xmlns="http://maven.apache.org/POM/4.0.0"; 
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance";
+  xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 
http://maven.apache.org/maven-v4_0_0.xsd";>
+  <modelVersion>4.0.0</modelVersion>
+  <parent>
+    <groupId>org.apache.maven.plugin.assembly.test</groupId>
+    <artifactId>it-project-parent</artifactId>
+    <version>1</version>
+  </parent>
+  
+  <groupId>org.test</groupId>
+  <artifactId>massembly-291</artifactId>
+  <packaging>pom</packaging>
+  <version>1</version>
+  <build>
+    <plugins>
+      <plugin>
+        <artifactId>maven-assembly-plugin</artifactId>
+        <executions>
+          <execution>
+            <id>assembly</id>
+            <phase>package</phase>
+            <goals>
+              <goal>single</goal>
+            </goals>
+            <configuration>
+              <descriptors>
+                <descriptor>src/main/assembly/src.xml</descriptor>
+              </descriptors>
+              <appendAssemblyId>false</appendAssemblyId>
+            </configuration>
+          </execution>
+        </executions>
+      </plugin>
+    </plugins>
+  </build>
+</project>

Propchange: 
maven/plugins/trunk/maven-assembly-plugin/src/it/projects/mojo-configuration/massembly-291/pom.xml
------------------------------------------------------------------------------
    svn:eol-style = native

Added: 
maven/plugins/trunk/maven-assembly-plugin/src/it/projects/mojo-configuration/massembly-291/src/main/assembly/src.xml
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-assembly-plugin/src/it/projects/mojo-configuration/massembly-291/src/main/assembly/src.xml?rev=699061&view=auto
==============================================================================
--- 
maven/plugins/trunk/maven-assembly-plugin/src/it/projects/mojo-configuration/massembly-291/src/main/assembly/src.xml
 (added)
+++ 
maven/plugins/trunk/maven-assembly-plugin/src/it/projects/mojo-configuration/massembly-291/src/main/assembly/src.xml
 Thu Sep 25 12:35:49 2008
@@ -0,0 +1,17 @@
+<assembly>
+  <id>src</id>
+  <formats>
+    <format>zip</format>
+    <format>tar.gz</format>
+  </formats>
+  <fileSets>
+    <fileSet>
+      <directory>${project.basedir}</directory>
+      <outputDirectory>/</outputDirectory>
+      <useDefaultExcludes>true</useDefaultExcludes>
+      <excludes>
+        <exclude>target</exclude>
+      </excludes>
+    </fileSet>
+  </fileSets>
+</assembly>
\ No newline at end of file

Propchange: 
maven/plugins/trunk/maven-assembly-plugin/src/it/projects/mojo-configuration/massembly-291/src/main/assembly/src.xml
------------------------------------------------------------------------------
    svn:eol-style = native

Added: 
maven/plugins/trunk/maven-assembly-plugin/src/it/projects/mojo-configuration/massembly-291/test.properties
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-assembly-plugin/src/it/projects/mojo-configuration/massembly-291/test.properties?rev=699061&view=auto
==============================================================================
--- 
maven/plugins/trunk/maven-assembly-plugin/src/it/projects/mojo-configuration/massembly-291/test.properties
 (added)
+++ 
maven/plugins/trunk/maven-assembly-plugin/src/it/projects/mojo-configuration/massembly-291/test.properties
 Thu Sep 25 12:35:49 2008
@@ -0,0 +1 @@
+altDeploymentRepository=test::default::file://${project.build.directory}/deploy

Propchange: 
maven/plugins/trunk/maven-assembly-plugin/src/it/projects/mojo-configuration/massembly-291/test.properties
------------------------------------------------------------------------------
    svn:eol-style = native

Added: 
maven/plugins/trunk/maven-assembly-plugin/src/it/projects/mojo-configuration/massembly-291/verify.bsh
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-assembly-plugin/src/it/projects/mojo-configuration/massembly-291/verify.bsh?rev=699061&view=auto
==============================================================================
--- 
maven/plugins/trunk/maven-assembly-plugin/src/it/projects/mojo-configuration/massembly-291/verify.bsh
 (added)
+++ 
maven/plugins/trunk/maven-assembly-plugin/src/it/projects/mojo-configuration/massembly-291/verify.bsh
 Thu Sep 25 12:35:49 2008
@@ -0,0 +1,38 @@
+import java.io.*;
+
+String base = "target/deploy/org/test/massembly-291/1/massembly-291-1";
+File pom = new File( basedir, base + ".pom" );
+File zip = new File( basedir, base + ".zip" );
+File tgz = new File( basedir, base + ".tar.gz" );
+
+{
+       File f = pom;
+       String type = "POM";
+       
+       if ( !f.exists() )
+       {
+               System.out.println( "Project " + type + " is missing in 
deployment repository. File was: " + f );
+       }
+}
+
+{
+       File f = zip;
+       String type = "sources zip";
+       
+       if ( !f.exists() )
+       {
+               System.out.println( "Project " + type + " is missing in 
deployment repository. File was: " + f );
+       }
+}
+
+{
+       File f = tgz;
+       String type = "sources tar.gz";
+       
+       if ( !f.exists() )
+       {
+               System.out.println( "Project " + type + " is missing in 
deployment repository. File was: " + f );
+       }
+}
+
+return pom.exists() && zip.exists() && tgz.exists();
\ No newline at end of file

Propchange: 
maven/plugins/trunk/maven-assembly-plugin/src/it/projects/mojo-configuration/massembly-291/verify.bsh
------------------------------------------------------------------------------
    svn:eol-style = native

Modified: 
maven/plugins/trunk/maven-assembly-plugin/src/main/java/org/apache/maven/plugin/assembly/mojos/AbstractAssemblyMojo.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-assembly-plugin/src/main/java/org/apache/maven/plugin/assembly/mojos/AbstractAssemblyMojo.java?rev=699061&r1=699060&r2=699061&view=diff
==============================================================================
--- 
maven/plugins/trunk/maven-assembly-plugin/src/main/java/org/apache/maven/plugin/assembly/mojos/AbstractAssemblyMojo.java
 (original)
+++ 
maven/plugins/trunk/maven-assembly-plugin/src/main/java/org/apache/maven/plugin/assembly/mojos/AbstractAssemblyMojo.java
 Thu Sep 25 12:35:49 2008
@@ -364,7 +364,7 @@
                         {
                             projectHelper.attachArtifact( project, format, 
classifier, destFile );
                         }
-                        else if ( destFile.getPath().endsWith( type ) )
+                        else if ( !"pom".equals( type ) && format.equals( type 
) )
                         {
                             if ( !warnedAboutMainProjectArtifact )
                             {


Reply via email to