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 2007/03/26 15:41:31 UTC

svn commit: r522513 - /incubator/felix/trunk/framework/src/main/java/org/apache/felix/framework/util/manifestparser/Requirement.java

Author: rickhall
Date: Mon Mar 26 06:41:26 2007
New Revision: 522513

URL: http://svn.apache.org/viewvc?view=rev&rev=522513
Log:
To avoid potential bugs with arbitrary attributes named "version", I 
modified Requirement to only cache the version for package dependencies.

Modified:
    incubator/felix/trunk/framework/src/main/java/org/apache/felix/framework/util/manifestparser/Requirement.java

Modified: incubator/felix/trunk/framework/src/main/java/org/apache/felix/framework/util/manifestparser/Requirement.java
URL: http://svn.apache.org/viewvc/incubator/felix/trunk/framework/src/main/java/org/apache/felix/framework/util/manifestparser/Requirement.java?view=diff&rev=522513&r1=522512&r2=522513
==============================================================================
--- incubator/felix/trunk/framework/src/main/java/org/apache/felix/framework/util/manifestparser/Requirement.java (original)
+++ incubator/felix/trunk/framework/src/main/java/org/apache/felix/framework/util/manifestparser/Requirement.java Mon Mar 26 06:41:26 2007
@@ -61,7 +61,8 @@
             {
                 m_pkgName = (String) m_attributes[i].getValue();
             }
-            else if (m_attributes[i].getName().equals(ICapability.VERSION_PROPERTY))
+            else if (m_namespace.equals(ICapability.PACKAGE_NAMESPACE) &&
+                     m_attributes[i].getName().equals(ICapability.VERSION_PROPERTY))
             {
                 m_pkgVersionRange = (VersionRange) m_attributes[i].getValue();
             }