Author: brianf
Date: Wed Nov  7 04:01:44 2007
New Revision: 592722

URL: http://svn.apache.org/viewvc?rev=592722&view=rev
Log:
MENFORCER-18: beanshell swallows exceptions

Modified:
    
maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/EvaluateBeanshell.java

Modified: 
maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/EvaluateBeanshell.java
URL: 
http://svn.apache.org/viewvc/maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/EvaluateBeanshell.java?rev=592722&r1=592721&r2=592722&view=diff
==============================================================================
--- 
maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/EvaluateBeanshell.java
 (original)
+++ 
maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/EvaluateBeanshell.java
 Wed Nov  7 04:01:44 2007
@@ -27,6 +27,8 @@
 import org.apache.maven.execution.RuntimeInformation;
 import org.apache.maven.plugin.logging.Log;
 import org.apache.maven.project.MavenProject;
+import 
org.codehaus.plexus.component.configurator.expression.ExpressionEvaluationException;
+import 
org.codehaus.plexus.component.repository.exception.ComponentLookupException;
 import org.codehaus.plexus.util.StringUtils;
 
 import bsh.EvalError;
@@ -99,9 +101,13 @@
                 throw new EnforcerRuleException( this.message );
             }
         }
-        catch ( Exception e )
+        catch (ComponentLookupException e)
         {
             throw new EnforcerRuleException( "Unable to lookup a component", e 
);
+        }
+        catch ( ExpressionEvaluationException e )
+        {
+            throw new EnforcerRuleException( "Unable to evaluate an 
expression", e );
         }
     }
 


Reply via email to