Uploaded image for project: 'Felix'
  1. Felix
  2. FELIX-5111

Persistence Manager complete support for comments

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Duplicate
    • Affects Version/s: fileinstall-3.5.0
    • Fix Version/s: None
    • Component/s: File Install
    • Labels:
      None

      Description

      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

        Attachments

        1. GEConfigurationHandler.java
          30 kB
          munene kiruja
        2. ConfigInstallerTest.java
          16 kB
          munene kiruja
        3. ConfigInstaller.java
          16 kB
          munene kiruja

          Issue Links

            Activity

              People

              • Assignee:
                gnodet Guillaume Nodet
                Reporter:
                munene.kiruja munene kiruja
              • Votes:
                1 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: