Andreas, after searching for other locations of this string, it's not used at all beside the actual xsd which is the referenced schema :) Therefore I leave it that way.
regards, Achim > Andreas, > > your absolutely right. Will do a small refactoring while introducing > pax-url 1.3.2 in trunk :) I think it's OK for 2.2.x though. > > regards, Achim > > > Am 09.04.2011 08:35, schrieb Andreas Pieber: >> mhm, maybe we would like to keep >> "http://karaf.apache.org/xmlns/features/v1.0.0" in some constants file >> to avoid future changes at multiple locations? >> >> On Sat, Apr 9, 2011 at 12:00 AM, <[email protected]> wrote: >>> Author: anierbeck >>> Date: Fri Apr 8 22:00:13 2011 >>> New Revision: 1090462 >>> >>> URL: http://svn.apache.org/viewvc?rev=1090462&view=rev >>> Log: >>> [KARAF-518] - Unable to load features.xml with included schema attribute >>> >>> Modified: >>> >>> karaf/branches/karaf-2.2.x/deployer/features/src/main/java/org/apache/karaf/deployer/features/FeatureDeploymentListener.java >>> >>> Modified: >>> karaf/branches/karaf-2.2.x/deployer/features/src/main/java/org/apache/karaf/deployer/features/FeatureDeploymentListener.java >>> URL: >>> http://svn.apache.org/viewvc/karaf/branches/karaf-2.2.x/deployer/features/src/main/java/org/apache/karaf/deployer/features/FeatureDeploymentListener.java?rev=1090462&r1=1090461&r2=1090462&view=diff >>> ============================================================================== >>> --- >>> karaf/branches/karaf-2.2.x/deployer/features/src/main/java/org/apache/karaf/deployer/features/FeatureDeploymentListener.java >>> (original) >>> +++ >>> karaf/branches/karaf-2.2.x/deployer/features/src/main/java/org/apache/karaf/deployer/features/FeatureDeploymentListener.java >>> Fri Apr 8 22:00:13 2011 >>> @@ -92,7 +92,7 @@ public class FeatureDeploymentListener i >>> Document doc = parse(artifact); >>> String name = doc.getDocumentElement().getLocalName(); >>> String uri = doc.getDocumentElement().getNamespaceURI(); >>> - if ("features".equals(name) && (uri == null || >>> "".equals(uri))) { >>> + if ("features".equals(name) && (uri == null || >>> "".equals(uri) || >>> "http://karaf.apache.org/xmlns/features/v1.0.0".equalsIgnoreCase(uri))) { >>> return true; >>> } >>> } >>> >>> >>>
