You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by pa...@apache.org on 2018/08/24 21:59:30 UTC

svn commit: r1838974 - /felix/trunk/framework/src/main/java/org/apache/felix/framework/ExtensionManager.java

Author: pauls
Date: Fri Aug 24 21:59:30 2018
New Revision: 1838974

URL: http://svn.apache.org/viewvc?rev=1838974&view=rev
Log:
FELIX-5912: Handle empty package definitions in system package definitions more gracefully

Modified:
    felix/trunk/framework/src/main/java/org/apache/felix/framework/ExtensionManager.java

Modified: felix/trunk/framework/src/main/java/org/apache/felix/framework/ExtensionManager.java
URL: http://svn.apache.org/viewvc/felix/trunk/framework/src/main/java/org/apache/felix/framework/ExtensionManager.java?rev=1838974&r1=1838973&r2=1838974&view=diff
==============================================================================
--- felix/trunk/framework/src/main/java/org/apache/felix/framework/ExtensionManager.java (original)
+++ felix/trunk/framework/src/main/java/org/apache/felix/framework/ExtensionManager.java Fri Aug 24 21:59:30 2018
@@ -952,6 +952,11 @@ class ExtensionManager implements Conten
             syspkgs = ((pkgextra == null) || (pkgextra.trim().length() == 0))
                 ? syspkgs : syspkgs + (pkgextra.trim().startsWith(",") ? pkgextra : "," + pkgextra);
 
+            if (syspkgs.startsWith(","))
+            {
+                syspkgs = syspkgs.substring(1);
+            }
+            
             m_headerMap.put(FelixConstants.EXPORT_PACKAGE, syspkgs);
 
             // The system bundle alsp provides framework generic capabilities