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

    • Icon: Technical task Technical task
    • Resolution: Fixed
    • Icon: P2: Important 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)

      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

        For Gerrit Dashboard: QTBUG-121708
        # Subject Branch Project Status CR V

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

              Created:
              Updated:
              Resolved: