[FELIX-6218] Replace kxml2 with standard SAX XML parser

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: scr-2.1.16
• Fix Version/s: scr-2.1.18
• Component/s: Declarative Services (SCR)
• Flags: Patch
• Assignee: Carsten Ziegeler
• Reporter: Mat Booth
• Created: 29/Jan/20 18:06
• Updated: 03/Jun/20 11:54
• Resolved: 14/Feb/20 15:24

Description

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/

Issue Links

• links to: GitHub Pull Request #25