You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by ev...@apache.org on 2005/05/22 15:15:16 UTC

svn commit: r171311 - in /maven/components/trunk/maven-plugins/maven-release-plugin/src/main/java/org/apache/maven/plugin/transformer: AbstractPomTransformer.java VersionTransformer.java

Author: evenisse
Date: Sun May 22 06:15:15 2005
New Revision: 171311

URL: http://svn.apache.org/viewcvs?rev=171311&view=rev
Log:
Check if pom is in the correct format.

Modified:
    maven/components/trunk/maven-plugins/maven-release-plugin/src/main/java/org/apache/maven/plugin/transformer/AbstractPomTransformer.java
    maven/components/trunk/maven-plugins/maven-release-plugin/src/main/java/org/apache/maven/plugin/transformer/VersionTransformer.java

Modified: maven/components/trunk/maven-plugins/maven-release-plugin/src/main/java/org/apache/maven/plugin/transformer/AbstractPomTransformer.java
URL: http://svn.apache.org/viewcvs/maven/components/trunk/maven-plugins/maven-release-plugin/src/main/java/org/apache/maven/plugin/transformer/AbstractPomTransformer.java?rev=171311&r1=171310&r2=171311&view=diff
==============================================================================
--- maven/components/trunk/maven-plugins/maven-release-plugin/src/main/java/org/apache/maven/plugin/transformer/AbstractPomTransformer.java (original)
+++ maven/components/trunk/maven-plugins/maven-release-plugin/src/main/java/org/apache/maven/plugin/transformer/AbstractPomTransformer.java Sun May 22 06:15:15 2005
@@ -187,6 +187,11 @@
     public void transformNodes()
         throws Exception
     {
+        if ( getSelectedNodes().isEmpty() )
+        {
+            throw new Exception( "Your pom doesn't respect the standard format." );
+        }
+
         for ( Iterator i = getSelectedNodes().iterator(); i.hasNext(); )
         {
             Object o = i.next();

Modified: maven/components/trunk/maven-plugins/maven-release-plugin/src/main/java/org/apache/maven/plugin/transformer/VersionTransformer.java
URL: http://svn.apache.org/viewcvs/maven/components/trunk/maven-plugins/maven-release-plugin/src/main/java/org/apache/maven/plugin/transformer/VersionTransformer.java?rev=171311&r1=171310&r2=171311&view=diff
==============================================================================
--- maven/components/trunk/maven-plugins/maven-release-plugin/src/main/java/org/apache/maven/plugin/transformer/VersionTransformer.java (original)
+++ maven/components/trunk/maven-plugins/maven-release-plugin/src/main/java/org/apache/maven/plugin/transformer/VersionTransformer.java Sun May 22 06:15:15 2005
@@ -30,8 +30,6 @@
  * @author <a href="mailto:brett@apache.org">Brett Porter</a>
  *
  * @version $Id: VersionTransformer.java 115421 2004-06-01 02:20:18Z dion $
- * @todo check to make sure the basic structure of the document is good. just ran into a bug
- * with the first version of the converter and the root element was <model> not <project>.
  */
 public class VersionTransformer
     extends AbstractPomTransformer



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org