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
        For Gerrit Dashboard: AUTOSUITE-1595
        # Subject Branch Project Status CR V

            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