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;
>>>                 }
>>>             }
>>>
>>>
>>>

Reply via email to