Uploaded image for project: 'Qt'
  1. Qt
  2. QTBUG-131141

Target finalizers are not run for qt_internal_add_executable/app/tool

    XMLWordPrintable

Details

    • Bug
    • Resolution: Unresolved
    • P3: Somewhat important
    • None
    • 6.8.0 Beta1, 6.8.1
    • Build System: CMake
    • None

    Description

      This is split from QTBUG-112212

      Comment from above issue.

      Running finalizers should now work for qt_internal_add_tests and friends executables.

      They are still not run for qt_internal_add_executable, qt_internal_add_app, qt_internal_add_tool (EDIT: except for ANDROID and WASM where we do run them for any kind of internal executables).

      That needs more work.
      Part of the reasoning is described at https://codereview.qt-project.org/c/qt/qtbase/+/599348/10/cmake/QtExecutableHelpers.cmake#46

      One of the issues is that trying to run finalizers for the qmlimportscanner executable (which is a tool) would create a circular depenendecy trying to run
      qmlimportscanner on itself.

      Attachments

        Issue Links

          resulted from

          Bug - A problem which impairs or prevents the functions of the product. QTBUG-93625 qt_internal_add_test doesn't call qt_import_qml_plugins

          • P2: Important - Urgent, should be fixed, but will not stop the release.
          • Closed

          Task - A task that needs to be done. QTBUG-112212 Unify target finalizer execution (e.g. auto-linking plugins) for tests across platforms

          • P2: Important - Urgent, should be fixed, but will not stop the release.
          • Closed
          No reviews matched the request. Check your Options in the drop-down menu of this sections header.

          Activity

            People

              qtbuildsystem Qt Build System Team
              alexandru.croitor Alexandru Croitor
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:

                Gerrit Reviews

                  There are no open Gerrit changes