[
https://issues.apache.org/jira/browse/SLING-10224?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Bertrand Delacretaz resolved SLING-10224.
-----------------------------------------
Resolution: Fixed
Ok, thank you very much for your contribution, I have merged my branch now,
which should include all your changes.
If you have ideas for more tests they are welcome of course!
> Feature-Launcher: only embed a minimal set of apache-commons classes
> ---------------------------------------------------------------------
>
> Key: SLING-10224
> URL: https://issues.apache.org/jira/browse/SLING-10224
> Project: Sling
> Issue Type: Improvement
> Components: Feature Model
> Affects Versions: Feature Model Launcher 1.1.16
> Reporter: Stefan Bischof
> Assignee: Bertrand Delacretaz
> Priority: Major
> Fix For: Feature Model Launcher 1.1.18
>
>
> StringSubstitutor from commons-text is a great Util to replace '${thisVar}'
> with something. In the SlingFeatureLauncher case it replaces placeholder with
> the value of the map`context.getFrameworkProperties()` , or an empty string.
> But commons-lang3 and commons-text brings 600kb to the Bundle.
> Maybe we can Substitute it with a more lightweight StringSubstitutor.
> _Update: as discussed below, embedding just the required apache-commons
> classes has the same effect, without having to rewrite code_
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
