geronimo-scm mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ga...@apache.org
Subject svn commit: r783821 - /geronimo/server/trunk/plugins/j2ee/geronimo-j2ee-builder/src/main/java/org/apache/geronimo/j2ee/deployment/EARConfigBuilder.java
Date Thu, 11 Jun 2009 16:02:50 GMT
Author: gawor
Date: Thu Jun 11 16:02:49 2009
New Revision: 783821

URL: http://svn.apache.org/viewvc?rev=783821&view=rev
Log:
fix error message when plan contains modules which are not in the ear. Based on patch from
Ivan (GERONIMO-4661)

Modified:
    geronimo/server/trunk/plugins/j2ee/geronimo-j2ee-builder/src/main/java/org/apache/geronimo/j2ee/deployment/EARConfigBuilder.java

Modified: geronimo/server/trunk/plugins/j2ee/geronimo-j2ee-builder/src/main/java/org/apache/geronimo/j2ee/deployment/EARConfigBuilder.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/plugins/j2ee/geronimo-j2ee-builder/src/main/java/org/apache/geronimo/j2ee/deployment/EARConfigBuilder.java?rev=783821&r1=783820&r2=783821&view=diff
==============================================================================
--- geronimo/server/trunk/plugins/j2ee/geronimo-j2ee-builder/src/main/java/org/apache/geronimo/j2ee/deployment/EARConfigBuilder.java
(original)
+++ geronimo/server/trunk/plugins/j2ee/geronimo-j2ee-builder/src/main/java/org/apache/geronimo/j2ee/deployment/EARConfigBuilder.java
Thu Jun 11 16:02:49 2009
@@ -901,7 +901,9 @@
 
             //all the modules in the geronimo plan should have been found by now.
             if (!moduleLocations.containsAll(altVendorDDs.keySet())) {
-                throw new DeploymentException("Geronimo ear plan contains modules that aren't
in the ear: " + new HashSet<String>(moduleLocations).removeAll(altVendorDDs.keySet()));
+                HashSet<String> missingModules = new HashSet<String>(moduleLocations);
+                missingModules.removeAll(altVendorDDs.keySet());
+                throw new DeploymentException("Geronimo ear plan contains modules that are
not in the ear: " + missingModules);
             }
             //deploy the extension modules
             for (GerExtModuleType gerExtModule : gerApplication.getExtModuleArray()) {



Mime
View raw message