Uploaded image for project: 'Qt'
  1. Qt
  2. QTBUG-121136 CMake API Review for Qt 6.7
  3. QTBUG-121708

Address Qt 6.7 CMake API review notes for deployment api

    XMLWordPrintable

Details

    • Technical task
    • Resolution: Fixed
    • P2: Important
    • 6.7.0
    • Build System: CMake
    • None
    • e39d13bc8 (dev), 1902f98a9 (dev), b8a78054c (dev), 313b7c56b (dev), 5d1ddee4c (dev), 01a0b8199 (6.7), 101846afa (6.7), b09c637c4 (6.7), 7ffe54833 (6.7), 564e3ed62 (6.7), 355a2f1ef (dev), 3973a3e9a (6.7)

    Description

      See the following action points

      https://git.qt.io/alcroito/qt6-cmake-api-review/-/blob/master/api_review_qt_6_7.md#qt_generate_deploy_app_script-qt_deploy_runtime_dependencies-new-api-option

      AP: Add since qt version

      AP: Add example snippet

      AP: Clarify in docs that it only works for windeployqt / macdeployqt

      AP: Clarify in docs how to use tool-specific with an if/else dance, probably in example snippet as well

      AP: Add warning when a value is passed when targeting Linux, because it's a no-op

      https://git.qt.io/alcroito/qt6-cmake-api-review/-/blob/master/api_review_qt_6_7.md#qt_deploy_libexec_dir-new-deploy-time-variable

      AP: Fix since qt version blurb

      AP: Modify example snippet usage, add inline comment that `CMAKE_INSTALL_LIBEXECDIR` sets `QT_DEPLOY_LIBEXEC_DIR`

      AP: Add a mention of QtWebEngineProcess to docs as an example why one would want to use this

      AP: Convert the single bullet point in docs into regular paragraph

      AP: Add a note to all deploy variables that they are low-level options that you probably don't care about

      AP: Create a task to split configure vars reference from deploy vars reference

      https://git.qt.io/alcroito/qt6-cmake-api-review/-/blob/master/api_review_qt_6_7.md#qt_deploy_runtime_dependencies-libexec_dir-new-api-option

      AP: Add since qt version into text or synopsis

      AP: Add a link to https://doc-snapshots.qt.io/qt6-dev/cmake-variable-reference.html#qt6-core
      after the paragraph that lists LIBEXEC_DIR, with wording like
      "See the deploy variables that are overridden by the options mentioned above"
      Or Add table with 2 columnes, option and associated variable

      AP: Don't add example snippet, just link from the table

      https://git.qt.io/alcroito/qt6-cmake-api-review/-/blob/master/api_review_qt_6_7.md#qt_generate_deploy_qml_app_script-new-api-option

      AP: Do the same APs as for the qtbase equivalent

      Attachments

        Issue Links

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

          Activity

            People

              jbornema Joerg Bornemann
              alexandru.croitor Alexandru Croitor
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: