You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@maven.apache.org by st...@apache.org on 2011/07/01 14:18:04 UTC

svn commit: r1141912 - /maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/exec/ForkedMavenExecutor.java

Author: stephenc
Date: Fri Jul  1 12:18:03 2011
New Revision: 1141912

URL: http://svn.apache.org/viewvc?rev=1141912&view=rev
Log:
[MRELEASE-239] release:perform and release:prepare should accept multi-line goals/preparationGoals configurations

Submitted by Steve Rowe

Modified:
    maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/exec/ForkedMavenExecutor.java

Modified: maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/exec/ForkedMavenExecutor.java
URL: http://svn.apache.org/viewvc/maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/exec/ForkedMavenExecutor.java?rev=1141912&r1=1141911&r2=1141912&view=diff
==============================================================================
--- maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/exec/ForkedMavenExecutor.java (original)
+++ maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/exec/ForkedMavenExecutor.java Fri Jul  1 12:18:03 2011
@@ -86,7 +86,9 @@ public class ForkedMavenExecutor
         if ( goals != null )
         {
             // accept both space and comma, so the old way still work
-            String[] tokens = StringUtils.split( goals, ", " );
+            // also accept line separators, so that goal lists can be spread
+            // across multiple lines in the POM.
+            String[] tokens = StringUtils.split( goals, ", \n\r" );
 
             for ( int i = 0; i < tokens.length; ++i )
             {