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.");
}
//