You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by jd...@apache.org on 2006/08/07 21:41:26 UTC

svn commit: r429449 - /geronimo/trunk/modules/system/src/java/org/apache/geronimo/system/repository/AbstractRepository.java

Author: jdillon
Date: Mon Aug  7 12:41:26 2006
New Revision: 429449

URL: http://svn.apache.org/viewvc?rev=429449&view=rev
Log:
(GERONIMO-2131) Use logging instead of evil System.out's

Modified:
    geronimo/trunk/modules/system/src/java/org/apache/geronimo/system/repository/AbstractRepository.java

Modified: geronimo/trunk/modules/system/src/java/org/apache/geronimo/system/repository/AbstractRepository.java
URL: http://svn.apache.org/viewvc/geronimo/trunk/modules/system/src/java/org/apache/geronimo/system/repository/AbstractRepository.java?rev=429449&r1=429448&r2=429449&view=diff
==============================================================================
--- geronimo/trunk/modules/system/src/java/org/apache/geronimo/system/repository/AbstractRepository.java (original)
+++ geronimo/trunk/modules/system/src/java/org/apache/geronimo/system/repository/AbstractRepository.java Mon Aug  7 12:41:26 2006
@@ -233,16 +233,8 @@
         if (typeHandler == null) typeHandler = DEFAULT_TYPE_HANDLER;
         typeHandler.install(source, size, destination, monitor, location);
         
-        //
-        // FIXME: This should not be here... if you need this intel then add logging
-        //
-        
         if (destination.getType().equalsIgnoreCase("car")) {
-            System.out.println("############################################################");
-            System.out.println("# Installed configuration");
-            System.out.println("#   id = " + destination);
-            System.out.println("#   location = " + location);
-            System.out.println("############################################################");
+            log.info("Installed module configuration; id=" + destination + "; location=" + location);
         }
     }
 }