You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by ol...@apache.org on 2012/10/10 17:16:52 UTC

svn commit: r1396641 - /maven/sandbox/trunk/plugins/maven-patch-tracker-plugin/src/main/java/org/apache/maven/plugins/patchtracker/PostPatchMojo.java

Author: olamy
Date: Wed Oct 10 15:16:51 2012
New Revision: 1396641

URL: http://svn.apache.org/viewvc?rev=1396641&view=rev
Log:
fix message when no id created by the remote system

Modified:
    maven/sandbox/trunk/plugins/maven-patch-tracker-plugin/src/main/java/org/apache/maven/plugins/patchtracker/PostPatchMojo.java

Modified: maven/sandbox/trunk/plugins/maven-patch-tracker-plugin/src/main/java/org/apache/maven/plugins/patchtracker/PostPatchMojo.java
URL: http://svn.apache.org/viewvc/maven/sandbox/trunk/plugins/maven-patch-tracker-plugin/src/main/java/org/apache/maven/plugins/patchtracker/PostPatchMojo.java?rev=1396641&r1=1396640&r2=1396641&view=diff
==============================================================================
--- maven/sandbox/trunk/plugins/maven-patch-tracker-plugin/src/main/java/org/apache/maven/plugins/patchtracker/PostPatchMojo.java (original)
+++ maven/sandbox/trunk/plugins/maven-patch-tracker-plugin/src/main/java/org/apache/maven/plugins/patchtracker/PostPatchMojo.java Wed Oct 10 15:16:51 2012
@@ -62,7 +62,12 @@ public class PostPatchMojo
             getLog().debug( patchTrackerRequest.toString() );
 
             PatchTrackerResult result = patchTracker.createPatch( patchTrackerRequest, getLog() );
-            getLog().info( "issue created with id:" + result.getPatchId() + ", url:" + result.getPatchUrl() );
+            StringBuilder msg = new StringBuilder( "Patch posted to patch tracker" );
+            if ( result.getPatchId() != null )
+            {
+                msg.append( ". Review created with id:" + result.getPatchId() + ", url:" + result.getPatchUrl() );
+            }
+            getLog().info( msg.toString() );
         }
         catch ( ComponentLookupException e )
         {