You are viewing a plain text version of this content. The canonical link for it is here.
Posted to ivy-commits@incubator.apache.org by gs...@apache.org on 2007/10/08 12:54:45 UTC

svn commit: r582811 - /incubator/ivy/core/trunk/src/java/org/apache/ivy/core/resolve/ResolveEngine.java

Author: gscokart
Date: Mon Oct  8 05:54:44 2007
New Revision: 582811

URL: http://svn.apache.org/viewvc?rev=582811&view=rev
Log:
IVY-618 : fix NPE when the module is evicted and the dependency descriptor is null

Modified:
    incubator/ivy/core/trunk/src/java/org/apache/ivy/core/resolve/ResolveEngine.java

Modified: incubator/ivy/core/trunk/src/java/org/apache/ivy/core/resolve/ResolveEngine.java
URL: http://svn.apache.org/viewvc/incubator/ivy/core/trunk/src/java/org/apache/ivy/core/resolve/ResolveEngine.java?rev=582811&r1=582810&r2=582811&view=diff
==============================================================================
--- incubator/ivy/core/trunk/src/java/org/apache/ivy/core/resolve/ResolveEngine.java (original)
+++ incubator/ivy/core/trunk/src/java/org/apache/ivy/core/resolve/ResolveEngine.java Mon Oct  8 05:54:44 2007
@@ -257,16 +257,13 @@
                                 throw new NullPointerException("getResolvedId() is null for " 
                                     + dependencies[i].toString());
                             }
-                            if (depDescriptor == null) {
-                                throw new NullPointerException("getDescriptor() is null for " 
-                                    + dependencies[i].toString());
-                            }
                             if (depRevisionId == null) {
                                 throw new NullPointerException("getDependencyRevisionId() "
                                     + "is null for " + dd.toString());
                             }
                             String rev = depResolvedId.getRevision();
-                            String status = depDescriptor.getStatus();
+                            //The evicted modules have no descritpion, so we can't put their status. 
+                            String status = depDescriptor==null ? "?" : depDescriptor.getStatus();
                             props.put(depRevisionId.encodeToString(), rev + " " + status);
                         }
                     }