You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by ga...@apache.org on 2007/12/07 06:55:26 UTC

svn commit: r601999 - /geronimo/server/trunk/framework/modules/geronimo-system/src/main/java/org/apache/geronimo/system/plugin/PluginInstallerGBean.java

Author: gawor
Date: Thu Dec  6 21:55:19 2007
New Revision: 601999

URL: http://svn.apache.org/viewvc?rev=601999&view=rev
Log:
should return null so other repos are tried and/or local maven metadata file is checked

Modified:
    geronimo/server/trunk/framework/modules/geronimo-system/src/main/java/org/apache/geronimo/system/plugin/PluginInstallerGBean.java

Modified: geronimo/server/trunk/framework/modules/geronimo-system/src/main/java/org/apache/geronimo/system/plugin/PluginInstallerGBean.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/framework/modules/geronimo-system/src/main/java/org/apache/geronimo/system/plugin/PluginInstallerGBean.java?rev=601999&r1=601998&r2=601999&view=diff
==============================================================================
--- geronimo/server/trunk/framework/modules/geronimo-system/src/main/java/org/apache/geronimo/system/plugin/PluginInstallerGBean.java (original)
+++ geronimo/server/trunk/framework/modules/geronimo-system/src/main/java/org/apache/geronimo/system/plugin/PluginInstallerGBean.java Thu Dec  6 21:55:19 2007
@@ -1444,12 +1444,14 @@
     /**
      * Does the meat of connecting to a URL.  Can be used to just test the existance of
      * something at the specified URL by passing the method 'HEAD'.
+     * 
+     * @return null if cannot connect to the URL. 
      */
     private static InputStream connect(URL url, String username, String password, ResultsFileWriteMonitor monitor, String method) throws IOException, FailedLoginException {
         if (url.getProtocol().equals("file")) {
             File path = new File(url.getPath());
             if (!path.exists()) {
-                throw new IOException("No file at " + path);
+                return null;
             }
             if (path.isDirectory()) {
                 //todo this is awfully redundantly copying over and over again