You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by ha...@apache.org on 2007/03/10 12:06:29 UTC

svn commit: r516680 - /maven/shared/trunk/maven-verifier/src/main/java/org/apache/maven/it/Verifier.java

Author: handyande
Date: Sat Mar 10 03:06:26 2007
New Revision: 516680

URL: http://svn.apache.org/viewvc?view=rev&rev=516680
Log:
Don't fail silently (our sout / serr are being consumed) if the maven script has lines above 'Maven version' the jdk or script interpreter could output a line, which we don't care about.

Modified:
    maven/shared/trunk/maven-verifier/src/main/java/org/apache/maven/it/Verifier.java

Modified: maven/shared/trunk/maven-verifier/src/main/java/org/apache/maven/it/Verifier.java
URL: http://svn.apache.org/viewvc/maven/shared/trunk/maven-verifier/src/main/java/org/apache/maven/it/Verifier.java?view=diff&rev=516680&r1=516679&r2=516680
==============================================================================
--- maven/shared/trunk/maven-verifier/src/main/java/org/apache/maven/it/Verifier.java (original)
+++ maven/shared/trunk/maven-verifier/src/main/java/org/apache/maven/it/Verifier.java Sat Mar 10 03:06:26 2007
@@ -973,13 +973,23 @@
 
         List l = loadFile( log, false );
 
-        String first = (String) l.get( 0 );
-        if ( !first.startsWith( "Maven version: " ) )
+        Iterator lines = l.iterator();
+        String version = (String) lines.next();
+        boolean matched = false;
+
+        while ( !version.startsWith( "Maven version: " ) && lines.hasNext() )
+        {
+            version = (String) lines.next();
+            matched = true;
+            break;
+        }
+
+        if ( !matched )
         {
-            throw new VerificationException( "Illegal maven output: expecting 'Maven version: ' but got " + first );
+            throw new VerificationException( "Illegal maven output: expecting 'Maven version: ' but got " + l.get( 0 ) );
         }
 
-        return first.substring( "Maven version: ".length() ).trim();
+        return version.substring( "Maven version: ".length() ).trim();
     }