Uploaded image for project: 'Qbs ("Cubes")'
  1. Qbs ("Cubes")
  2. QBS-1517

List properties should be merged in reverse module dependency order

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Done
    • Icon: Not Evaluated Not Evaluated
    • 1.16.0
    • 1.14.1
    • Dependency Tracking
    • None
    • All
    • 8ff1dd0044a32b6658cf05a923946d75baf33a66 (qbs/qbs/master)

      Consider the following project:

      Module {
          name: "common"
          property stringList listProp
      }
      Module {
          name: "higher1"
          Depends { name: "common" }
          common.listProp: "higher1"
      }
      Module {
          name: "higher2"
          Depends { name: "higher1" }
          common.listProp: "higher2"
      }
      
      Product {
          name: "p"
          Depends { name: "higher2" }
          common.listProp: "product"
          property stringList showMeTheOrder: {
              console.info(common.listProp);
          }
      } 
      • Expected output of listprop: [product, higher2, higher1, common]
      • Actual order: [product, higher1, higher2, common]

      Looks like common is correctly placed at the end, but the order of higher1 and higher2 is incorrect.

      cpp.includePaths. Module dependency relationships should be respected. Ideally, the merge order should be controllable by a property in PropertyOptions.

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

            rweickelt Richard Weickelt
            rweickelt Richard Weickelt
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved:

                There are no open Gerrit changes