svn commit: r399082 - /geronimo/branches/1.1/modules/deployment/src/java/org/apache/geronimo/deployment/Deployer.java

[am...@apache.org]

Author: ammulder
Date: Tue May  2 16:22:09 2006
New Revision: 399082

URL: http://svn.apache.org/viewcvs?rev=399082&view=rev
Log:
More informative error message if no deployers handle a module
  (GERONIMO-1360)  Thanks Erin for the patch!

Modified:
    geronimo/branches/1.1/modules/deployment/src/java/org/apache/geronimo/deployment/Deployer.java

Modified: geronimo/branches/1.1/modules/deployment/src/java/org/apache/geronimo/deployment/Deployer.java
URL: http://svn.apache.org/viewcvs/geronimo/branches/1.1/modules/deployment/src/java/org/apache/geronimo/deployment/Deployer.java?rev=399082&r1=399081&r2=399082&view=diff
==============================================================================
--- geronimo/branches/1.1/modules/deployment/src/java/org/apache/geronimo/deployment/Deployer.java (original)
+++ geronimo/branches/1.1/modules/deployment/src/java/org/apache/geronimo/deployment/Deployer.java Tue May  2 16:22:09 2006
@@ -231,7 +231,9 @@
                 }
             }
             if (builder == null) {
-                throw new DeploymentException("Cannot deploy the requested application module because no builder is able to handle it (" +
+                throw new DeploymentException("Cannot deploy the requested application module because no deployer is able to handle it. " +
+                        " This can happen if you have omitted the J2EE deployment descriptor, disabled a deployer module, or if, for example, you are trying to deploy an" +
+                        " EJB module on a minimal Geronimo server that does not have EJB support installed.  (" +
                         (planFile == null ? "" : "planFile=" + planFile.getAbsolutePath()) +
                         (moduleFile == null ? "" : (planFile == null ? "" : ", ") + "moduleFile=" + moduleFile.getAbsolutePath()) + ")");
             }