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

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

XMLWordPrintable

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

      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()?

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

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

              Created:
              Updated:

                There are no open Gerrit changes