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

CMake: qt6_add_qml_module() doesn't set arch suffix on Android

    XMLWordPrintable

Details

    • Bug
    • Resolution: Unresolved
    • P2: Important
    • None
    • 6.1.0 Beta 1
    • Build System: CMake
    • None
    • Android

    Description

      The qt6_add_qml_module() function currently contains the following code:

          if (ANDROID)
              # Adjust Qml plugin names on Android similar to qml_plugin.prf which calls
              # $$qt5LibraryTarget($$TARGET, "qml/$$TARGETPATH/").
              # Example plugin names:
              # qtdeclarative
              #   TARGET_PATH: QtQml/Models
              #   file name:   libqml_QtQml_Models_modelsplugin_arm64-v8a.so
              # qtquickcontrols2
              #   TARGET_PATH: QtQuick/Controls.2/Material
              #   file name:
              #     libqml_QtQuick_Controls.2_Material_qtquickcontrols2materialstyleplugin_arm64-v8a.so
              string(REPLACE "/" "_" android_plugin_name_infix_name "${arg_TARGET_PATH}")
      
              set(final_android_qml_plugin_name "qml_${android_plugin_name_infix_name}_${target}")
              set_target_properties(${target}
                  PROPERTIES
                  LIBRARY_OUTPUT_NAME "${final_android_qml_plugin_name}"
              )
          endif()
      

      The comments show that the file name should end in an arch-specific suffix, but the target's SUFFIX property is not set. This means there will be no such arch-specific suffix.

      In qt6_add_plugin() (defined in qtbase), there is a call to qt6_android_apply_arch_suffix() for the analogous code there. Should there also be a call to qt6_android_apply_arch_suffix() here in qt6_add_qml_module()?

      Attachments

        Issue Links

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

          Activity

            People

              qtbuildsystem Qt Build System Team
              crscott Craig Scott
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:

                Gerrit Reviews

                  There are no open Gerrit changes