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/01/25 20:36:44 UTC

svn commit: r902953 - /felix/sandbox/rickhall/framework-proto/src/main/java/org/apache/felix/framework/util/manifestparser/ManifestParser.java

Author: rickhall
Date: Mon Jan 25 19:36:43 2010
New Revision: 902953

URL: http://svn.apache.org/viewvc?rev=902953&view=rev
Log:
Incorrectly checking for symbolic name when parsing metadata.

Modified:
    felix/sandbox/rickhall/framework-proto/src/main/java/org/apache/felix/framework/util/manifestparser/ManifestParser.java

Modified: felix/sandbox/rickhall/framework-proto/src/main/java/org/apache/felix/framework/util/manifestparser/ManifestParser.java
URL: http://svn.apache.org/viewvc/felix/sandbox/rickhall/framework-proto/src/main/java/org/apache/felix/framework/util/manifestparser/ManifestParser.java?rev=902953&r1=902952&r2=902953&view=diff
==============================================================================
--- felix/sandbox/rickhall/framework-proto/src/main/java/org/apache/felix/framework/util/manifestparser/ManifestParser.java (original)
+++ felix/sandbox/rickhall/framework-proto/src/main/java/org/apache/felix/framework/util/manifestparser/ManifestParser.java Mon Jan 25 19:36:43 2010
@@ -112,13 +112,13 @@
                     owner, Capability.HOST_NAMESPACE, new ArrayList<Directive>(0),
                     ((CapabilityImpl) moduleCap).getAttributes()));
             }
+        }
 
-            // Verify that bundle symbolic name is specified.
-            if (getManifestVersion().equals("2") && (m_bundleSymbolicName == null))
-            {
-                throw new BundleException(
-                    "R4 bundle manifests must include bundle symbolic name.");
-            }
+        // Verify that bundle symbolic name is specified.
+        if (getManifestVersion().equals("2") && (m_bundleSymbolicName == null))
+        {
+            throw new BundleException(
+                "R4 bundle manifests must include bundle symbolic name.");
         }
 
         //