geronimo-scm mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ga...@apache.org
Subject svn commit: r654135 - /geronimo/server/branches/2.1/buildsupport/geronimo-maven-plugin/src/main/java/org/apache/geronimo/mavenplugins/geronimo/module/DeployModuleMojo.java
Date Wed, 07 May 2008 15:01:09 GMT
Author: gawor
Date: Wed May  7 08:01:08 2008
New Revision: 654135

URL: http://svn.apache.org/viewvc?rev=654135&view=rev
Log:
nicer logging when no plan is specified (GERONIMO-3848)

Modified:
    geronimo/server/branches/2.1/buildsupport/geronimo-maven-plugin/src/main/java/org/apache/geronimo/mavenplugins/geronimo/module/DeployModuleMojo.java

Modified: geronimo/server/branches/2.1/buildsupport/geronimo-maven-plugin/src/main/java/org/apache/geronimo/mavenplugins/geronimo/module/DeployModuleMojo.java
URL: http://svn.apache.org/viewvc/geronimo/server/branches/2.1/buildsupport/geronimo-maven-plugin/src/main/java/org/apache/geronimo/mavenplugins/geronimo/module/DeployModuleMojo.java?rev=654135&r1=654134&r2=654135&view=diff
==============================================================================
--- geronimo/server/branches/2.1/buildsupport/geronimo-maven-plugin/src/main/java/org/apache/geronimo/mavenplugins/geronimo/module/DeployModuleMojo.java
(original)
+++ geronimo/server/branches/2.1/buildsupport/geronimo-maven-plugin/src/main/java/org/apache/geronimo/mavenplugins/geronimo/module/DeployModuleMojo.java
Wed May  7 08:01:08 2008
@@ -78,7 +78,9 @@
 
         if (moduleArchive != null || modulePlan != null) {
             log.info("Using non-artifact based module archive: " + moduleArchive);
-            log.info("Using non-artifact based plan: " + modulePlan);
+            if (modulePlan != null) {
+                log.info("Using non-artifact based plan: " + modulePlan);
+            }
 
             TargetModuleID[] ids = distribute(moduleArchive, modulePlan);
             completed.add(ids);
@@ -143,7 +145,13 @@
     }
 
     private TargetModuleID[] distribute(final File file, final File plan) throws Exception
{
-        log.info("Distributing module artifact: " + file + " with plan " + plan);
+        if (log.isInfoEnabled()) {
+            String msg = "Distributing module artifact: " + file;
+            if (plan != null) {
+                msg += " with plan " + plan;
+            }
+            log.info(msg);
+        }
 
         DeploymentManager manager = getDeploymentManager();
         Target[] targets = manager.getTargets();



Mime
View raw message