You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@felix.apache.org by "Guillaume Nodet (JIRA)" <ji...@apache.org> on 2017/09/08 08:19:00 UTC
[jira] [Closed] (FELIX-5111) Persistence Manager complete support
for comments
[ https://issues.apache.org/jira/browse/FELIX-5111?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Guillaume Nodet closed FELIX-5111.
----------------------------------
Resolution: Duplicate
Assignee: Guillaume Nodet
> Persistence Manager complete support for comments
> -------------------------------------------------
>
> Key: FELIX-5111
> URL: https://issues.apache.org/jira/browse/FELIX-5111
> Project: Felix
> Issue Type: Bug
> Components: File Install
> Affects Versions: fileinstall-3.5.0
> Reporter: munene kiruja
> Assignee: Guillaume Nodet
> Attachments: ConfigInstaller.java, ConfigInstallerTest.java, GEConfigurationHandler.java
>
>
> Felix Fileinstall 3.5.0 added support for .config files with an elaborate format for type specification and also support for arrays (and collections).
> However it did not support comments but would choke on them, combining comments with the configuration data and corrupting the configuration file. Also the .config files are sorted alphabetically, possibly altering the original order.
> https://issues.apache.org/jira/browse/FELIX-4917 was created and a change submitted that was accepted, that allows the comments to be skipped over, but when the config is written back, the comments are lost. The sorting also still takes place.
> Comments are a vital part of the user interface, communicating information about how to customize the OSGi services. To maintain equivalent functionality with the .cfg format, the following are necessary:
> - reading the .config file should be able to skip the comments
> - writing the .config file back should retain the comments and the structuer of the .config file
> - updating should modify in the update map, leaving any in file but missing in the update set intact
> - updating should add any values in update set but absent in the file
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)