Carsten Ziegeler resolved FELIX-6218.
    Resolution: Fixed

Thanks [~mbooth] for the patch. It's applied in rev 1874024
I needed to handle an additional special case in the parser (difference in the 
behaviour of the two parsers). I covered that with a new test case

> Replace kxml2 with standard SAX XML parser
> ------------------------------------------
>                 Key: FELIX-6218
>                 URL: https://issues.apache.org/jira/browse/FELIX-6218
>             Project: Felix
>          Issue Type: Improvement
>          Components: Declarative Services (SCR)
>    Affects Versions: scr-2.1.16
>            Reporter: Mat Booth
>            Assignee: Carsten Ziegeler
>            Priority: Major
>             Fix For: scr-2.1.18
> Currently SCR requires kxml2 (and transitively xpp3) at build time and them 
> embeds some classes from these two libraries in the resulting Felix SCR 
> bundle. However these projects seem quite dead and have not seen any official 
> activity in many years. [1]
> When I tried to investigate why Felix SCR uses kxml2, I hit a dead-end 
> because the repository history does not go back far enough. SCR seems to have 
> been imported already using kxml2/xpp3.
> Two possibilities spring to mind:
>  * It was before Java 1.5, so there was no SAX implementation in the JDK yet
>  * SCR used to target some constrained execution environment that it no 
> longer supports.
> In either case, since Felix SCR now requires JavaSE-1.7 as its minimal 
> execution environment I advocate switching to the SAX implementation provided 
> by the JDK
> I will post a PR
> [1] http://www.xmlpull.org/

This message was sent by Atlassian Jira

Reply via email to