You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2008/09/25 12:41:31 UTC

svn commit: r698920 - in /tomcat/tc6.0.x/trunk: STATUS.txt java/org/apache/catalina/util/ManifestResource.java webapps/docs/changelog.xml

Author: markt
Date: Thu Sep 25 03:41:30 2008
New Revision: 698920

URL: http://svn.apache.org/viewvc?rev=698920&view=rev
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=45628
JARs without deps should always be fulfilled

Modified:
    tomcat/tc6.0.x/trunk/STATUS.txt
    tomcat/tc6.0.x/trunk/java/org/apache/catalina/util/ManifestResource.java
    tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=698920&r1=698919&r2=698920&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Thu Sep 25 03:41:30 2008
@@ -88,12 +88,6 @@
   +1: markt, remm
   -1:
 
-* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=45628
-  http://svn.apache.org/viewvc?rev=691282&view=rev
-  JARs without deps should always be fulfilled
-  +1: markt, remm, rjung
-  -1: 
-
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=45618
   Make sure selector is closed.
   Unlikely to be an issue for most (all?) circumstances but technically

Modified: tomcat/tc6.0.x/trunk/java/org/apache/catalina/util/ManifestResource.java
URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/catalina/util/ManifestResource.java?rev=698920&r1=698919&r2=698920&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/java/org/apache/catalina/util/ManifestResource.java (original)
+++ tomcat/tc6.0.x/trunk/java/org/apache/catalina/util/ManifestResource.java Thu Sep 25 03:41:30 2008
@@ -117,7 +117,7 @@
      */
     public boolean isFulfilled() {
         if (requiredExtensions == null) {
-            return false;
+            return true;
         }
         Iterator it = requiredExtensions.iterator();
         while (it.hasNext()) {

Modified: tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml
URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml?rev=698920&r1=698919&r2=698920&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml Thu Sep 25 03:41:30 2008
@@ -70,6 +70,10 @@
         be instantiated. (funkman) 
       </add>
       <fix>
+        <bug>45628</bug>: When checking MANIFEST dependancies, JARs without
+        dependencies should allows be considered to be full-filled. (markt) 
+      </fix>
+      <fix>
         <bug>45785</bug>: Ignore directories named xxx.jar in WEB-INF/lib.
         (markt)
       </fix>



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org