You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by dj...@apache.org on 2010/01/26 02:08:12 UTC

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

Author: djencks
Date: Tue Jan 26 01:08:12 2010
New Revision: 903034

URL: http://svn.apache.org/viewvc?rev=903034&view=rev
Log:
provide more info on file copy error

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

Modified: geronimo/server/trunk/framework/modules/geronimo-plugin/src/main/java/org/apache/geronimo/system/plugin/PluginInstallerGBean.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/framework/modules/geronimo-plugin/src/main/java/org/apache/geronimo/system/plugin/PluginInstallerGBean.java?rev=903034&r1=903033&r2=903034&view=diff
==============================================================================
--- geronimo/server/trunk/framework/modules/geronimo-plugin/src/main/java/org/apache/geronimo/system/plugin/PluginInstallerGBean.java (original)
+++ geronimo/server/trunk/framework/modules/geronimo-plugin/src/main/java/org/apache/geronimo/system/plugin/PluginInstallerGBean.java Tue Jan 26 01:08:12 2010
@@ -1438,7 +1438,13 @@
                 continue;
             }
             InputStream in = url.openStream();
-            copyFile(in, new FileOutputStream(target));
+            try {
+                copyFile(in, new FileOutputStream(target));
+            } catch (IOException e) {
+                throw new IOException("Could not copy " + url + " in artifact " + configID, e);
+            } catch (NullPointerException e) {
+                throw new IOException("Could not copy " + url + " in artifact " + configID, e);
+            }
         }
     }