Uploaded image for project: 'Qt Automotive Suite'
  1. Qt Automotive Suite
  2. AUTOSUITE-1595

containers:selection maps from multiple info.yaml files are not merged

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Done
    • Icon: Not Evaluated Not Evaluated
    • None
    • 5.15.0
    • Qt Application Manager
    • None
    • Linux/Wayland
    • 8ed984b218737d5c0689abb9038d4695ccf62cf3 (qt/qtapplicationmanager/5.15)

      During the config parsing phase, these multiple configurations files can be parsed separately and merge together.

      From 5.13 to 5.15 of qtapplicationmanager, configuration handling has been modified drastically.

      Config merging works for everything else, but "containers.selection"

      Ex.

      Below maps of multiple info.yaml should be merged together.

       

      containers:

          selection:

          - some.pattern: "process"

          - "*": "softwarecontainer"

       

      Configurations are merged here.

      https://github.com/qt/qtapplicationmanager/blob/5.15/src/main-lib/configuration.cpp#L531

       

      Here only containers.configurations maps are merged.
      containers.selection lists must also be merged.

       

      I have attached some sample patches made based on 5.15.0 3276c4feae52ed033be2c2b7b1e15ba3d2dcb470

       

       

        1. 0001-mergeListsOfPairs.patch
          2 kB
          Yasas Perera
        2. 0002-mergeContainersSelection.patch
          0.6 kB
          Yasas Perera
        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

            rgriebl Robert Griebl
            yasascperera Yasas Perera
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved:

                There are no open Gerrit changes