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

Support richer format for configurations

    Details

    • Type: New Feature
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: fileinstall-3.1.0
    • Component/s: File Install
    • Labels:
      None

      Description

      The current format read by file install for configurations uses the java.util.Properties - this format does not support multi values and objects other than strings.
      As configurations usually use other types than string, it would be nice to support the richer format like for example the config admin implementation uses.

      To be compatible, I suggest we leave the old behaviour for ".cfg" files and use a new extension ".config" for the other format.

      1. config.patch
        23 kB
        Carsten Ziegeler

        Activity

        Hide
        cziegeler Carsten Ziegeler added a comment -

        I've attached a potential fix for this new feature

        Show
        cziegeler Carsten Ziegeler added a comment - I've attached a potential fix for this new feature
        Hide
        fmeschbe Felix Meschberger added a comment -

        Is it not possible to just embed the file reader class from the Configuration Admin bundle ?

        Show
        fmeschbe Felix Meschberger added a comment - Is it not possible to just embed the file reader class from the Configuration Admin bundle ?
        Hide
        cziegeler Carsten Ziegeler added a comment -

        That should be possible, it's basically just a copy with all write operations removed.

        Show
        cziegeler Carsten Ziegeler added a comment - That should be possible, it's basically just a copy with all write operations removed.
        Hide
        cziegeler Carsten Ziegeler added a comment -

        Applied a modified patch which just embedds the configuration handler from the latest config admin release in revision 982991

        Show
        cziegeler Carsten Ziegeler added a comment - Applied a modified patch which just embedds the configuration handler from the latest config admin release in revision 982991

          People

          • Assignee:
            cziegeler Carsten Ziegeler
            Reporter:
            cziegeler Carsten Ziegeler
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development