You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by ri...@apache.org on 2010/05/14 21:21:19 UTC

svn commit: r944411 - /felix/trunk/framework/src/main/java/org/apache/felix/framework/resolver/ResolverImpl.java

Author: rickhall
Date: Fri May 14 19:21:19 2010
New Revision: 944411

URL: http://svn.apache.org/viewvc?rev=944411&view=rev
Log:
Improve exception messages. (FELIX-2035)

Modified:
    felix/trunk/framework/src/main/java/org/apache/felix/framework/resolver/ResolverImpl.java

Modified: felix/trunk/framework/src/main/java/org/apache/felix/framework/resolver/ResolverImpl.java
URL: http://svn.apache.org/viewvc/felix/trunk/framework/src/main/java/org/apache/felix/framework/resolver/ResolverImpl.java?rev=944411&r1=944410&r2=944411&view=diff
==============================================================================
--- felix/trunk/framework/src/main/java/org/apache/felix/framework/resolver/ResolverImpl.java (original)
+++ felix/trunk/framework/src/main/java/org/apache/felix/framework/resolver/ResolverImpl.java Fri May 14 19:21:19 2010
@@ -985,10 +985,11 @@ public class ResolverImpl implements Res
                         ResolveException ex = new ResolveException(
                             "Constraint violation for package '"
                             + entry.getKey() + "' when resolving module "
-                            + module + " between an imported constraint "
-                            + sourceBlame + " and an additional imported constraint "
+                            + module + " between an import "
+                            + sourceBlame + " and a fragment import "
                             + blame, module, blame.m_reqs.get(0));
                         m_logger.log(Logger.LOG_DEBUG, "Conflicting fragment import", ex);
+                        throw ex;
                     }
                 }
             }
@@ -1013,7 +1014,7 @@ public class ResolverImpl implements Res
                         : new ResolveException(
                             "Constraint violation for package '"
                             + pkgName + "' when resolving module "
-                            + module + " between existing exported constraint "
+                            + module + " between existing export "
                             + entry.getValue() + " and uses constraint "
                             + blame, null, null);
                     mutated = (mutated != null)
@@ -1072,7 +1073,7 @@ public class ResolverImpl implements Res
                             : new ResolveException(
                                 "Constraint violation for package '"
                                 + pkgName + "' when resolving module "
-                                + module + " between existing imported constraint "
+                                + module + " between existing import "
                                 + importBlame + " and uses constraint "
                                 + usedBlame, null, null);