diff mbox series

[PATCH-for-2020.02.x,v2] package/qt5xmlpatterns: enable qml module

Message ID 20200313204415.2036-1-ps.report@gmx.net
State Accepted
Headers show
Series [PATCH-for-2020.02.x,v2] package/qt5xmlpatterns: enable qml module | expand

Commit Message

Peter Seiderer March 13, 2020, 8:44 p.m. UTC
qt5declarative:
 - remove unneded dependency on qt5mlpatterns

qt5xmlpatterns:
 - add optional dependency on qt5declarative
 - add target install step of the optional qml files

Reported-by: Nimai Mahajan <nimaim@gmail.com>
Signed-off-by: Peter Seiderer <ps.report@gmx.net>
---
Changes v1 -> v2:
  - add Reported-by
  - s/=/+=/ for qt5declarative dependency
---
 package/qt5/qt5declarative/Config.in         |  1 -
 package/qt5/qt5declarative/qt5declarative.mk |  2 +-
 package/qt5/qt5xmlpatterns/qt5xmlpatterns.mk | 11 +++++++++++
 3 files changed, 12 insertions(+), 2 deletions(-)

Comments

Thomas Petazzoni March 14, 2020, 2:18 p.m. UTC | #1
Hello Peter,

On Fri, 13 Mar 2020 21:44:15 +0100
Peter Seiderer <ps.report@gmx.net> wrote:

> qt5declarative:
>  - remove unneded dependency on qt5mlpatterns
> 
> qt5xmlpatterns:
>  - add optional dependency on qt5declarative
>  - add target install step of the optional qml files
> 
> Reported-by: Nimai Mahajan <nimaim@gmail.com>
> Signed-off-by: Peter Seiderer <ps.report@gmx.net>

Will you send a similar patch for master ?

Thanks,

Thomas
Nimai Mahajan March 14, 2020, 3:22 p.m. UTC | #2
On Sat, Mar 14, 2020 at 10:18 AM Thomas Petazzoni <
thomas.petazzoni@bootlin.com> wrote:

> Hello Peter,
>
> On Fri, 13 Mar 2020 21:44:15 +0100
> Peter Seiderer <ps.report@gmx.net> wrote:
>
> > qt5declarative:
> >  - remove unneded dependency on qt5mlpatterns
> >
> > qt5xmlpatterns:
> >  - add optional dependency on qt5declarative
> >  - add target install step of the optional qml files
> >
> > Reported-by: Nimai Mahajan <nimaim@gmail.com>
> > Signed-off-by: Peter Seiderer <ps.report@gmx.net>
>
> Will you send a similar patch for master ?
>
> Thanks,
>
> Thomas
>

Thomas,

Please wait for my Tested-by: :)

Peter,

This is not working for me after using your V2 patch. Perhaps it is my own
user error. The patch simply seems to change some dependencies around
(which looks correct now) and explicitly copies a XmlListModel folder into
$(TARGET_DIR)/usr/qml/QtQuick, but there is nothing to copy, even with the
qt5declarative and qt5xmlpatterns packages checked. For reference, here is
everything mentioning xmllistmodel in the qt5 build directory:

~/Desktop/test_defconfig_output/build$ find . -iname "*xmllistmodel*"
./qt5declarative-5.12.7/src/quick/doc/images/qml-xmllistmodel-example.png
./qt5declarative-5.12.7/tests/testapplications/elements/content/XmlListModelElement.qml
./qt5xmlpatterns-5.12.7/src/imports/xmllistmodel
./qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/xmllistmodel.pro
./qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/qqmlxmllistmodel_p.h
./qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/qqmlxmllistmodel.cpp
./qt5xmlpatterns-5.12.7/tests/auto/qquickxmllistmodel
./qt5xmlpatterns-5.12.7/tests/auto/qquickxmllistmodel/qquickxmllistmodel.pro
./qt5xmlpatterns-5.12.7/tests/auto/qquickxmllistmodel/tst_qquickxmllistmodel.cpp

It seems like the xmllistmodel.pro file there is indeed compiling it as a
plugin, but there is no XmlListModel in $(STAGING_DIR)/usr/qml because
again, it doesn't seem to be compiling it. I don't see anything in the
xmlpatterns Makefile that explicitly goes into src/imports and compiles it.

Also, in the case there are more imports there in the future, should it not
be compiling all the projects in build/qt5xmlpatterns-5.12.7/src/imports/
imports.pro? This would compile all the projects listed in SUBDIRS there,
and output the appropriate folder with it's .so, plugins.qmltypes, and
qmldir, which then could be copied over.

Let me know if I'm missing something.

Thanks,
Nimai



> --
> Thomas Petazzoni, CTO, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com
>
Peter Seiderer March 14, 2020, 5:53 p.m. UTC | #3
Hello Nimai,

On Sat, 14 Mar 2020 11:22:31 -0400, Nimai Mahajan <nimaim@gmail.com> wrote:

> On Sat, Mar 14, 2020 at 10:18 AM Thomas Petazzoni <
> thomas.petazzoni@bootlin.com> wrote:
>
> > Hello Peter,
> >
> > On Fri, 13 Mar 2020 21:44:15 +0100
> > Peter Seiderer <ps.report@gmx.net> wrote:
> >
> > > qt5declarative:
> > >  - remove unneded dependency on qt5mlpatterns
> > >
> > > qt5xmlpatterns:
> > >  - add optional dependency on qt5declarative
> > >  - add target install step of the optional qml files
> > >
> > > Reported-by: Nimai Mahajan <nimaim@gmail.com>
> > > Signed-off-by: Peter Seiderer <ps.report@gmx.net>
> >
> > Will you send a similar patch for master ?
> >
> > Thanks,
> >
> > Thomas
> >
>
> Thomas,
>
> Please wait for my Tested-by: :)
>
> Peter,
>
> This is not working for me after using your V2 patch. Perhaps it is my own
> user error. The patch simply seems to change some dependencies around
> (which looks correct now) and explicitly copies a XmlListModel folder into
> $(TARGET_DIR)/usr/qml/QtQuick, but there is nothing to copy, even with the
> qt5declarative and qt5xmlpatterns packages checked. For reference, here is
> everything mentioning xmllistmodel in the qt5 build directory:

In this case your should get a buildroot error for the copy command..., or
your .config is wrong..., or you did no re-build of the package?

>
> ~/Desktop/test_defconfig_output/build$ find . -iname "*xmllistmodel*"
> ./qt5declarative-5.12.7/src/quick/doc/images/qml-xmllistmodel-example.png
> ./qt5declarative-5.12.7/tests/testapplications/elements/content/XmlListModelElement.qml
> ./qt5xmlpatterns-5.12.7/src/imports/xmllistmodel
> ./qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/xmllistmodel.pro
> ./qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/qqmlxmllistmodel_p.h
> ./qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/qqmlxmllistmodel.cpp
> ./qt5xmlpatterns-5.12.7/tests/auto/qquickxmllistmodel
> ./qt5xmlpatterns-5.12.7/tests/auto/qquickxmllistmodel/qquickxmllistmodel.pro
> ./qt5xmlpatterns-5.12.7/tests/auto/qquickxmllistmodel/tst_qquickxmllistmodel.cpp
>
> It seems like the xmllistmodel.pro file there is indeed compiling it as a
> plugin, but there is no XmlListModel in $(STAGING_DIR)/usr/qml because
> again, it doesn't seem to be compiling it. I don't see anything in the
> xmlpatterns Makefile that explicitly goes into src/imports and compiles it.

The decision is made in the file build/qt5xmlpatterns-5.12.7/src/src.pro:

TEMPLATE = subdirs
SUBDIRS +=  xmlpatterns

qtHaveModule(qml){
    SUBDIRS += imports
    imports.depends = xmlpatterns
}

>
> Also, in the case there are more imports there in the future, should it not
> be compiling all the projects in build/qt5xmlpatterns-5.12.7/src/imports/
> imports.pro? This would compile all the projects listed in SUBDIRS there,
> and output the appropriate folder with it's .so, plugins.qmltypes, and
> qmldir, which then could be copied over.
>
> Let me know if I'm missing something.

Try a complete re-build (after make clean)...., in case of failure please
provide your .config/defconfig...

Regards,
Peter


Tested locally with the following defconfig:

BR2_arm=y
BR2_cortex_a53=y
BR2_ARM_FPU_NEON_VFPV4=y
BR2_TOOLCHAIN_BUILDROOT_GLIBC=y
BR2_PACKAGE_HOST_LINUX_HEADERS_CUSTOM_4_19=y
BR2_BINUTILS_VERSION_2_33_X=y
BR2_GCC_VERSION_9_X=y
BR2_TOOLCHAIN_BUILDROOT_CXX=y
BR2_SYSTEM_DHCP="eth0"
BR2_ROOTFS_POST_BUILD_SCRIPT="board/raspberrypi3/post-build.sh"
BR2_ROOTFS_POST_IMAGE_SCRIPT="board/raspberrypi3/post-image.sh"
BR2_ROOTFS_POST_SCRIPT_ARGS="--add-miniuart-bt-overlay"
BR2_LINUX_KERNEL=y
BR2_LINUX_KERNEL_CUSTOM_TARBALL=y
BR2_LINUX_KERNEL_CUSTOM_TARBALL_LOCATION="$(call github,raspberrypi,linux,64d0a9870ac14d5eb5253f67d984ae348eec1393)/linux-64d0a9870ac14d5eb5253f67d984ae348eec1393.tar.gz"
BR2_LINUX_KERNEL_DEFCONFIG="bcm2709"
BR2_LINUX_KERNEL_DTS_SUPPORT=y
BR2_LINUX_KERNEL_INTREE_DTS_NAME="bcm2710-rpi-3-b bcm2710-rpi-3-b-plus bcm2710-rpi-cm3"
BR2_LINUX_KERNEL_NEEDS_HOST_OPENSSL=y
BR2_PACKAGE_QT5=y
BR2_PACKAGE_QT5BASE_EGLFS=y
BR2_PACKAGE_QT5QUICKCONTROLS=y
BR2_PACKAGE_QT5QUICKCONTROLS2=y
BR2_PACKAGE_QT5XMLPATTERNS=y
BR2_PACKAGE_RPI_FIRMWARE=y
BR2_PACKAGE_RPI_USERLAND=y
BR2_TARGET_ROOTFS_EXT2=y
BR2_TARGET_ROOTFS_EXT2_4=y
BR2_TARGET_ROOTFS_EXT2_SIZE="120M"
# BR2_TARGET_ROOTFS_TAR is not set
BR2_PACKAGE_HOST_DOSFSTOOLS=y
BR2_PACKAGE_HOST_GENIMAGE=y
BR2_PACKAGE_HOST_MTOOLS=y

	$ find . -iname "*xmllistmodel*"
./build/qt5declarative-5.12.7/src/quick/doc/images/qml-xmllistmodel-example.png
./build/qt5declarative-5.12.7/tests/testapplications/elements/content/XmlListModelElement.qml
./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel
./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/qqmlxmllistmodel.cpp
./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/qqmlxmllistmodel_p.h
./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/xmllistmodel.pro
./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/.obj/moc_qqmlxmllistmodel_p.o
./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/.obj/qqmlxmllistmodel.o
./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/.moc/moc_qqmlxmllistmodel_p.cpp
./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/.moc/qqmlxmllistmodel.moc
./build/qt5xmlpatterns-5.12.7/tests/auto/qquickxmllistmodel
./build/qt5xmlpatterns-5.12.7/tests/auto/qquickxmllistmodel/qquickxmllistmodel.pro
./build/qt5xmlpatterns-5.12.7/tests/auto/qquickxmllistmodel/tst_qquickxmllistmodel.cpp
./build/qt5xmlpatterns-5.12.7/qml/QtQuick/XmlListModel
./build/qt5xmlpatterns-5.12.7/qml/QtQuick/XmlListModel/libqmlxmllistmodelplugin.so
./host/arm-buildroot-linux-gnueabihf/sysroot/usr/qml/QtQuick/XmlListModel
./host/arm-buildroot-linux-gnueabihf/sysroot/usr/qml/QtQuick/XmlListModel/libqmlxmllistmodelplugin.so
./target/usr/qml/QtQuick/XmlListModel
./target/usr/qml/QtQuick/XmlListModel/libqmlxmllistmodelplugin.so

>
> Thanks,
> Nimai
>
>
>
> > --
> > Thomas Petazzoni, CTO, Bootlin
> > Embedded Linux and Kernel engineering
> > https://bootlin.com
> >
Peter Seiderer March 14, 2020, 5:54 p.m. UTC | #4
Hello Thomas,

On Sat, 14 Mar 2020 15:18:23 +0100, Thomas Petazzoni <thomas.petazzoni@bootlin.com> wrote:

> Hello Peter,
>
> On Fri, 13 Mar 2020 21:44:15 +0100
> Peter Seiderer <ps.report@gmx.net> wrote:
>
> > qt5declarative:
> >  - remove unneded dependency on qt5mlpatterns
> >
> > qt5xmlpatterns:
> >  - add optional dependency on qt5declarative
> >  - add target install step of the optional qml files
> >
> > Reported-by: Nimai Mahajan <nimaim@gmail.com>
> > Signed-off-by: Peter Seiderer <ps.report@gmx.net>
>
> Will you send a similar patch for master ?

Yes, already prepared...., will send it out soon ;-)

Regards,
Peter

>
> Thanks,
>
> Thomas
Nimai Mahajan March 14, 2020, 6:04 p.m. UTC | #5
On Sat, Mar 14, 2020 at 1:53 PM Peter Seiderer <ps.report@gmx.net> wrote:

> Hello Nimai,
>
> On Sat, 14 Mar 2020 11:22:31 -0400, Nimai Mahajan <nimaim@gmail.com>
> wrote:
>
> > On Sat, Mar 14, 2020 at 10:18 AM Thomas Petazzoni <
> > thomas.petazzoni@bootlin.com> wrote:
> >
> > > Hello Peter,
> > >
> > > On Fri, 13 Mar 2020 21:44:15 +0100
> > > Peter Seiderer <ps.report@gmx.net> wrote:
> > >
> > > > qt5declarative:
> > > >  - remove unneded dependency on qt5mlpatterns
> > > >
> > > > qt5xmlpatterns:
> > > >  - add optional dependency on qt5declarative
> > > >  - add target install step of the optional qml files
> > > >
> > > > Reported-by: Nimai Mahajan <nimaim@gmail.com>
> > > > Signed-off-by: Peter Seiderer <ps.report@gmx.net>
> > >
> > > Will you send a similar patch for master ?
> > >
> > > Thanks,
> > >
> > > Thomas
> > >
> >
> > Thomas,
> >
> > Please wait for my Tested-by: :)
> >
> > Peter,
> >
> > This is not working for me after using your V2 patch. Perhaps it is my
> own
> > user error. The patch simply seems to change some dependencies around
> > (which looks correct now) and explicitly copies a XmlListModel folder
> into
> > $(TARGET_DIR)/usr/qml/QtQuick, but there is nothing to copy, even with
> the
> > qt5declarative and qt5xmlpatterns packages checked. For reference, here
> is
> > everything mentioning xmllistmodel in the qt5 build directory:
>
> In this case your should get a buildroot error for the copy command..., or
> your .config is wrong..., or you did no re-build of the package?
>

I rebuilt from scratch after patching, so definitely is strange why I did
not get output like yours below. But I will wipe out the entire project,
re-clone, patch, and try again.

>
> >
> > ~/Desktop/test_defconfig_output/build$ find . -iname "*xmllistmodel*"
> > ./qt5declarative-5.12.7/src/quick/doc/images/qml-xmllistmodel-example.png
> >
> ./qt5declarative-5.12.7/tests/testapplications/elements/content/XmlListModelElement.qml
> > ./qt5xmlpatterns-5.12.7/src/imports/xmllistmodel
> > ./qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/xmllistmodel.pro
> > ./qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/qqmlxmllistmodel_p.h
> > ./qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/qqmlxmllistmodel.cpp
> > ./qt5xmlpatterns-5.12.7/tests/auto/qquickxmllistmodel
> > ./qt5xmlpatterns-5.12.7/tests/auto/qquickxmllistmodel/
> qquickxmllistmodel.pro
> >
> ./qt5xmlpatterns-5.12.7/tests/auto/qquickxmllistmodel/tst_qquickxmllistmodel.cpp
> >
> > It seems like the xmllistmodel.pro file there is indeed compiling it as
> a
> > plugin, but there is no XmlListModel in $(STAGING_DIR)/usr/qml because
> > again, it doesn't seem to be compiling it. I don't see anything in the
> > xmlpatterns Makefile that explicitly goes into src/imports and compiles
> it.
>
> The decision is made in the file build/qt5xmlpatterns-5.12.7/src/src.pro:
>
> TEMPLATE = subdirs
> SUBDIRS +=  xmlpatterns
>
> qtHaveModule(qml){
>     SUBDIRS += imports
>     imports.depends = xmlpatterns
> }
>
Ahh I see, thanks!


>
> >
> > Also, in the case there are more imports there in the future, should it
> not
> > be compiling all the projects in build/qt5xmlpatterns-5.12.7/src/imports/
> > imports.pro? This would compile all the projects listed in SUBDIRS
> there,
> > and output the appropriate folder with it's .so, plugins.qmltypes, and
> > qmldir, which then could be copied over.
> >
> > Let me know if I'm missing something.
>
> Try a complete re-build (after make clean)...., in case of failure please
> provide your .config/defconfig...
>

Already provided here:
http://buildroot-busybox.2317881.n4.nabble.com/2020-02-Buildroot-Qt-5-12-missing-XmlListModel-tp249854p249918.html
.

>
> Regards,
> Peter
>
>
> Tested locally with the following defconfig:
>
> BR2_arm=y
> BR2_cortex_a53=y
> BR2_ARM_FPU_NEON_VFPV4=y
> BR2_TOOLCHAIN_BUILDROOT_GLIBC=y
> BR2_PACKAGE_HOST_LINUX_HEADERS_CUSTOM_4_19=y
> BR2_BINUTILS_VERSION_2_33_X=y
> BR2_GCC_VERSION_9_X=y
> BR2_TOOLCHAIN_BUILDROOT_CXX=y
> BR2_SYSTEM_DHCP="eth0"
> BR2_ROOTFS_POST_BUILD_SCRIPT="board/raspberrypi3/post-build.sh"
> BR2_ROOTFS_POST_IMAGE_SCRIPT="board/raspberrypi3/post-image.sh"
> BR2_ROOTFS_POST_SCRIPT_ARGS="--add-miniuart-bt-overlay"
> BR2_LINUX_KERNEL=y
> BR2_LINUX_KERNEL_CUSTOM_TARBALL=y
> BR2_LINUX_KERNEL_CUSTOM_TARBALL_LOCATION="$(call
> github,raspberrypi,linux,64d0a9870ac14d5eb5253f67d984ae348eec1393)/linux-64d0a9870ac14d5eb5253f67d984ae348eec1393.tar.gz"
> BR2_LINUX_KERNEL_DEFCONFIG="bcm2709"
> BR2_LINUX_KERNEL_DTS_SUPPORT=y
> BR2_LINUX_KERNEL_INTREE_DTS_NAME="bcm2710-rpi-3-b bcm2710-rpi-3-b-plus
> bcm2710-rpi-cm3"
> BR2_LINUX_KERNEL_NEEDS_HOST_OPENSSL=y
> BR2_PACKAGE_QT5=y
> BR2_PACKAGE_QT5BASE_EGLFS=y
> BR2_PACKAGE_QT5QUICKCONTROLS=y
> BR2_PACKAGE_QT5QUICKCONTROLS2=y
> BR2_PACKAGE_QT5XMLPATTERNS=y
> BR2_PACKAGE_RPI_FIRMWARE=y
> BR2_PACKAGE_RPI_USERLAND=y
> BR2_TARGET_ROOTFS_EXT2=y
> BR2_TARGET_ROOTFS_EXT2_4=y
> BR2_TARGET_ROOTFS_EXT2_SIZE="120M"
> # BR2_TARGET_ROOTFS_TAR is not set
> BR2_PACKAGE_HOST_DOSFSTOOLS=y
> BR2_PACKAGE_HOST_GENIMAGE=y
> BR2_PACKAGE_HOST_MTOOLS=y
>
>         $ find . -iname "*xmllistmodel*"
>
> ./build/qt5declarative-5.12.7/src/quick/doc/images/qml-xmllistmodel-example.png
>
> ./build/qt5declarative-5.12.7/tests/testapplications/elements/content/XmlListModelElement.qml
> ./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel
> ./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/qqmlxmllistmodel.cpp
> ./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/qqmlxmllistmodel_p.h
> ./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/xmllistmodel.pro
>
> ./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/.obj/moc_qqmlxmllistmodel_p.o
>
> ./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/.obj/qqmlxmllistmodel.o
>
> ./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/.moc/moc_qqmlxmllistmodel_p.cpp
>
> ./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/.moc/qqmlxmllistmodel.moc
> ./build/qt5xmlpatterns-5.12.7/tests/auto/qquickxmllistmodel
> ./build/qt5xmlpatterns-5.12.7/tests/auto/qquickxmllistmodel/
> qquickxmllistmodel.pro
>
> ./build/qt5xmlpatterns-5.12.7/tests/auto/qquickxmllistmodel/tst_qquickxmllistmodel.cpp
> ./build/qt5xmlpatterns-5.12.7/qml/QtQuick/XmlListModel
>
> ./build/qt5xmlpatterns-5.12.7/qml/QtQuick/XmlListModel/libqmlxmllistmodelplugin.so
> ./host/arm-buildroot-linux-gnueabihf/sysroot/usr/qml/QtQuick/XmlListModel
>
> ./host/arm-buildroot-linux-gnueabihf/sysroot/usr/qml/QtQuick/XmlListModel/libqmlxmllistmodelplugin.so
> ./target/usr/qml/QtQuick/XmlListModel
> ./target/usr/qml/QtQuick/XmlListModel/libqmlxmllistmodelplugin.so
>
> >
> > Thanks,
> > Nimai
> >
> >
> >
> > > --
> > > Thomas Petazzoni, CTO, Bootlin
> > > Embedded Linux and Kernel engineering
> > > https://bootlin.com
> > >
>
>
Peter Seiderer March 14, 2020, 6:14 p.m. UTC | #6
On Sat, 14 Mar 2020 14:04:30 -0400, Nimai Mahajan <nimaim@gmail.com> wrote:

> On Sat, Mar 14, 2020 at 1:53 PM Peter Seiderer <ps.report@gmx.net> wrote:
>
> > Hello Nimai,
> >
> > On Sat, 14 Mar 2020 11:22:31 -0400, Nimai Mahajan <nimaim@gmail.com>
> > wrote:
> >
> > > On Sat, Mar 14, 2020 at 10:18 AM Thomas Petazzoni <
> > > thomas.petazzoni@bootlin.com> wrote:
> > >
> > > > Hello Peter,
> > > >
> > > > On Fri, 13 Mar 2020 21:44:15 +0100
> > > > Peter Seiderer <ps.report@gmx.net> wrote:
> > > >
> > > > > qt5declarative:
> > > > >  - remove unneded dependency on qt5mlpatterns
> > > > >
> > > > > qt5xmlpatterns:
> > > > >  - add optional dependency on qt5declarative
> > > > >  - add target install step of the optional qml files
> > > > >
> > > > > Reported-by: Nimai Mahajan <nimaim@gmail.com>
> > > > > Signed-off-by: Peter Seiderer <ps.report@gmx.net>
> > > >
> > > > Will you send a similar patch for master ?
> > > >
> > > > Thanks,
> > > >
> > > > Thomas
> > > >
> > >
> > > Thomas,
> > >
> > > Please wait for my Tested-by: :)
> > >
> > > Peter,
> > >
> > > This is not working for me after using your V2 patch. Perhaps it is my
> > own
> > > user error. The patch simply seems to change some dependencies around
> > > (which looks correct now) and explicitly copies a XmlListModel folder
> > into
> > > $(TARGET_DIR)/usr/qml/QtQuick, but there is nothing to copy, even with
> > the
> > > qt5declarative and qt5xmlpatterns packages checked. For reference, here
> > is
> > > everything mentioning xmllistmodel in the qt5 build directory:
> >
> > In this case your should get a buildroot error for the copy command..., or
> > your .config is wrong..., or you did no re-build of the package?
> >
>
> I rebuilt from scratch after patching, so definitely is strange why I did
> not get output like yours below. But I will wipe out the entire project,
> re-clone, patch, and try again.
>
> >
> > >
> > > ~/Desktop/test_defconfig_output/build$ find . -iname "*xmllistmodel*"
> > > ./qt5declarative-5.12.7/src/quick/doc/images/qml-xmllistmodel-example.png
> > >
> > ./qt5declarative-5.12.7/tests/testapplications/elements/content/XmlListModelElement.qml
> > > ./qt5xmlpatterns-5.12.7/src/imports/xmllistmodel
> > > ./qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/xmllistmodel.pro
> > > ./qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/qqmlxmllistmodel_p.h
> > > ./qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/qqmlxmllistmodel.cpp
> > > ./qt5xmlpatterns-5.12.7/tests/auto/qquickxmllistmodel
> > > ./qt5xmlpatterns-5.12.7/tests/auto/qquickxmllistmodel/
> > qquickxmllistmodel.pro
> > >
> > ./qt5xmlpatterns-5.12.7/tests/auto/qquickxmllistmodel/tst_qquickxmllistmodel.cpp
> > >
> > > It seems like the xmllistmodel.pro file there is indeed compiling it as
> > a
> > > plugin, but there is no XmlListModel in $(STAGING_DIR)/usr/qml because
> > > again, it doesn't seem to be compiling it. I don't see anything in the
> > > xmlpatterns Makefile that explicitly goes into src/imports and compiles
> > it.
> >
> > The decision is made in the file build/qt5xmlpatterns-5.12.7/src/src.pro:
> >
> > TEMPLATE = subdirs
> > SUBDIRS +=  xmlpatterns
> >
> > qtHaveModule(qml){
> >     SUBDIRS += imports
> >     imports.depends = xmlpatterns
> > }
> >
> Ahh I see, thanks!
>
>
> >
> > >
> > > Also, in the case there are more imports there in the future, should it
> > not
> > > be compiling all the projects in build/qt5xmlpatterns-5.12.7/src/imports/
> > > imports.pro? This would compile all the projects listed in SUBDIRS
> > there,
> > > and output the appropriate folder with it's .so, plugins.qmltypes, and
> > > qmldir, which then could be copied over.
> > >
> > > Let me know if I'm missing something.
> >
> > Try a complete re-build (after make clean)...., in case of failure please
> > provide your .config/defconfig...
> >
>
> Already provided here:
> http://buildroot-busybox.2317881.n4.nabble.com/2020-02-Buildroot-Qt-5-12-missing-XmlListModel-tp249854p249918.html

No plain config,...dependecies to $(BR2_EXTERNAL), and:

BR2_PACKAGE_QT5=y
BR2_PACKAGE_QT5BASE_CONCURRENT=y
BR2_PACKAGE_QT5BASE_OPENGL_LIB=y
BR2_PACKAGE_QT5BASE_LINUXFB=y
BR2_PACKAGE_QT5BASE_XCB=y
BR2_PACKAGE_QT5BASE_DEFAULT_QPA="xcb"
BR2_PACKAGE_QT5BASE_TSLIB=y
BR2_PACKAGE_QT5SERIALBUS=y
BR2_PACKAGE_QT5SVG=y


And the following is not selected:

BR2_PACKAGE_QT5DECLARATIVE=y
BR2_PACKAGE_QT5DECLARATIVE_QUICK=y
BR2_PACKAGE_QT5QUICKCONTROLS=y
BR2_PACKAGE_QT5QUICKCONTROLS2=y
BR2_PACKAGE_QT5XMLPATTERNS=y

Try to provide a plain .config against 2020.02..., without externals...

Regards,
Peter

>
>
> >
> > Regards,
> > Peter
> >
> >
> > Tested locally with the following defconfig:
> >
> > BR2_arm=y
> > BR2_cortex_a53=y
> > BR2_ARM_FPU_NEON_VFPV4=y
> > BR2_TOOLCHAIN_BUILDROOT_GLIBC=y
> > BR2_PACKAGE_HOST_LINUX_HEADERS_CUSTOM_4_19=y
> > BR2_BINUTILS_VERSION_2_33_X=y
> > BR2_GCC_VERSION_9_X=y
> > BR2_TOOLCHAIN_BUILDROOT_CXX=y
> > BR2_SYSTEM_DHCP="eth0"
> > BR2_ROOTFS_POST_BUILD_SCRIPT="board/raspberrypi3/post-build.sh"
> > BR2_ROOTFS_POST_IMAGE_SCRIPT="board/raspberrypi3/post-image.sh"
> > BR2_ROOTFS_POST_SCRIPT_ARGS="--add-miniuart-bt-overlay"
> > BR2_LINUX_KERNEL=y
> > BR2_LINUX_KERNEL_CUSTOM_TARBALL=y
> > BR2_LINUX_KERNEL_CUSTOM_TARBALL_LOCATION="$(call
> > github,raspberrypi,linux,64d0a9870ac14d5eb5253f67d984ae348eec1393)/linux-64d0a9870ac14d5eb5253f67d984ae348eec1393.tar.gz"
> > BR2_LINUX_KERNEL_DEFCONFIG="bcm2709"
> > BR2_LINUX_KERNEL_DTS_SUPPORT=y
> > BR2_LINUX_KERNEL_INTREE_DTS_NAME="bcm2710-rpi-3-b bcm2710-rpi-3-b-plus
> > bcm2710-rpi-cm3"
> > BR2_LINUX_KERNEL_NEEDS_HOST_OPENSSL=y
> > BR2_PACKAGE_QT5=y
> > BR2_PACKAGE_QT5BASE_EGLFS=y
> > BR2_PACKAGE_QT5QUICKCONTROLS=y
> > BR2_PACKAGE_QT5QUICKCONTROLS2=y
> > BR2_PACKAGE_QT5XMLPATTERNS=y
> > BR2_PACKAGE_RPI_FIRMWARE=y
> > BR2_PACKAGE_RPI_USERLAND=y
> > BR2_TARGET_ROOTFS_EXT2=y
> > BR2_TARGET_ROOTFS_EXT2_4=y
> > BR2_TARGET_ROOTFS_EXT2_SIZE="120M"
> > # BR2_TARGET_ROOTFS_TAR is not set
> > BR2_PACKAGE_HOST_DOSFSTOOLS=y
> > BR2_PACKAGE_HOST_GENIMAGE=y
> > BR2_PACKAGE_HOST_MTOOLS=y
> >
> >         $ find . -iname "*xmllistmodel*"
> >
> > ./build/qt5declarative-5.12.7/src/quick/doc/images/qml-xmllistmodel-example.png
> >
> > ./build/qt5declarative-5.12.7/tests/testapplications/elements/content/XmlListModelElement.qml
> > ./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel
> > ./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/qqmlxmllistmodel.cpp
> > ./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/qqmlxmllistmodel_p.h
> > ./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/xmllistmodel.pro
> >
> > ./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/.obj/moc_qqmlxmllistmodel_p.o
> >
> > ./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/.obj/qqmlxmllistmodel.o
> >
> > ./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/.moc/moc_qqmlxmllistmodel_p.cpp
> >
> > ./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/.moc/qqmlxmllistmodel.moc
> > ./build/qt5xmlpatterns-5.12.7/tests/auto/qquickxmllistmodel
> > ./build/qt5xmlpatterns-5.12.7/tests/auto/qquickxmllistmodel/
> > qquickxmllistmodel.pro
> >
> > ./build/qt5xmlpatterns-5.12.7/tests/auto/qquickxmllistmodel/tst_qquickxmllistmodel.cpp
> > ./build/qt5xmlpatterns-5.12.7/qml/QtQuick/XmlListModel
> >
> > ./build/qt5xmlpatterns-5.12.7/qml/QtQuick/XmlListModel/libqmlxmllistmodelplugin.so
> > ./host/arm-buildroot-linux-gnueabihf/sysroot/usr/qml/QtQuick/XmlListModel
> >
> > ./host/arm-buildroot-linux-gnueabihf/sysroot/usr/qml/QtQuick/XmlListModel/libqmlxmllistmodelplugin.so
> > ./target/usr/qml/QtQuick/XmlListModel
> > ./target/usr/qml/QtQuick/XmlListModel/libqmlxmllistmodelplugin.so
> >
> > >
> > > Thanks,
> > > Nimai
> > >
> > >
> > >
> > > > --
> > > > Thomas Petazzoni, CTO, Bootlin
> > > > Embedded Linux and Kernel engineering
> > > > https://bootlin.com
> > > >
> >
> >
Nimai Mahajan March 14, 2020, 11:58 p.m. UTC | #7
On Sat, Mar 14, 2020 at 2:14 PM Peter Seiderer <ps.report@gmx.net> wrote:

> On Sat, 14 Mar 2020 14:04:30 -0400, Nimai Mahajan <nimaim@gmail.com>
> wrote:
>
> > On Sat, Mar 14, 2020 at 1:53 PM Peter Seiderer <ps.report@gmx.net>
> wrote:
> >
> > > Hello Nimai,
> > >
> > > On Sat, 14 Mar 2020 11:22:31 -0400, Nimai Mahajan <nimaim@gmail.com>
> > > wrote:
> > >
> > > > On Sat, Mar 14, 2020 at 10:18 AM Thomas Petazzoni <
> > > > thomas.petazzoni@bootlin.com> wrote:
> > > >
> > > > > Hello Peter,
> > > > >
> > > > > On Fri, 13 Mar 2020 21:44:15 +0100
> > > > > Peter Seiderer <ps.report@gmx.net> wrote:
> > > > >
> > > > > > qt5declarative:
> > > > > >  - remove unneded dependency on qt5mlpatterns
> > > > > >
> > > > > > qt5xmlpatterns:
> > > > > >  - add optional dependency on qt5declarative
> > > > > >  - add target install step of the optional qml files
> > > > > >
> > > > > > Reported-by: Nimai Mahajan <nimaim@gmail.com>
> > > > > > Signed-off-by: Peter Seiderer <ps.report@gmx.net>
> > > > >
> > > > > Will you send a similar patch for master ?
> > > > >
> > > > > Thanks,
> > > > >
> > > > > Thomas
> > > > >
> > > >
> > > > Thomas,
> > > >
> > > > Please wait for my Tested-by: :)
> > > >
> > > > Peter,
> > > >
> > > > This is not working for me after using your V2 patch. Perhaps it is
> my
> > > own
> > > > user error. The patch simply seems to change some dependencies around
> > > > (which looks correct now) and explicitly copies a XmlListModel folder
> > > into
> > > > $(TARGET_DIR)/usr/qml/QtQuick, but there is nothing to copy, even
> with
> > > the
> > > > qt5declarative and qt5xmlpatterns packages checked. For reference,
> here
> > > is
> > > > everything mentioning xmllistmodel in the qt5 build directory:
> > >
> > > In this case your should get a buildroot error for the copy
> command..., or
> > > your .config is wrong..., or you did no re-build of the package?
> > >
> >
> > I rebuilt from scratch after patching, so definitely is strange why I did
> > not get output like yours below. But I will wipe out the entire project,
> > re-clone, patch, and try again.
> >
> > >
> > > >
> > > > ~/Desktop/test_defconfig_output/build$ find . -iname "*xmllistmodel*"
> > > >
> ./qt5declarative-5.12.7/src/quick/doc/images/qml-xmllistmodel-example.png
> > > >
> > >
> ./qt5declarative-5.12.7/tests/testapplications/elements/content/XmlListModelElement.qml
> > > > ./qt5xmlpatterns-5.12.7/src/imports/xmllistmodel
> > > > ./qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/xmllistmodel.pro
> > > > ./qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/qqmlxmllistmodel_p.h
> > > > ./qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/qqmlxmllistmodel.cpp
> > > > ./qt5xmlpatterns-5.12.7/tests/auto/qquickxmllistmodel
> > > > ./qt5xmlpatterns-5.12.7/tests/auto/qquickxmllistmodel/
> > > qquickxmllistmodel.pro
> > > >
> > >
> ./qt5xmlpatterns-5.12.7/tests/auto/qquickxmllistmodel/tst_qquickxmllistmodel.cpp
> > > >
> > > > It seems like the xmllistmodel.pro file there is indeed compiling
> it as
> > > a
> > > > plugin, but there is no XmlListModel in $(STAGING_DIR)/usr/qml
> because
> > > > again, it doesn't seem to be compiling it. I don't see anything in
> the
> > > > xmlpatterns Makefile that explicitly goes into src/imports and
> compiles
> > > it.
> > >
> > > The decision is made in the file build/qt5xmlpatterns-5.12.7/src/
> src.pro:
> > >
> > > TEMPLATE = subdirs
> > > SUBDIRS +=  xmlpatterns
> > >
> > > qtHaveModule(qml){
> > >     SUBDIRS += imports
> > >     imports.depends = xmlpatterns
> > > }
> > >
> > Ahh I see, thanks!
> >
> >
> > >
> > > >
> > > > Also, in the case there are more imports there in the future, should
> it
> > > not
> > > > be compiling all the projects in
> build/qt5xmlpatterns-5.12.7/src/imports/
> > > > imports.pro? This would compile all the projects listed in SUBDIRS
> > > there,
> > > > and output the appropriate folder with it's .so, plugins.qmltypes,
> and
> > > > qmldir, which then could be copied over.
> > > >
> > > > Let me know if I'm missing something.
> > >
> > > Try a complete re-build (after make clean)...., in case of failure
> please
> > > provide your .config/defconfig...
> > >
> >
> > Already provided here:
> >
> http://buildroot-busybox.2317881.n4.nabble.com/2020-02-Buildroot-Qt-5-12-missing-XmlListModel-tp249854p249918.html
>
> No plain config,...dependecies to $(BR2_EXTERNAL), and:
>
> BR2_PACKAGE_QT5=y
> BR2_PACKAGE_QT5BASE_CONCURRENT=y
> BR2_PACKAGE_QT5BASE_OPENGL_LIB=y
> BR2_PACKAGE_QT5BASE_LINUXFB=y
> BR2_PACKAGE_QT5BASE_XCB=y
> BR2_PACKAGE_QT5BASE_DEFAULT_QPA="xcb"
> BR2_PACKAGE_QT5BASE_TSLIB=y
> BR2_PACKAGE_QT5SERIALBUS=y
> BR2_PACKAGE_QT5SVG=y
>
>
> And the following is not selected:
>
> BR2_PACKAGE_QT5DECLARATIVE=y
> BR2_PACKAGE_QT5DECLARATIVE_QUICK=y
> BR2_PACKAGE_QT5QUICKCONTROLS=y
> BR2_PACKAGE_QT5QUICKCONTROLS2=y
> BR2_PACKAGE_QT5XMLPATTERNS=y
>
> Try to provide a plain .config against 2020.02..., without externals...
>
> Regards,
> Peter
>

Thanks for your patience Peter. All is good, my application DOES indeed
work. I nuked my previous config, ccache, etc. and started from scratch
cloning a fresh copy of BR 2020.02 and redoing my config and now it
correctly builds. The module is there and loads with my old import
statement.

>
> >
> >
> > >
> > > Regards,
> > > Peter
> > >
> > >
> > > Tested locally with the following defconfig:
> > >
> > > BR2_arm=y
> > > BR2_cortex_a53=y
> > > BR2_ARM_FPU_NEON_VFPV4=y
> > > BR2_TOOLCHAIN_BUILDROOT_GLIBC=y
> > > BR2_PACKAGE_HOST_LINUX_HEADERS_CUSTOM_4_19=y
> > > BR2_BINUTILS_VERSION_2_33_X=y
> > > BR2_GCC_VERSION_9_X=y
> > > BR2_TOOLCHAIN_BUILDROOT_CXX=y
> > > BR2_SYSTEM_DHCP="eth0"
> > > BR2_ROOTFS_POST_BUILD_SCRIPT="board/raspberrypi3/post-build.sh"
> > > BR2_ROOTFS_POST_IMAGE_SCRIPT="board/raspberrypi3/post-image.sh"
> > > BR2_ROOTFS_POST_SCRIPT_ARGS="--add-miniuart-bt-overlay"
> > > BR2_LINUX_KERNEL=y
> > > BR2_LINUX_KERNEL_CUSTOM_TARBALL=y
> > > BR2_LINUX_KERNEL_CUSTOM_TARBALL_LOCATION="$(call
> > >
> github,raspberrypi,linux,64d0a9870ac14d5eb5253f67d984ae348eec1393)/linux-64d0a9870ac14d5eb5253f67d984ae348eec1393.tar.gz"
> > > BR2_LINUX_KERNEL_DEFCONFIG="bcm2709"
> > > BR2_LINUX_KERNEL_DTS_SUPPORT=y
> > > BR2_LINUX_KERNEL_INTREE_DTS_NAME="bcm2710-rpi-3-b bcm2710-rpi-3-b-plus
> > > bcm2710-rpi-cm3"
> > > BR2_LINUX_KERNEL_NEEDS_HOST_OPENSSL=y
> > > BR2_PACKAGE_QT5=y
> > > BR2_PACKAGE_QT5BASE_EGLFS=y
> > > BR2_PACKAGE_QT5QUICKCONTROLS=y
> > > BR2_PACKAGE_QT5QUICKCONTROLS2=y
> > > BR2_PACKAGE_QT5XMLPATTERNS=y
> > > BR2_PACKAGE_RPI_FIRMWARE=y
> > > BR2_PACKAGE_RPI_USERLAND=y
> > > BR2_TARGET_ROOTFS_EXT2=y
> > > BR2_TARGET_ROOTFS_EXT2_4=y
> > > BR2_TARGET_ROOTFS_EXT2_SIZE="120M"
> > > # BR2_TARGET_ROOTFS_TAR is not set
> > > BR2_PACKAGE_HOST_DOSFSTOOLS=y
> > > BR2_PACKAGE_HOST_GENIMAGE=y
> > > BR2_PACKAGE_HOST_MTOOLS=y
> > >
> > >         $ find . -iname "*xmllistmodel*"
> > >
> > >
> ./build/qt5declarative-5.12.7/src/quick/doc/images/qml-xmllistmodel-example.png
> > >
> > >
> ./build/qt5declarative-5.12.7/tests/testapplications/elements/content/XmlListModelElement.qml
> > > ./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel
> > >
> ./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/qqmlxmllistmodel.cpp
> > >
> ./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/qqmlxmllistmodel_p.h
> > > ./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/
> xmllistmodel.pro
> > >
> > >
> ./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/.obj/moc_qqmlxmllistmodel_p.o
> > >
> > >
> ./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/.obj/qqmlxmllistmodel.o
> > >
> > >
> ./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/.moc/moc_qqmlxmllistmodel_p.cpp
> > >
> > >
> ./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/.moc/qqmlxmllistmodel.moc
> > > ./build/qt5xmlpatterns-5.12.7/tests/auto/qquickxmllistmodel
> > > ./build/qt5xmlpatterns-5.12.7/tests/auto/qquickxmllistmodel/
> > > qquickxmllistmodel.pro
> > >
> > >
> ./build/qt5xmlpatterns-5.12.7/tests/auto/qquickxmllistmodel/tst_qquickxmllistmodel.cpp
> > > ./build/qt5xmlpatterns-5.12.7/qml/QtQuick/XmlListModel
> > >
> > >
> ./build/qt5xmlpatterns-5.12.7/qml/QtQuick/XmlListModel/libqmlxmllistmodelplugin.so
> > >
> ./host/arm-buildroot-linux-gnueabihf/sysroot/usr/qml/QtQuick/XmlListModel
> > >
> > >
> ./host/arm-buildroot-linux-gnueabihf/sysroot/usr/qml/QtQuick/XmlListModel/libqmlxmllistmodelplugin.so
> > > ./target/usr/qml/QtQuick/XmlListModel
> > > ./target/usr/qml/QtQuick/XmlListModel/libqmlxmllistmodelplugin.so
> > >
> > > >
> > > > Thanks,
> > > > Nimai
> > > >
> > > >
> > > >
> > > > > --
> > > > > Thomas Petazzoni, CTO, Bootlin
> > > > > Embedded Linux and Kernel engineering
> > > > > https://bootlin.com
> > > > >
> > >
> > >
>
>
Nimai Mahajan March 15, 2020, 12:04 a.m. UTC | #8
On Sat, Mar 14, 2020 at 7:58 PM Nimai Mahajan <nimaim@gmail.com> wrote:

>
> On Sat, Mar 14, 2020 at 2:14 PM Peter Seiderer <ps.report@gmx.net> wrote:
>
>> On Sat, 14 Mar 2020 14:04:30 -0400, Nimai Mahajan <nimaim@gmail.com>
>> wrote:
>>
>> > On Sat, Mar 14, 2020 at 1:53 PM Peter Seiderer <ps.report@gmx.net>
>> wrote:
>> >
>> > > Hello Nimai,
>> > >
>> > > On Sat, 14 Mar 2020 11:22:31 -0400, Nimai Mahajan <nimaim@gmail.com>
>> > > wrote:
>> > >
>> > > > On Sat, Mar 14, 2020 at 10:18 AM Thomas Petazzoni <
>> > > > thomas.petazzoni@bootlin.com> wrote:
>> > > >
>> > > > > Hello Peter,
>> > > > >
>> > > > > On Fri, 13 Mar 2020 21:44:15 +0100
>> > > > > Peter Seiderer <ps.report@gmx.net> wrote:
>> > > > >
>> > > > > > qt5declarative:
>> > > > > >  - remove unneded dependency on qt5mlpatterns
>> > > > > >
>> > > > > > qt5xmlpatterns:
>> > > > > >  - add optional dependency on qt5declarative
>> > > > > >  - add target install step of the optional qml files
>> > > > > >
>> > > > > > Reported-by: Nimai Mahajan <nimaim@gmail.com>
>> > > > > > Signed-off-by: Peter Seiderer <ps.report@gmx.net>
>> > > > >
>> > > > > Will you send a similar patch for master ?
>> > > > >
>> > > > > Thanks,
>> > > > >
>> > > > > Thomas
>> > > > >
>> > > >
>> > > > Thomas,
>> > > >
>> > > > Please wait for my Tested-by: :)
>> > > >
>> > > > Peter,
>> > > >
>> > > > This is not working for me after using your V2 patch. Perhaps it is
>> my
>> > > own
>> > > > user error. The patch simply seems to change some dependencies
>> around
>> > > > (which looks correct now) and explicitly copies a XmlListModel
>> folder
>> > > into
>> > > > $(TARGET_DIR)/usr/qml/QtQuick, but there is nothing to copy, even
>> with
>> > > the
>> > > > qt5declarative and qt5xmlpatterns packages checked. For reference,
>> here
>> > > is
>> > > > everything mentioning xmllistmodel in the qt5 build directory:
>> > >
>> > > In this case your should get a buildroot error for the copy
>> command..., or
>> > > your .config is wrong..., or you did no re-build of the package?
>> > >
>> >
>> > I rebuilt from scratch after patching, so definitely is strange why I
>> did
>> > not get output like yours below. But I will wipe out the entire project,
>> > re-clone, patch, and try again.
>> >
>> > >
>> > > >
>> > > > ~/Desktop/test_defconfig_output/build$ find . -iname
>> "*xmllistmodel*"
>> > > >
>> ./qt5declarative-5.12.7/src/quick/doc/images/qml-xmllistmodel-example.png
>> > > >
>> > >
>> ./qt5declarative-5.12.7/tests/testapplications/elements/content/XmlListModelElement.qml
>> > > > ./qt5xmlpatterns-5.12.7/src/imports/xmllistmodel
>> > > > ./qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/xmllistmodel.pro
>> > > >
>> ./qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/qqmlxmllistmodel_p.h
>> > > >
>> ./qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/qqmlxmllistmodel.cpp
>> > > > ./qt5xmlpatterns-5.12.7/tests/auto/qquickxmllistmodel
>> > > > ./qt5xmlpatterns-5.12.7/tests/auto/qquickxmllistmodel/
>> > > qquickxmllistmodel.pro
>> > > >
>> > >
>> ./qt5xmlpatterns-5.12.7/tests/auto/qquickxmllistmodel/tst_qquickxmllistmodel.cpp
>> > > >
>> > > > It seems like the xmllistmodel.pro file there is indeed compiling
>> it as
>> > > a
>> > > > plugin, but there is no XmlListModel in $(STAGING_DIR)/usr/qml
>> because
>> > > > again, it doesn't seem to be compiling it. I don't see anything in
>> the
>> > > > xmlpatterns Makefile that explicitly goes into src/imports and
>> compiles
>> > > it.
>> > >
>> > > The decision is made in the file build/qt5xmlpatterns-5.12.7/src/
>> src.pro:
>> > >
>> > > TEMPLATE = subdirs
>> > > SUBDIRS +=  xmlpatterns
>> > >
>> > > qtHaveModule(qml){
>> > >     SUBDIRS += imports
>> > >     imports.depends = xmlpatterns
>> > > }
>> > >
>> > Ahh I see, thanks!
>> >
>> >
>> > >
>> > > >
>> > > > Also, in the case there are more imports there in the future,
>> should it
>> > > not
>> > > > be compiling all the projects in
>> build/qt5xmlpatterns-5.12.7/src/imports/
>> > > > imports.pro? This would compile all the projects listed in SUBDIRS
>> > > there,
>> > > > and output the appropriate folder with it's .so, plugins.qmltypes,
>> and
>> > > > qmldir, which then could be copied over.
>> > > >
>> > > > Let me know if I'm missing something.
>> > >
>> > > Try a complete re-build (after make clean)...., in case of failure
>> please
>> > > provide your .config/defconfig...
>> > >
>> >
>> > Already provided here:
>> >
>> http://buildroot-busybox.2317881.n4.nabble.com/2020-02-Buildroot-Qt-5-12-missing-XmlListModel-tp249854p249918.html
>>
>> No plain config,...dependecies to $(BR2_EXTERNAL), and:
>>
>> BR2_PACKAGE_QT5=y
>> BR2_PACKAGE_QT5BASE_CONCURRENT=y
>> BR2_PACKAGE_QT5BASE_OPENGL_LIB=y
>> BR2_PACKAGE_QT5BASE_LINUXFB=y
>> BR2_PACKAGE_QT5BASE_XCB=y
>> BR2_PACKAGE_QT5BASE_DEFAULT_QPA="xcb"
>> BR2_PACKAGE_QT5BASE_TSLIB=y
>> BR2_PACKAGE_QT5SERIALBUS=y
>> BR2_PACKAGE_QT5SVG=y
>>
>>
>> And the following is not selected:
>>
>> BR2_PACKAGE_QT5DECLARATIVE=y
>> BR2_PACKAGE_QT5DECLARATIVE_QUICK=y
>> BR2_PACKAGE_QT5QUICKCONTROLS=y
>> BR2_PACKAGE_QT5QUICKCONTROLS2=y
>> BR2_PACKAGE_QT5XMLPATTERNS=y
>>
>> Try to provide a plain .config against 2020.02..., without externals...
>>
>> Regards,
>> Peter
>>
>
> Thanks for your patience Peter. All is good, my application DOES indeed
> work. I nuked my previous config, ccache, etc. and started from scratch
> cloning a fresh copy of BR 2020.02 and redoing my config and now it
> correctly builds. The module is there and loads with my old import
> statement.
>

Tested-by: Nimai Mahajan <nimaim@gmail.com>


>> >
>> >
>> > >
>> > > Regards,
>> > > Peter
>> > >
>> > >
>> > > Tested locally with the following defconfig:
>> > >
>> > > BR2_arm=y
>> > > BR2_cortex_a53=y
>> > > BR2_ARM_FPU_NEON_VFPV4=y
>> > > BR2_TOOLCHAIN_BUILDROOT_GLIBC=y
>> > > BR2_PACKAGE_HOST_LINUX_HEADERS_CUSTOM_4_19=y
>> > > BR2_BINUTILS_VERSION_2_33_X=y
>> > > BR2_GCC_VERSION_9_X=y
>> > > BR2_TOOLCHAIN_BUILDROOT_CXX=y
>> > > BR2_SYSTEM_DHCP="eth0"
>> > > BR2_ROOTFS_POST_BUILD_SCRIPT="board/raspberrypi3/post-build.sh"
>> > > BR2_ROOTFS_POST_IMAGE_SCRIPT="board/raspberrypi3/post-image.sh"
>> > > BR2_ROOTFS_POST_SCRIPT_ARGS="--add-miniuart-bt-overlay"
>> > > BR2_LINUX_KERNEL=y
>> > > BR2_LINUX_KERNEL_CUSTOM_TARBALL=y
>> > > BR2_LINUX_KERNEL_CUSTOM_TARBALL_LOCATION="$(call
>> > >
>> github,raspberrypi,linux,64d0a9870ac14d5eb5253f67d984ae348eec1393)/linux-64d0a9870ac14d5eb5253f67d984ae348eec1393.tar.gz"
>> > > BR2_LINUX_KERNEL_DEFCONFIG="bcm2709"
>> > > BR2_LINUX_KERNEL_DTS_SUPPORT=y
>> > > BR2_LINUX_KERNEL_INTREE_DTS_NAME="bcm2710-rpi-3-b bcm2710-rpi-3-b-plus
>> > > bcm2710-rpi-cm3"
>> > > BR2_LINUX_KERNEL_NEEDS_HOST_OPENSSL=y
>> > > BR2_PACKAGE_QT5=y
>> > > BR2_PACKAGE_QT5BASE_EGLFS=y
>> > > BR2_PACKAGE_QT5QUICKCONTROLS=y
>> > > BR2_PACKAGE_QT5QUICKCONTROLS2=y
>> > > BR2_PACKAGE_QT5XMLPATTERNS=y
>> > > BR2_PACKAGE_RPI_FIRMWARE=y
>> > > BR2_PACKAGE_RPI_USERLAND=y
>> > > BR2_TARGET_ROOTFS_EXT2=y
>> > > BR2_TARGET_ROOTFS_EXT2_4=y
>> > > BR2_TARGET_ROOTFS_EXT2_SIZE="120M"
>> > > # BR2_TARGET_ROOTFS_TAR is not set
>> > > BR2_PACKAGE_HOST_DOSFSTOOLS=y
>> > > BR2_PACKAGE_HOST_GENIMAGE=y
>> > > BR2_PACKAGE_HOST_MTOOLS=y
>> > >
>> > >         $ find . -iname "*xmllistmodel*"
>> > >
>> > >
>> ./build/qt5declarative-5.12.7/src/quick/doc/images/qml-xmllistmodel-example.png
>> > >
>> > >
>> ./build/qt5declarative-5.12.7/tests/testapplications/elements/content/XmlListModelElement.qml
>> > > ./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel
>> > >
>> ./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/qqmlxmllistmodel.cpp
>> > >
>> ./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/qqmlxmllistmodel_p.h
>> > > ./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/
>> xmllistmodel.pro
>> > >
>> > >
>> ./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/.obj/moc_qqmlxmllistmodel_p.o
>> > >
>> > >
>> ./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/.obj/qqmlxmllistmodel.o
>> > >
>> > >
>> ./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/.moc/moc_qqmlxmllistmodel_p.cpp
>> > >
>> > >
>> ./build/qt5xmlpatterns-5.12.7/src/imports/xmllistmodel/.moc/qqmlxmllistmodel.moc
>> > > ./build/qt5xmlpatterns-5.12.7/tests/auto/qquickxmllistmodel
>> > > ./build/qt5xmlpatterns-5.12.7/tests/auto/qquickxmllistmodel/
>> > > qquickxmllistmodel.pro
>> > >
>> > >
>> ./build/qt5xmlpatterns-5.12.7/tests/auto/qquickxmllistmodel/tst_qquickxmllistmodel.cpp
>> > > ./build/qt5xmlpatterns-5.12.7/qml/QtQuick/XmlListModel
>> > >
>> > >
>> ./build/qt5xmlpatterns-5.12.7/qml/QtQuick/XmlListModel/libqmlxmllistmodelplugin.so
>> > >
>> ./host/arm-buildroot-linux-gnueabihf/sysroot/usr/qml/QtQuick/XmlListModel
>> > >
>> > >
>> ./host/arm-buildroot-linux-gnueabihf/sysroot/usr/qml/QtQuick/XmlListModel/libqmlxmllistmodelplugin.so
>> > > ./target/usr/qml/QtQuick/XmlListModel
>> > > ./target/usr/qml/QtQuick/XmlListModel/libqmlxmllistmodelplugin.so
>> > >
>> > > >
>> > > > Thanks,
>> > > > Nimai
>> > > >
>> > > >
>> > > >
>> > > > > --
>> > > > > Thomas Petazzoni, CTO, Bootlin
>> > > > > Embedded Linux and Kernel engineering
>> > > > > https://bootlin.com
>> > > > >
>> > >
>> > >
>>
>>
Peter Seiderer March 15, 2020, 4:52 p.m. UTC | #9
Hello Nimai,

On Sat, 14 Mar 2020 19:58:09 -0400, Nimai Mahajan <nimaim@gmail.com> wrote:


>
> Thanks for your patience Peter. All is good, my application DOES indeed
> work. I nuked my previous config, ccache, etc. and started from scratch
> cloning a fresh copy of BR 2020.02 and redoing my config and now it
> correctly builds. The module is there and loads with my old import
> statement.
>

That's good news, and many thanks back to you for your patience to re-test ;-)

Regards,
Peter
Peter Korsgaard July 22, 2020, 9:15 p.m. UTC | #10
>>>>> "Peter" == Peter Seiderer <ps.report@gmx.net> writes:

 > Hello Nimai,
 > On Sat, 14 Mar 2020 19:58:09 -0400, Nimai Mahajan <nimaim@gmail.com> wrote:


 >> 
 >> Thanks for your patience Peter. All is good, my application DOES indeed
 >> work. I nuked my previous config, ccache, etc. and started from scratch
 >> cloning a fresh copy of BR 2020.02 and redoing my config and now it
 >> correctly builds. The module is there and loads with my old import
 >> statement.
 >> 

 > That's good news, and many thanks back to you for your patience to re-test ;-)

Committed to 2020.02.x, thanks.
diff mbox series

Patch

diff --git a/package/qt5/qt5declarative/Config.in b/package/qt5/qt5declarative/Config.in
index 3a353a8326..d85f4c7136 100644
--- a/package/qt5/qt5declarative/Config.in
+++ b/package/qt5/qt5declarative/Config.in
@@ -1,7 +1,6 @@ 
 config BR2_PACKAGE_QT5DECLARATIVE
 	bool "qt5declarative"
 	depends on BR2_PACKAGE_QT5_JSCORE_AVAILABLE
-	select BR2_PACKAGE_QT5XMLPATTERNS
 	select BR2_PACKAGE_QT5BASE
 	select BR2_PACKAGE_QT5BASE_GUI
 	help
diff --git a/package/qt5/qt5declarative/qt5declarative.mk b/package/qt5/qt5declarative/qt5declarative.mk
index 6210611961..200a5578f8 100644
--- a/package/qt5/qt5declarative/qt5declarative.mk
+++ b/package/qt5/qt5declarative/qt5declarative.mk
@@ -7,7 +7,7 @@ 
 QT5DECLARATIVE_VERSION = $(QT5_VERSION)
 QT5DECLARATIVE_SITE = $(QT5_SITE)
 QT5DECLARATIVE_SOURCE = qtdeclarative-$(QT5_SOURCE_TARBALL_PREFIX)-$(QT5DECLARATIVE_VERSION).tar.xz
-QT5DECLARATIVE_DEPENDENCIES = qt5base qt5xmlpatterns
+QT5DECLARATIVE_DEPENDENCIES = qt5base
 QT5DECLARATIVE_INSTALL_STAGING = YES
 
 ifeq ($(BR2_PACKAGE_QT5_VERSION_LATEST),y)
diff --git a/package/qt5/qt5xmlpatterns/qt5xmlpatterns.mk b/package/qt5/qt5xmlpatterns/qt5xmlpatterns.mk
index 6ed247fd59..5ce4064cb6 100644
--- a/package/qt5/qt5xmlpatterns/qt5xmlpatterns.mk
+++ b/package/qt5/qt5xmlpatterns/qt5xmlpatterns.mk
@@ -21,6 +21,10 @@  ifeq ($(BR2_PACKAGE_QT5BASE_EXAMPLES),y)
 QT5XMLPATTERNS_LICENSE += , BSD-3-Clause (examples)
 endif
 
+ifeq ($(BR2_PACKAGE_QT5DECLARATIVE),y)
+QT5XMLPATTERNS_DEPENDENCIES += qt5declarative
+endif
+
 define QT5XMLPATTERNS_CONFIGURE_CMDS
 	(cd $(@D); $(TARGET_MAKE_ENV) $(HOST_DIR)/bin/qmake)
 endef
@@ -39,6 +43,12 @@  define QT5XMLPATTERNS_INSTALL_TARGET_LIBS
 endef
 endif
 
+ifeq ($(BR2_PACKAGE_QT5DECLARATIVE_QUICK),y)
+define QT5XMLPATTERNS_INSTALL_TARGET_QMLS
+	cp -dpfr $(STAGING_DIR)/usr/qml/QtQuick/XmlListModel $(TARGET_DIR)/usr/qml/QtQuick
+endef
+endif
+
 ifeq ($(BR2_PACKAGE_QT5BASE_EXAMPLES),y)
 define QT5XMLPATTERNS_INSTALL_TARGET_EXAMPLES
 	cp -dpfr $(STAGING_DIR)/usr/lib/qt/examples/xmlpatterns $(TARGET_DIR)/usr/lib/qt/examples/
@@ -47,6 +57,7 @@  endif
 
 define QT5XMLPATTERNS_INSTALL_TARGET_CMDS
 	$(QT5XMLPATTERNS_INSTALL_TARGET_LIBS)
+	$(QT5XMLPATTERNS_INSTALL_TARGET_QMLS)
 	$(QT5XMLPATTERNS_INSTALL_TARGET_EXAMPLES)
 endef