DPML Notify

[mcconnell at BerliOS <mcconnell AT mail.berlios.de>]

Author: mcconnell
Date: 2006-01-12 08:41:18 +0100 (Thu, 12 Jan 2006)
New Revision: 909

Modified:
   trunk/main/depot/core/console/src/main/net/dpml/depot/Main.java
   trunk/main/depot/library/console/etc/bin/build.lap
   
trunk/main/depot/library/console/src/main/net/dpml/library/console/BuilderPlugin.java
   
trunk/main/metro/station/console/src/main/net/dpml/station/console/StationPlugin.java
   
trunk/main/metro/station/server/src/main/net/dpml/station/server/RemoteApplication.java
   trunk/main/transit/console/etc/bin/transit.lap
   trunk/main/transit/core/src/main/net/dpml/transit/DefaultCacheHandler.java
   trunk/main/transit/core/src/main/net/dpml/transit/NetworkLoader.java
   trunk/main/transit/core/src/main/net/dpml/transit/StandardLoader.java
   trunk/main/transit/core/src/main/net/dpml/transit/SystemClassLoader.java
   trunk/main/transit/core/src/main/net/dpml/transit/monitor/CacheMonitor.java
   
trunk/main/transit/core/src/main/net/dpml/transit/monitor/CacheMonitorAdapter.java
   
trunk/main/transit/core/src/main/net/dpml/transit/monitor/CacheMonitorRouter.java
   
trunk/main/transit/core/src/main/net/dpml/transit/monitor/LoggingAdapter.java
   
trunk/main/transit/core/src/main/net/dpml/transit/monitor/NetworkMonitorAdapter.java
   
trunk/main/transit/core/src/main/net/dpml/transit/monitor/RepositoryMonitor.java
   
trunk/main/transit/core/src/main/net/dpml/transit/monitor/RepositoryMonitorAdapter.java
   
trunk/main/transit/core/src/main/net/dpml/transit/monitor/RepositoryMonitorRouter.java
Log:
resolve a bug in the resource monitor that was tripping artifact resolution 
when evaluating if we are running in an ant context - also updated some of 
the monitors to improve error reporting in transit 

Modified: trunk/main/depot/core/console/src/main/net/dpml/depot/Main.java
===================================================================
--- trunk/main/depot/core/console/src/main/net/dpml/depot/Main.java     
2006-01-12 03:20:37 UTC (rev 908)
+++ trunk/main/depot/core/console/src/main/net/dpml/depot/Main.java     
2006-01-12 07:41:18 UTC (rev 909)
@@ -84,7 +84,7 @@
         String[] args = arguments;
         
         //
-        // check for debug mode and tools flag
+        // check for debug mode
         //
         
         if( CLIHelper.isOptionPresent( args, "-debug" ) )
@@ -134,7 +134,7 @@
         String[] args = processSystemProperties( arguments );
         String name = "build";
         String spec = "@DEPOT-BUILDER-URI@";
-        handlePlugin( name, spec, args, false, true );
+        handlePlugin( name, spec, args, false );
     }
 
     private void handleMetro( String[] arguments )
@@ -142,7 +142,7 @@
         String[] args = processSystemProperties( arguments );
         String name = "metro";
         String spec = "@DEPOT-EXEC-URI@";
-        handlePlugin( name, spec, args );
+        handlePlugin( name, spec, args, true );
     }
 
     private void handleTransit( String[] arguments )
@@ -162,7 +162,7 @@
             String[] args = CLIHelper.consolidate( arguments, "-server" );
             args = processSystemProperties( args );
             String spec = "@DEPOT-STATION-SERVER-URI@";
-            handlePlugin( name, spec, args );
+            handlePlugin( name, spec, args, true );
         }
         else
         {
@@ -171,24 +171,10 @@
         }
     }
 
-    private void handlePlugin( String name, String spec, String[] args )
-    {
-        handlePlugin( name, spec, args, true );
-    }
-
     private void handlePlugin( String name, String spec, String[] args, 
boolean wait )
     {
-        handlePlugin( name, spec, args, wait, false );
-    }
-    
-    private void handlePlugin( String name, String spec, String[] args, 
boolean wait, boolean tools )
-    {
-        if( tools )
-        {
-            System.setProperty( "dpml.transit.include.tools", "true" );
-        }
         System.setSecurityManager( new RMISecurityManager() );
-        boolean waitForCompletion = deployHandler( name, spec, args, wait, 
tools );
+        boolean waitForCompletion = deployHandler( name, spec, args, wait );
         if( !waitForCompletion )
         {
             System.exit( 0 );
@@ -196,7 +182,7 @@
     }
 
     private boolean deployHandler( 
-      String command, String path, String[] args, boolean waitFor, boolean 
tools )
+      String command, String path, String[] args, boolean waitFor )
     {
         Logger logger = getLogger().getChildLogger( command );
         if( m_debug )

Modified: trunk/main/depot/library/console/etc/bin/build.lap
===================================================================
--- trunk/main/depot/library/console/etc/bin/build.lap  2006-01-12 03:20:37 
UTC (rev 908)
+++ trunk/main/depot/library/console/etc/bin/build.lap  2006-01-12 07:41:18 
UTC (rev 909)
@@ -6,10 +6,10 @@
 -Djava.system.class.loader=net.dpml.transit.SystemClassLoader
 -Djava.security.policy=${env.DPML_HOME}\share\bin\security.policy
 -Ddpml.depot.application=dpml.builder
+-Djava.protocol.handler.pkgs=net.dpml.transit
 janel.main.class=net.dpml.depot.Main
 janel.min.java.version=1.4.0
-#janel.max.java.version=1.4.2_09
-janel.max.java.version=1.5
+#janel.max.java.version=1.5
 janel.classpath.jars.dir=${env.DPML_HOME}\share\lib
 janel.sysprop.process.id=true
 janel.java.bundle=require_sdk

Modified: 
trunk/main/depot/library/console/src/main/net/dpml/library/console/BuilderPlugin.java
===================================================================
--- 
trunk/main/depot/library/console/src/main/net/dpml/library/console/BuilderPlugin.java
       2006-01-12 03:20:37 UTC (rev 908)
+++ 
trunk/main/depot/library/console/src/main/net/dpml/library/console/BuilderPlugin.java
       2006-01-12 07:41:18 UTC (rev 909)
@@ -67,7 +67,6 @@
     private final DefaultLibrary m_library;
     
     private boolean m_verbose;
-    private Builder m_builder;
     
     // 
------------------------------------------------------------------------
     // constructors
@@ -113,17 +112,6 @@
                     }
                 }
                 
-                //
-                // setup the default project builder
-                //
-                
-                URI uri = (URI) line.getValue( BUILDER_URI_OPTION, 
ANT_BUILDER_URI );
-                m_builder = createBuilder( uri );
-                
-                //
-                // resolve and build the sorted set of projects
-                //
-                
                 if( line.hasOption( LIST_OPTION ) )
                 {
                     Resource[] resources = getTargetSelection( line );
@@ -136,7 +124,6 @@
                     {
                         list( resources );
                     }
-                    
                 }
                 else
                 {
@@ -267,8 +254,11 @@
     * @param line the commandline
     * @param resources the sorted sequence of prouject to build
     */
-    private void process( CommandLine line, Resource[] resources )
+    private void process( CommandLine line, Resource[] resources ) throws 
Exception
     {
+        URI uri = (URI) line.getValue( BUILDER_URI_OPTION, ANT_BUILDER_URI );
+        Builder builder = createBuilder( uri );
+        
         if( resources.length > 1 )
         {
             StringBuffer buffer = 
@@ -281,12 +271,13 @@
             buffer.append( "\n" );
             getLogger().info( buffer.toString() );
         }
+        
         List list = line.getValues( TARGETS );
         String[] targets = (String[]) list.toArray( new String[ list.size() 
] );
         for( int i=0; i<resources.length; i++ )
         {
             Resource resource = resources[i];
-            boolean status = m_builder.build( resource, targets );
+            boolean status = builder.build( resource, targets );
             if( !status )
             {
                 return;

Modified: 
trunk/main/metro/station/console/src/main/net/dpml/station/console/StationPlugin.java
===================================================================
--- 
trunk/main/metro/station/console/src/main/net/dpml/station/console/StationPlugin.java
       2006-01-12 03:20:37 UTC (rev 908)
+++ 
trunk/main/metro/station/console/src/main/net/dpml/station/console/StationPlugin.java
       2006-01-12 07:41:18 UTC (rev 909)
@@ -441,6 +441,7 @@
         list.add( "" + port );
         
         list.add( 
"-Ddpml.logging.config=local:properties:dpml/station/logging" );
+        list.add( "-Ddpml.subprocess=true" );
         Set propertyValue = line.getProperties();
         Iterator iterator = propertyValue.iterator();
         while( iterator.hasNext() )

Modified: 
trunk/main/metro/station/server/src/main/net/dpml/station/server/RemoteApplication.java
===================================================================
--- 
trunk/main/metro/station/server/src/main/net/dpml/station/server/RemoteApplication.java
     2006-01-12 03:20:37 UTC (rev 908)
+++ 
trunk/main/metro/station/server/src/main/net/dpml/station/server/RemoteApplication.java
     2006-01-12 07:41:18 UTC (rev 909)
@@ -335,6 +335,9 @@
             properties.setProperty( 
               "java.util.logging.config.class", 
               "net.dpml.depot.DepotLoggingConfiguration" );
+            properties.setProperty(
+              "dpml.subprocess",
+              "true" );
         }
         Enumeration names = properties.propertyNames();
         while( names.hasMoreElements() )

Modified: trunk/main/transit/console/etc/bin/transit.lap
===================================================================
--- trunk/main/transit/console/etc/bin/transit.lap      2006-01-12 03:20:37 
UTC (rev 908)
+++ trunk/main/transit/console/etc/bin/transit.lap      2006-01-12 07:41:18 
UTC (rev 909)
@@ -9,6 +9,6 @@
 -Djava.rmi.server.RMIClassLoaderSpi=net.dpml.depot.DepotRMIClassLoaderSpi
 janel.main.class=net.dpml.depot.Main
 janel.min.java.version=1.4
-janel.max.java.version=1.5
+#janel.max.java.version=1.5
 janel.classpath.jars.dir=${env.DPML_HOME}\share\lib
 janel.sysprop.process.id=true

Modified: 
trunk/main/transit/core/src/main/net/dpml/transit/DefaultCacheHandler.java
===================================================================
--- 
trunk/main/transit/core/src/main/net/dpml/transit/DefaultCacheHandler.java  
2006-01-12 03:20:37 UTC (rev 908)
+++ 
trunk/main/transit/core/src/main/net/dpml/transit/DefaultCacheHandler.java  
2006-01-12 07:41:18 UTC (rev 909)
@@ -568,13 +568,14 @@
         {
             return false;
         }
+        
+        CacheMonitorRouter monitor = 
Transit.getInstance().getCacheMonitorRouter();
+        
         File parentDir = destination.getParentFile();
         File tempFile = File.createTempFile( "~dpml", ".tmp", parentDir );
         tempFile.deleteOnExit(); // safety harness in case we abort 
abnormally
-                                 // like a Ctrl-C.
         FileOutputStream tempOut = new FileOutputStream( tempFile );
-
-        CacheMonitorRouter monitor = 
Transit.getInstance().getCacheMonitorRouter();
+        
         try
         {
             Date lastModified = host.download( artifact, tempOut );
@@ -584,7 +585,7 @@
             destination.setLastModified( lastModified.getTime() );
             return true;
         }
-        catch( IOException e )
+        catch( Throwable e )
         {
             tempFile.delete();
             if( monitor != null )

Modified: trunk/main/transit/core/src/main/net/dpml/transit/NetworkLoader.java
===================================================================
--- trunk/main/transit/core/src/main/net/dpml/transit/NetworkLoader.java      
  2006-01-12 03:20:37 UTC (rev 908)
+++ trunk/main/transit/core/src/main/net/dpml/transit/NetworkLoader.java      
  2006-01-12 07:41:18 UTC (rev 909)
@@ -80,6 +80,7 @@
                 throw new IOException( "Not authorized." );
             }
         }
+        
         InputStream in = connection.getInputStream();
         int expected = connection.getContentLength();
         StreamUtils.copyStream( m_monitor, url, expected, in, destination, 
true );

Modified: 
trunk/main/transit/core/src/main/net/dpml/transit/StandardLoader.java
===================================================================
--- trunk/main/transit/core/src/main/net/dpml/transit/StandardLoader.java     
  2006-01-12 03:20:37 UTC (rev 908)
+++ trunk/main/transit/core/src/main/net/dpml/transit/StandardLoader.java     
  2006-01-12 07:41:18 UTC (rev 909)
@@ -597,7 +597,7 @@
 
         URI[] systemArtifacts = descriptor.getDependencies( Category.SYSTEM 
);
         URL[] sysUrls = getURLs( systemArtifacts );
-        updateSystemClassLoader( sysUrls );
+        updateSystemClassLoader( plugin, sysUrls );
 
         URI[] apiArtifacts = descriptor.getDependencies( Category.PUBLIC );
         URL[] apis = getURLs( apiArtifacts  );
@@ -614,21 +614,25 @@
         return classloader;
     }
 
-    private void updateSystemClassLoader( URL[] urls ) throws 
TransitException
+    private void updateSystemClassLoader( URI plugin, URL[] urls ) throws 
TransitException
     {
         ClassLoader parent = ClassLoader.getSystemClassLoader();
-        if( parent instanceof SystemClassLoader )
+        synchronized( parent )
         {
-            SystemClassLoader loader = (SystemClassLoader) parent;
-            loader.addDelegates( urls );
+            if( parent instanceof SystemClassLoader )
+            {
+                SystemClassLoader loader = (SystemClassLoader) parent;
+                loader.addDelegates( urls );
+                getMonitor().systemExpanded( plugin, urls );
+            }
+            else
+            {
+                final String message =
+                  "Cannot load system artifacts into a foreign classloader.";
+                getMonitor().sequenceInfo( message );
+                //throw new TransitException( error );
+            }
         }
-        else
-        {
-            final String message =
-              "Cannot load system artifacts into a foreign classloader.";
-            getMonitor().sequenceInfo( message );
-            //throw new TransitException( error );
-        }
     }
 
    /**
@@ -658,10 +662,6 @@
     */
     private ClassLoader buildClassLoader( URI plugin, Category category, 
ClassLoader parent, URL[] urls )
     {
-        //ClassLoader loader = new StandardClassLoader( category, urls, 
parent );
-        //getMonitor().classloaderConstructed( category.toString(), loader );
-        //return loader;
-
         if( 0 == urls.length )
         {
             return parent;

Modified: 
trunk/main/transit/core/src/main/net/dpml/transit/SystemClassLoader.java
===================================================================
--- trunk/main/transit/core/src/main/net/dpml/transit/SystemClassLoader.java  
  2006-01-12 03:20:37 UTC (rev 908)
+++ trunk/main/transit/core/src/main/net/dpml/transit/SystemClassLoader.java  
  2006-01-12 07:41:18 UTC (rev 909)
@@ -76,7 +76,17 @@
             URL url = urls[i];
             if( !list.contains( url ) )
             {
-                addURL( url );
+                try
+                {
+                    url.getContent();
+                    addURL( url );
+                }
+                catch( Exception e )
+                {
+                    final String error = 
+                      "Failed to resolve url " + url;
+                    System.err.println( error );
+                }
             }
         }
     }

Modified: 
trunk/main/transit/core/src/main/net/dpml/transit/monitor/CacheMonitor.java
===================================================================
--- 
trunk/main/transit/core/src/main/net/dpml/transit/monitor/CacheMonitor.java 
2006-01-12 03:20:37 UTC (rev 908)
+++ 
trunk/main/transit/core/src/main/net/dpml/transit/monitor/CacheMonitor.java 
2006-01-12 07:41:18 UTC (rev 909)
@@ -70,7 +70,7 @@
     * @param artifact the requested artifact
     * @param cause the exception causing the failure
     */
-    void failedDownloadFromHost( String host, Artifact artifact, Exception 
cause );
+    void failedDownloadFromHost( String host, Artifact artifact, Throwable 
cause );
 
    /**
     * Notify the monitor of a failed download attempt.

Modified: 
trunk/main/transit/core/src/main/net/dpml/transit/monitor/CacheMonitorAdapter.java
===================================================================
--- 
trunk/main/transit/core/src/main/net/dpml/transit/monitor/CacheMonitorAdapter.java
  2006-01-12 03:20:37 UTC (rev 908)
+++ 
trunk/main/transit/core/src/main/net/dpml/transit/monitor/CacheMonitorAdapter.java
  2006-01-12 07:41:18 UTC (rev 909)
@@ -97,7 +97,7 @@
     * @param artifact the requested artifact
     * @param e the exception causing the failure
     */
-    public void failedDownloadFromHost( String host, Artifact artifact, 
Exception e )
+    public void failedDownloadFromHost( String host, Artifact artifact, 
Throwable e )
     {
         if( getAdapter().isDebugEnabled() )
         {

Modified: 
trunk/main/transit/core/src/main/net/dpml/transit/monitor/CacheMonitorRouter.java
===================================================================
--- 
trunk/main/transit/core/src/main/net/dpml/transit/monitor/CacheMonitorRouter.java
   2006-01-12 03:20:37 UTC (rev 908)
+++ 
trunk/main/transit/core/src/main/net/dpml/transit/monitor/CacheMonitorRouter.java
   2006-01-12 07:41:18 UTC (rev 909)
@@ -103,7 +103,7 @@
     * @param artifact the requested artifact
     * @param cause the cause of the failure
     */ 
-    public void failedDownloadFromHost( String host, Artifact artifact, 
Exception cause )
+    public void failedDownloadFromHost( String host, Artifact artifact, 
Throwable cause )
     {
         Monitor[] monitors = getMonitors();
         for( int i=0; i < monitors.length; i++ )

Modified: 
trunk/main/transit/core/src/main/net/dpml/transit/monitor/LoggingAdapter.java
===================================================================
--- 
trunk/main/transit/core/src/main/net/dpml/transit/monitor/LoggingAdapter.java 
      2006-01-12 03:20:37 UTC (rev 908)
+++ 
trunk/main/transit/core/src/main/net/dpml/transit/monitor/LoggingAdapter.java 
      2006-01-12 07:41:18 UTC (rev 909)
@@ -244,25 +244,28 @@
      */
     public void notify( URL resource, int total, int count )
     {
-        if( isAnt() || ( null != System.getProperty( "dpml.spawn" ) ) )
+        String path = resource.toString();
+        if( path.startsWith( "file:" ) )
         {
+            return;
+        }
+        if( isAnt() || ( "true".equals( System.getProperty( 
"dpml.subprocess" ) ) ) )
+        {
             if( count == 0 )
             {
                 info( "downloading [" + resource + "] (" + 
getFranctionalValue( total ) + ")" );
             }
             return;
         }
-
+        
         if( isInfoEnabled() )
         {
             String max = getFranctionalValue( total );
-
             String value = getFranctionalValue( count );
             int pad = max.length() - value.length();
             String level = getLogHeader();
             String process = getProcessHeader();
             StringBuffer buffer = new StringBuffer( process + level );
-            String path = resource.toString();
             String name = path.substring( path.lastIndexOf( '/' ) + 1 );
             buffer.append( "(" + m_logger.getName() + "): " );
             buffer.append( "retrieving: " + name + " " );
@@ -351,6 +354,8 @@
     */
     private boolean isAnt()
     {
+        return null != System.getProperty( "ant.home" );
+        /*
         try
         {
             ClassLoader.getSystemClassLoader().loadClass( 
"org.apache.tools.ant.launch.AntMain" );
@@ -360,6 +365,7 @@
         {
             return false;
         }
+        */
     }
 
    /**

Modified: 
trunk/main/transit/core/src/main/net/dpml/transit/monitor/NetworkMonitorAdapter.java
===================================================================
--- 
trunk/main/transit/core/src/main/net/dpml/transit/monitor/NetworkMonitorAdapter.java
        2006-01-12 03:20:37 UTC (rev 908)
+++ 
trunk/main/transit/core/src/main/net/dpml/transit/monitor/NetworkMonitorAdapter.java
        2006-01-12 07:41:18 UTC (rev 909)
@@ -46,13 +46,13 @@
     // NetworkMonitor
     // 
------------------------------------------------------------------------
 
-    /**
-     * Handle the notification of an update in the download status.
-     *
-     * @param resource the name of the remote resource being downloaded.
-     * @param expected the expected number of bytes to be downloaded.
-     * @param count the number of bytes downloaded.
-     */
+   /**
+    * Handle the notification of an update in the download status.
+    *
+    * @param resource the name of the remote resource being downloaded.
+    * @param expected the expected number of bytes to be downloaded.
+    * @param count the number of bytes downloaded.
+    */
     public void notifyUpdate( URL resource, int expected, int count )
     {
         getAdapter().notify( resource, expected, count );
@@ -64,14 +64,10 @@
      */
     public void notifyCompletion( URL resource )
     {
-        if( resource.toString().startsWith( "file:" ) )
+        if( getAdapter().isDebugEnabled() )
         {
-            getAdapter().debug( "located: " + resource );
+            getAdapter().debug( "downloaded: " + resource );
         }
-        else if( getAdapter().isInfoEnabled() )
-        {
-            getAdapter().info( "downloaded: " + resource );
-        }
     }
 }
 

Modified: 
trunk/main/transit/core/src/main/net/dpml/transit/monitor/RepositoryMonitor.java
===================================================================
--- 
trunk/main/transit/core/src/main/net/dpml/transit/monitor/RepositoryMonitor.java
    2006-01-12 03:20:37 UTC (rev 908)
+++ 
trunk/main/transit/core/src/main/net/dpml/transit/monitor/RepositoryMonitor.java
    2006-01-12 07:41:18 UTC (rev 909)
@@ -19,6 +19,7 @@
 package net.dpml.transit.monitor;
 
 import java.net.URI;
+import java.net.URL;
 
 import java.lang.reflect.Constructor;
  
@@ -76,4 +77,11 @@
     * @param classloader the new classloader 
     */
     void classloaderConstructed( String type, ClassLoader classloader );
+    
+   /**
+    * Handle notification of system classloader expansion.
+    * @param plugin the uri of the plugin requesting system classloader 
expansion
+    * @param urls the array of urls added to the system classloader
+    */
+    void systemExpanded( URI plugin, URL[] urls );
 }
\ No newline at end of file

Modified: 
trunk/main/transit/core/src/main/net/dpml/transit/monitor/RepositoryMonitorAdapter.java
===================================================================
--- 
trunk/main/transit/core/src/main/net/dpml/transit/monitor/RepositoryMonitorAdapter.java
     2006-01-12 03:20:37 UTC (rev 908)
+++ 
trunk/main/transit/core/src/main/net/dpml/transit/monitor/RepositoryMonitorAdapter.java
     2006-01-12 07:41:18 UTC (rev 909)
@@ -183,10 +183,31 @@
                 for( int i=0; i < urls.length; i++ )
                 {
                     URL url = urls[i];
-                    buffer.append( "\n(" + i + ") " + url.toString() );
+                    buffer.append( "\n  [" + i + "] \t" + url.toString() );
                 }
             }
             getAdapter().debug( buffer.toString() );
         }
     }
+    
+   /**
+    * Handle notification of system classloader expansion.
+    * @param plugin the uri of the plugin requesting system classloader 
expansion
+    * @param urls the array of urls added to the system classloader
+    */
+    public void systemExpanded( URI plugin, URL[] urls )
+    {
+        if( getAdapter().isDebugEnabled() )
+        {
+            StringBuffer buffer = new StringBuffer();
+            buffer.append( "system classloader expansion" );
+            buffer.append( "\n  plugin: " + plugin );
+            for( int i=0; i<urls.length; i++ )
+            {
+                int n = i+1;
+                buffer.append( "\n  [" + n + "] \t" + urls[i] );
+            }
+            getAdapter().debug( buffer.toString() );
+        }
+    }
 }

Modified: 
trunk/main/transit/core/src/main/net/dpml/transit/monitor/RepositoryMonitorRouter.java
===================================================================
--- 
trunk/main/transit/core/src/main/net/dpml/transit/monitor/RepositoryMonitorRouter.java
      2006-01-12 03:20:37 UTC (rev 908)
+++ 
trunk/main/transit/core/src/main/net/dpml/transit/monitor/RepositoryMonitorRouter.java
      2006-01-12 07:41:18 UTC (rev 909)
@@ -19,6 +19,7 @@
 package net.dpml.transit.monitor;
 
 import java.net.URI;
+import java.net.URL;
 import java.lang.reflect.Constructor;
  
 /**
@@ -139,6 +140,21 @@
     }
 
    /**
+    * Handle notification of system classloader expansion.
+    * @param plugin the uri of the plugin requesting system classloader 
expansion
+    * @param urls the array of urls added to the system classloader
+    */
+    public void systemExpanded( URI plugin, URL[] urls )
+    {
+        Monitor[] monitors = getMonitors();
+        for( int i=0; i < monitors.length; i++ )
+        {
+            RepositoryMonitor monitor = (RepositoryMonitor) monitors[i];
+            monitor.systemExpanded( plugin, urls );
+        }
+    }
+
+   /**
     * Add a monitor to the set of monitors managed by this router.
     * @param monitor the monitor to add
     * @exception IllegalArgumentException if the supplied monitor does not