svn commit: r375514 - /maven/sandbox/repoclean/src/main/java/org/apache/maven/tools/repoclean/phase/RewritePhase.java

[carlos]

Author: carlos
Date: Mon Feb  6 22:33:51 2006
New Revision: 375514

URL: http://svn.apache.org/viewcvs?rev=375514&view=rev
Log:
Prevent making empty poms and overwritting existing ones

Modified:
    
maven/sandbox/repoclean/src/main/java/org/apache/maven/tools/repoclean/phase/RewritePhase.java

Modified: 
maven/sandbox/repoclean/src/main/java/org/apache/maven/tools/repoclean/phase/RewritePhase.java
URL: 
http://svn.apache.org/viewcvs/maven/sandbox/repoclean/src/main/java/org/apache/maven/tools/repoclean/phase/RewritePhase.java?rev=375514&r1=375513&r2=375514&view=diff
==============================================================================
--- 
maven/sandbox/repoclean/src/main/java/org/apache/maven/tools/repoclean/phase/RewritePhase.java
 (original)
+++ 
maven/sandbox/repoclean/src/main/java/org/apache/maven/tools/repoclean/phase/RewritePhase.java
 Mon Feb  6 22:33:51 2006
@@ -256,19 +256,31 @@
 
         String pomContents = null;
 
-        boolean shouldRewritePom = false;
+        boolean shouldRewritePom = true;
 
-        if ( sourcePom.exists() && !targetPom.exists() )
+        if ( targetPom.exists() )
+        {
+            // we have a target pom for this artifact already, and we'll only 
be making up a new pom.
+            // let's leave the existing one alone.
+            artifactReporter.warn( "Not overwritting " + targetPom.getPath() );
+            return;
+        }
+
+        if ( !sourcePom.exists() )
+        {
+            // don't make empty poms
+            return;
+        }
+
+        if ( sourcePom.exists() )
         {
             pomContents = readPomContents( sourcePom );
 
             if ( pomContents.indexOf( "modelVersion" ) > -1 )
             {
+                shouldRewritePom = false;
+
                 copyMetadata( sourcePom, targetPom, transaction, 
artifactReporter, reportOnly );
-            }
-            else
-            {
-                shouldRewritePom = true;
             }
         }