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

mergeField does not merge two QStringLists correctly

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: P1: Critical
    • Resolution: Fixed
    • Affects Version/s: 5.15
    • Fix Version/s: 5.15.1
    • Component/s: Qt Application Manager
    • Labels:
      None

      Description

      If appman is provided with several configurations, it is supposed to merge the configuration files, resulting in a new configuration.

      For parameters like ui.importPaths the result should be a list of all defined import paths from both configurations.

      However, currently it will only use the ui.importPaths list from the latter of the two configuration files.

      This happens because ui.importPaths is a QStringList and the best overload match of mergeField would be:

      template <typename T> void mergeField(T &into, const T &from, const T &def)
      {
       if (from != def)
       into = from;
      }

       

      The same result would happen for every other QStringList in the configuration.

        Attachments

        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

          Activity

            People

            Assignee:
            rgriebl Robert Griebl
            Reporter:
            bjarne Bjarne Michelsen
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Gerrit Reviews

                There are no open Gerrit changes