Skip to content

Commit 1c186c6

Browse files
arnoutjacmet
authored andcommitted
qt5quickcontrols: fix installation of PrivateWidgets
Commit e482ebf attempted to fix the installation of qt5quickcontrols by only installing the PrivateWidgets directory for the 5.8.0 version. However, the availability of PrivateWidgets has nothing to do with the version; in both 5.6.2 and 5.8.0, the installation is gated by the following statement in src/src.pro: qtHaveModule(quick):qtHaveModule(widgets): SUBDIRS += widgets i.e. it is installed when both the Quick and the Widgets module are available. The Widgets module is controlled by Buildroot's BR2_PACKAGE_QT5BASE_WIDGETS symbol, the Quick module is controlled by Buildroot's BR2_PACKAGE_QT5DECLARATIVE_QUICK. The qt5quickcontrols package selects BR2_PACKAGE_QT5DECLARATIVE_QUICK so it is not really needed to include it in the condition. However, it is theoretically possible to build this package without QtQuick. Also, adding this condition makes it consistent with src.pro. Note that commit e482ebf introduces a second fix (not mentioned in the commit message): for version 5.6.2, the Layouts directory is installed, but in 5.8.0 this directory doesn't exist any more. Therefore, a separate condition on the version is still needed. Fixes: http://autobuild.buildroot.net/results/1ff3e9ad4ba518d0a37f9fc12038bf9020f28094 Cc: Peter Seiderer <[email protected]> Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <[email protected]> Signed-off-by: Peter Korsgaard <[email protected]>
1 parent 6395652 commit 1c186c6

File tree

1 file changed

+13
-8
lines changed

1 file changed

+13
-8
lines changed

package/qt5/qt5quickcontrols/qt5quickcontrols.mk

Lines changed: 13 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -30,20 +30,25 @@ define QT5QUICKCONTROLS_INSTALL_STAGING_CMDS
3030
$(TARGET_MAKE_ENV) $(MAKE) -C $(@D) install
3131
endef
3232

33-
ifeq ($(BR2_PACKAGE_QT5_VERSION_LATEST),y)
34-
define QT5QUICKCONTROLS_INSTALL_TARGET_CMDS
35-
cp -dpfr $(STAGING_DIR)/usr/qml/QtQuick/Controls $(TARGET_DIR)/usr/qml/QtQuick
36-
cp -dpfr $(STAGING_DIR)/usr/qml/QtQuick/Dialogs $(TARGET_DIR)/usr/qml/QtQuick
37-
cp -dpfr $(STAGING_DIR)/usr/qml/QtQuick/Extras $(TARGET_DIR)/usr/qml/QtQuick
33+
34+
ifeq ($(BR2_PACKAGE_QT5DECLARATIVE_QUICK)$(BR2_PACKAGE_QT5BASE_WIDGETS),yy)
35+
define QT5QUICKCONTROLS_INSTALL_TARGET_PRIVATEWIDGETS
3836
cp -dpfr $(STAGING_DIR)/usr/qml/QtQuick/PrivateWidgets $(TARGET_DIR)/usr/qml/QtQuick
3937
endef
40-
else
38+
endif
39+
40+
ifeq ($(BR2_PACKAGE_QT5_VERSION_5_6),y)
41+
define QT5QUICKCONTROLS_INSTALL_TARGET_LAYOUTS
42+
cp -dpfr $(STAGING_DIR)/usr/qml/QtQuick/Layouts $(TARGET_DIR)/usr/qml/QtQuick
43+
endef
44+
endif
45+
4146
define QT5QUICKCONTROLS_INSTALL_TARGET_CMDS
4247
cp -dpfr $(STAGING_DIR)/usr/qml/QtQuick/Controls $(TARGET_DIR)/usr/qml/QtQuick
4348
cp -dpfr $(STAGING_DIR)/usr/qml/QtQuick/Dialogs $(TARGET_DIR)/usr/qml/QtQuick
4449
cp -dpfr $(STAGING_DIR)/usr/qml/QtQuick/Extras $(TARGET_DIR)/usr/qml/QtQuick
45-
cp -dpfr $(STAGING_DIR)/usr/qml/QtQuick/Layouts $(TARGET_DIR)/usr/qml/QtQuick
50+
$(QT5QUICKCONTROLS_INSTALL_TARGET_PRIVATEWIDGETS)
51+
$(QT5QUICKCONTROLS_INSTALL_TARGET_LAYOUTS)
4652
endef
47-
endif
4853

4954
$(eval $(generic-package))

0 commit comments

Comments
 (0)