svn commit: r1326739 - in /felix/trunk/framework/src/main/java/org/apache/felix/framework: BundleImpl.java BundleRevisionImpl.java util/manifestparser/ManifestParser.java

[rickhall]

Author: rickhall
Date: Mon Apr 16 18:50:07 2012
New Revision: 1326739

URL: http://svn.apache.org/viewvc?rev=1326739&view=rev
Log:
Improve handling of "fragment" bundles that are not R4 bundles. (FELIX-3343)

Modified:
    
felix/trunk/framework/src/main/java/org/apache/felix/framework/BundleImpl.java
    
felix/trunk/framework/src/main/java/org/apache/felix/framework/BundleRevisionImpl.java
    
felix/trunk/framework/src/main/java/org/apache/felix/framework/util/manifestparser/ManifestParser.java

Modified: 
felix/trunk/framework/src/main/java/org/apache/felix/framework/BundleImpl.java
URL: 
http://svn.apache.org/viewvc/felix/trunk/framework/src/main/java/org/apache/felix/framework/BundleImpl.java?rev=1326739&r1=1326738&r2=1326739&view=diff
==============================================================================
--- 
felix/trunk/framework/src/main/java/org/apache/felix/framework/BundleImpl.java 
(original)
+++ 
felix/trunk/framework/src/main/java/org/apache/felix/framework/BundleImpl.java 
Mon Apr 16 18:50:07 2012
@@ -1082,6 +1082,7 @@ class BundleImpl implements Bundle, Bund
         return (thisBundleId < thatBundleId ? -1 : (thisBundleId == 
thatBundleId ? 0 : 1));
     }
 
+    @Override
     public String toString()
     {
         String sym = getSymbolicName();

Modified: 
felix/trunk/framework/src/main/java/org/apache/felix/framework/BundleRevisionImpl.java
URL: 
http://svn.apache.org/viewvc/felix/trunk/framework/src/main/java/org/apache/felix/framework/BundleRevisionImpl.java?rev=1326739&r1=1326738&r2=1326739&view=diff
==============================================================================
--- 
felix/trunk/framework/src/main/java/org/apache/felix/framework/BundleRevisionImpl.java
 (original)
+++ 
felix/trunk/framework/src/main/java/org/apache/felix/framework/BundleRevisionImpl.java
 Mon Apr 16 18:50:07 2012
@@ -225,7 +225,8 @@ public class BundleRevisionImpl implemen
 
     public int getTypes()
     {
-        if (getHeaders().containsKey(Constants.FRAGMENT_HOST))
+        if ((getManifestVersion() == "2")
+            && getHeaders().containsKey(Constants.FRAGMENT_HOST))
         {
             return BundleRevision.TYPE_FRAGMENT;
         }

Modified: 
felix/trunk/framework/src/main/java/org/apache/felix/framework/util/manifestparser/ManifestParser.java
URL: 
http://svn.apache.org/viewvc/felix/trunk/framework/src/main/java/org/apache/felix/framework/util/manifestparser/ManifestParser.java?rev=1326739&r1=1326738&r2=1326739&view=diff
==============================================================================
--- 
felix/trunk/framework/src/main/java/org/apache/felix/framework/util/manifestparser/ManifestParser.java
 (original)
+++ 
felix/trunk/framework/src/main/java/org/apache/felix/framework/util/manifestparser/ManifestParser.java
 Mon Apr 16 18:50:07 2012
@@ -1363,7 +1363,6 @@ public class ManifestParser
             s = (s == null) ? (String) headerMap.get(Constants.BUNDLE_NAME) : 
s;
             s = (s == null) ? headerMap.toString() : s;
             logger.log(
-                owner.getBundle(),
                 Logger.LOG_WARNING,
                 "Only R4 bundles can be fragments: " + s);
         }