geronimo-scm mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From lin...@apache.org
Subject svn commit: r702897 - /geronimo/server/trunk/framework/modules/geronimo-plugin/src/main/java/org/apache/geronimo/system/plugin/PluginInstallerGBean.java
Date Wed, 08 Oct 2008 14:43:27 GMT
Author: linsun
Date: Wed Oct  8 07:43:27 2008
New Revision: 702897

URL: http://svn.apache.org/viewvc?rev=702897&view=rev
Log:
inform users that the .car module is already installed, introduced by changes of GERONIMO-4318

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=702897&r1=702896&r2=702897&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
Wed Oct  8 07:43:27 2008
@@ -932,7 +932,7 @@
             // 2. Validate that we can install this
             if (!validatePlugin(data)) {
                 //already installed
-                return;
+                throw new MissingDependencyException("Already installed", toArtifact(data.getPluginArtifact().get(0).getModuleId()),
(Stack<Artifact>)null);
             }
 
             verifyPrerequisites(data);



Mime
View raw message