diff mbox series

[1/1] qt5cinex: depends also on qt5quickcontrols

Message ID 1510677027-24076-1-git-send-email-giulio.benetti@micronovasrl.com
State Rejected
Headers show
Series [1/1] qt5cinex: depends also on qt5quickcontrols | expand

Commit Message

Giulio Benetti Nov. 14, 2017, 4:30 p.m. UTC
qt5cinex failed to build if qt5quickcontrols is not selected
add qt5quickcontrols to qt5cinex dependencies in .mk
select BR2_PACKAGE_QT5QUICKCONTROLS in Config.in

Signed-off-by: Giulio Benetti <giulio.benetti@micronovasrl.com>
---
 package/qt5cinex/Config.in   | 1 +
 package/qt5cinex/qt5cinex.mk | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

Comments

Baruch Siach Nov. 14, 2017, 4:58 p.m. UTC | #1
Hi Giulio,

On Tue, Nov 14, 2017 at 05:32:26PM +0100, Giulio Benetti wrote:
> Errata:
> 
> Il 14/11/2017 17:30, Giulio Benetti ha scritto:
> > qt5cinex failed to build if qt5quickcontrols is not selected
> qt5cinex fail to execute on runtime if qt5quickcontrols is not selected

If it's a runtime only dependency you should not list it in _DEPENDENCIES ...

> > add qt5quickcontrols to qt5cinex dependencies in .mk
> > select BR2_PACKAGE_QT5QUICKCONTROLS in Config.in
> > 
> > Signed-off-by: Giulio Benetti <giulio.benetti@micronovasrl.com>
> > ---
> >   package/qt5cinex/Config.in   | 1 +
> >   package/qt5cinex/qt5cinex.mk | 2 +-
> >   2 files changed, 2 insertions(+), 1 deletion(-)
> > 
> > diff --git a/package/qt5cinex/Config.in b/package/qt5cinex/Config.in
> > index 5adf6c3..7cc459a 100644
> > --- a/package/qt5cinex/Config.in
> > +++ b/package/qt5cinex/Config.in
> > @@ -16,6 +16,7 @@ config BR2_PACKAGE_QT5CINEX
> >   	select BR2_PACKAGE_QT5BASE_WIDGETS
> >   	select BR2_PACKAGE_QT5BASE_EGLFS
> >   	select BR2_PACKAGE_QT5GRAPHICALEFFECTS
> > +	select BR2_PACKAGE_QT5QUICKCONTROLS

... but you should add a 'runtime' comment here.

> >   	help
> >   	  This application demonstrates the power of Qt5 and few of the new
> >   	  additions available in QtQuick 2.0.
> > diff --git a/package/qt5cinex/qt5cinex.mk b/package/qt5cinex/qt5cinex.mk
> > index d9254f5..44c53a1 100644
> > --- a/package/qt5cinex/qt5cinex.mk
> > +++ b/package/qt5cinex/qt5cinex.mk
> > @@ -12,7 +12,7 @@ QT5CINEX_HD = "rpi_"
> >   endif
> >   QT5CINEX_SOURCE = Qt5_CinematicExperience_$(QT5CINEX_HD)$(QT5CINEX_VERSION).tgz
> > -QT5CINEX_DEPENDENCIES = qt5base qt5declarative
> > +QT5CINEX_DEPENDENCIES = qt5base qt5declarative qt5quickcontrols
> >   QT5CINEX_LICENSE = CC-BY-3.0
> >   QT5CINEX_LICENSE_FILES = README

baruch
Giulio Benetti Nov. 14, 2017, 5:03 p.m. UTC | #2
Sorry, my mistake.
It installed correctly on STAGING_DIR, but not in TARGET_DIR.
This why it compiled fine, but on target it didn't find that 
libQtQuick.so library.
This is a problem on building Qt5 I've noticed.

If I build first time Qt5 with few features,
than select other more features,
those can be built partially and installed partially too.

To reproduce you can build qt5base.
After that, select qt5cinex,
then you should find that libraries are in STAGING_DIR but not in 
TARGET_DIR.

So to have it working properly you have to rm and rebuild qt5base and 
qt5declarative from beginning.

Did someone else noticed this?

Il 14/11/2017 17:58, Baruch Siach ha scritto:
> Hi Giulio,
> 
> On Tue, Nov 14, 2017 at 05:32:26PM +0100, Giulio Benetti wrote:
>> Errata:
>>
>> Il 14/11/2017 17:30, Giulio Benetti ha scritto:
>>> qt5cinex failed to build if qt5quickcontrols is not selected
>> qt5cinex fail to execute on runtime if qt5quickcontrols is not selected
> 
> If it's a runtime only dependency you should not list it in _DEPENDENCIES ...
> 
>>> add qt5quickcontrols to qt5cinex dependencies in .mk
>>> select BR2_PACKAGE_QT5QUICKCONTROLS in Config.in
>>>
>>> Signed-off-by: Giulio Benetti <giulio.benetti@micronovasrl.com>
>>> ---
>>>    package/qt5cinex/Config.in   | 1 +
>>>    package/qt5cinex/qt5cinex.mk | 2 +-
>>>    2 files changed, 2 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/package/qt5cinex/Config.in b/package/qt5cinex/Config.in
>>> index 5adf6c3..7cc459a 100644
>>> --- a/package/qt5cinex/Config.in
>>> +++ b/package/qt5cinex/Config.in
>>> @@ -16,6 +16,7 @@ config BR2_PACKAGE_QT5CINEX
>>>    	select BR2_PACKAGE_QT5BASE_WIDGETS
>>>    	select BR2_PACKAGE_QT5BASE_EGLFS
>>>    	select BR2_PACKAGE_QT5GRAPHICALEFFECTS
>>> +	select BR2_PACKAGE_QT5QUICKCONTROLS
> 
> ... but you should add a 'runtime' comment here.
> 
>>>    	help
>>>    	  This application demonstrates the power of Qt5 and few of the new
>>>    	  additions available in QtQuick 2.0.
>>> diff --git a/package/qt5cinex/qt5cinex.mk b/package/qt5cinex/qt5cinex.mk
>>> index d9254f5..44c53a1 100644
>>> --- a/package/qt5cinex/qt5cinex.mk
>>> +++ b/package/qt5cinex/qt5cinex.mk
>>> @@ -12,7 +12,7 @@ QT5CINEX_HD = "rpi_"
>>>    endif
>>>    QT5CINEX_SOURCE = Qt5_CinematicExperience_$(QT5CINEX_HD)$(QT5CINEX_VERSION).tgz
>>> -QT5CINEX_DEPENDENCIES = qt5base qt5declarative
>>> +QT5CINEX_DEPENDENCIES = qt5base qt5declarative qt5quickcontrols
>>>    QT5CINEX_LICENSE = CC-BY-3.0
>>>    QT5CINEX_LICENSE_FILES = README
> 
> baruch
>
Baruch Siach Nov. 14, 2017, 5:07 p.m. UTC | #3
Hi Giulio,

On Tue, Nov 14, 2017 at 06:03:53PM +0100, Giulio Benetti wrote:
> Sorry, my mistake.
> It installed correctly on STAGING_DIR, but not in TARGET_DIR.
> This why it compiled fine, but on target it didn't find that libQtQuick.so
> library.
> This is a problem on building Qt5 I've noticed.
> 
> If I build first time Qt5 with few features,
> than select other more features,
> those can be built partially and installed partially too.
> 
> To reproduce you can build qt5base.
> After that, select qt5cinex,
> then you should find that libraries are in STAGING_DIR but not in
> TARGET_DIR.
> 
> So to have it working properly you have to rm and rebuild qt5base and
> qt5declarative from beginning.
> 
> Did someone else noticed this?

Buildroot can't reliably rebuild reverse dependencies of packages that change 
configuration. This is a known issue. See the manual section:

  http://nightly.buildroot.org/manual.html#full-rebuild

baruch

> Il 14/11/2017 17:58, Baruch Siach ha scritto:
> > On Tue, Nov 14, 2017 at 05:32:26PM +0100, Giulio Benetti wrote:
> > > Errata:
> > > 
> > > Il 14/11/2017 17:30, Giulio Benetti ha scritto:
> > > > qt5cinex failed to build if qt5quickcontrols is not selected
> > > qt5cinex fail to execute on runtime if qt5quickcontrols is not selected
> > 
> > If it's a runtime only dependency you should not list it in _DEPENDENCIES ...
> > 
> > > > add qt5quickcontrols to qt5cinex dependencies in .mk
> > > > select BR2_PACKAGE_QT5QUICKCONTROLS in Config.in
> > > > 
> > > > Signed-off-by: Giulio Benetti <giulio.benetti@micronovasrl.com>
> > > > ---
> > > >    package/qt5cinex/Config.in   | 1 +
> > > >    package/qt5cinex/qt5cinex.mk | 2 +-
> > > >    2 files changed, 2 insertions(+), 1 deletion(-)
> > > > 
> > > > diff --git a/package/qt5cinex/Config.in b/package/qt5cinex/Config.in
> > > > index 5adf6c3..7cc459a 100644
> > > > --- a/package/qt5cinex/Config.in
> > > > +++ b/package/qt5cinex/Config.in
> > > > @@ -16,6 +16,7 @@ config BR2_PACKAGE_QT5CINEX
> > > >    	select BR2_PACKAGE_QT5BASE_WIDGETS
> > > >    	select BR2_PACKAGE_QT5BASE_EGLFS
> > > >    	select BR2_PACKAGE_QT5GRAPHICALEFFECTS
> > > > +	select BR2_PACKAGE_QT5QUICKCONTROLS
> > 
> > ... but you should add a 'runtime' comment here.
> > 
> > > >    	help
> > > >    	  This application demonstrates the power of Qt5 and few of the new
> > > >    	  additions available in QtQuick 2.0.
> > > > diff --git a/package/qt5cinex/qt5cinex.mk b/package/qt5cinex/qt5cinex.mk
> > > > index d9254f5..44c53a1 100644
> > > > --- a/package/qt5cinex/qt5cinex.mk
> > > > +++ b/package/qt5cinex/qt5cinex.mk
> > > > @@ -12,7 +12,7 @@ QT5CINEX_HD = "rpi_"
> > > >    endif
> > > >    QT5CINEX_SOURCE = Qt5_CinematicExperience_$(QT5CINEX_HD)$(QT5CINEX_VERSION).tgz
> > > > -QT5CINEX_DEPENDENCIES = qt5base qt5declarative
> > > > +QT5CINEX_DEPENDENCIES = qt5base qt5declarative qt5quickcontrols
> > > >    QT5CINEX_LICENSE = CC-BY-3.0
> > > >    QT5CINEX_LICENSE_FILES = README
> > 
> > baruch
> > 
> 
> 
> -- 
> Giulio Benetti
> R&D Manager &
> Advanced Research
> 
> MICRONOVA SRL
> Sede: Via A. Niedda 3 - 35010 Vigonza (PD)
> Tel. 049/8931563 - Fax 049/8931346
> Cod.Fiscale - P.IVA 02663420285
> Capitale Sociale € 26.000 i.v.
> Iscritta al Reg. Imprese di Padova N. 02663420285
> Numero R.E.A. 258642
Giulio Benetti Nov. 14, 2017, 5:11 p.m. UTC | #4
Hi,

Il 14/11/2017 18:07, Baruch Siach ha scritto:
> Hi Giulio,
> 
> On Tue, Nov 14, 2017 at 06:03:53PM +0100, Giulio Benetti wrote:
>> Sorry, my mistake.
>> It installed correctly on STAGING_DIR, but not in TARGET_DIR.
>> This why it compiled fine, but on target it didn't find that libQtQuick.so
>> library.
>> This is a problem on building Qt5 I've noticed.
>>
>> If I build first time Qt5 with few features,
>> than select other more features,
>> those can be built partially and installed partially too.
>>
>> To reproduce you can build qt5base.
>> After that, select qt5cinex,
>> then you should find that libraries are in STAGING_DIR but not in
>> TARGET_DIR.
>>
>> So to have it working properly you have to rm and rebuild qt5base and
>> qt5declarative from beginning.
>>
>> Did someone else noticed this?
> 
> Buildroot can't reliably rebuild reverse dependencies of packages that change
> configuration. This is a known issue. See the manual section:
> 
>    http://nightly.buildroot.org/manual.html#full-rebuild

Ok, thank you and sorry to rise useless question.

> 
> baruch
> 
>> Il 14/11/2017 17:58, Baruch Siach ha scritto:
>>> On Tue, Nov 14, 2017 at 05:32:26PM +0100, Giulio Benetti wrote:
>>>> Errata:
>>>>
>>>> Il 14/11/2017 17:30, Giulio Benetti ha scritto:
>>>>> qt5cinex failed to build if qt5quickcontrols is not selected
>>>> qt5cinex fail to execute on runtime if qt5quickcontrols is not selected
>>>
>>> If it's a runtime only dependency you should not list it in _DEPENDENCIES ...
>>>
>>>>> add qt5quickcontrols to qt5cinex dependencies in .mk
>>>>> select BR2_PACKAGE_QT5QUICKCONTROLS in Config.in
>>>>>
>>>>> Signed-off-by: Giulio Benetti <giulio.benetti@micronovasrl.com>
>>>>> ---
>>>>>     package/qt5cinex/Config.in   | 1 +
>>>>>     package/qt5cinex/qt5cinex.mk | 2 +-
>>>>>     2 files changed, 2 insertions(+), 1 deletion(-)
>>>>>
>>>>> diff --git a/package/qt5cinex/Config.in b/package/qt5cinex/Config.in
>>>>> index 5adf6c3..7cc459a 100644
>>>>> --- a/package/qt5cinex/Config.in
>>>>> +++ b/package/qt5cinex/Config.in
>>>>> @@ -16,6 +16,7 @@ config BR2_PACKAGE_QT5CINEX
>>>>>     	select BR2_PACKAGE_QT5BASE_WIDGETS
>>>>>     	select BR2_PACKAGE_QT5BASE_EGLFS
>>>>>     	select BR2_PACKAGE_QT5GRAPHICALEFFECTS
>>>>> +	select BR2_PACKAGE_QT5QUICKCONTROLS
>>>
>>> ... but you should add a 'runtime' comment here.
>>>
>>>>>     	help
>>>>>     	  This application demonstrates the power of Qt5 and few of the new
>>>>>     	  additions available in QtQuick 2.0.
>>>>> diff --git a/package/qt5cinex/qt5cinex.mk b/package/qt5cinex/qt5cinex.mk
>>>>> index d9254f5..44c53a1 100644
>>>>> --- a/package/qt5cinex/qt5cinex.mk
>>>>> +++ b/package/qt5cinex/qt5cinex.mk
>>>>> @@ -12,7 +12,7 @@ QT5CINEX_HD = "rpi_"
>>>>>     endif
>>>>>     QT5CINEX_SOURCE = Qt5_CinematicExperience_$(QT5CINEX_HD)$(QT5CINEX_VERSION).tgz
>>>>> -QT5CINEX_DEPENDENCIES = qt5base qt5declarative
>>>>> +QT5CINEX_DEPENDENCIES = qt5base qt5declarative qt5quickcontrols
>>>>>     QT5CINEX_LICENSE = CC-BY-3.0
>>>>>     QT5CINEX_LICENSE_FILES = README
>>>
>>> baruch
>>>
>>
>>
>> -- 
>> Giulio Benetti
>> R&D Manager &
>> Advanced Research
>>
>> MICRONOVA SRL
>> Sede: Via A. Niedda 3 - 35010 Vigonza (PD)
>> Tel. 049/8931563 - Fax 049/8931346
>> Cod.Fiscale - P.IVA 02663420285
>> Capitale Sociale € 26.000 i.v.
>> Iscritta al Reg. Imprese di Padova N. 02663420285
>> Numero R.E.A. 258642
>
Thomas Petazzoni Nov. 23, 2017, 8:45 p.m. UTC | #5
Hello,

On Tue, 14 Nov 2017 18:03:53 +0100, Giulio Benetti wrote:
> Sorry, my mistake.
> It installed correctly on STAGING_DIR, but not in TARGET_DIR.
> This why it compiled fine, but on target it didn't find that 
> libQtQuick.so library.
> This is a problem on building Qt5 I've noticed.
> 
> If I build first time Qt5 with few features,
> than select other more features,
> those can be built partially and installed partially too.
> 
> To reproduce you can build qt5base.
> After that, select qt5cinex,
> then you should find that libraries are in STAGING_DIR but not in 
> TARGET_DIR.
> 
> So to have it working properly you have to rm and rebuild qt5base and 
> qt5declarative from beginning.
> 
> Did someone else noticed this?

So in the end, is your patch needed ?

Peter (Seiderer), could you review this Qt5 related patch ? Thanks!

Best regards,

Thomas
Giulio Benetti Nov. 23, 2017, 9:19 p.m. UTC | #6
Hi Thomas,

Il 23/11/2017 21:45, Thomas Petazzoni ha scritto:
> Hello,
> 
> On Tue, 14 Nov 2017 18:03:53 +0100, Giulio Benetti wrote:
>> Sorry, my mistake.
>> It installed correctly on STAGING_DIR, but not in TARGET_DIR.
>> This why it compiled fine, but on target it didn't find that
>> libQtQuick.so library.
>> This is a problem on building Qt5 I've noticed.
>>
>> If I build first time Qt5 with few features,
>> than select other more features,
>> those can be built partially and installed partially too.
>>
>> To reproduce you can build qt5base.
>> After that, select qt5cinex,
>> then you should find that libraries are in STAGING_DIR but not in
>> TARGET_DIR.
>>
>> So to have it working properly you have to rm and rebuild qt5base and
>> qt5declarative from beginning.
>>
>> Did someone else noticed this?
> 
> So in the end, is your patch needed ?

No it's not needed, my fault,
sorry.

> 
> Peter (Seiderer), could you review this Qt5 related patch ? Thanks!
> 
> Best regards,
> 
> Thomas
>
Peter Seiderer Nov. 23, 2017, 9:28 p.m. UTC | #7
Hello *,

On Thu, 23 Nov 2017 21:45:46 +0100, Thomas Petazzoni <thomas.petazzoni@free-electrons.com> wrote:

> Hello,
> 
> On Tue, 14 Nov 2017 18:03:53 +0100, Giulio Benetti wrote:
> > Sorry, my mistake.
> > It installed correctly on STAGING_DIR, but not in TARGET_DIR.
> > This why it compiled fine, but on target it didn't find that 
> > libQtQuick.so library.
> > This is a problem on building Qt5 I've noticed.
> > 
> > If I build first time Qt5 with few features,
> > than select other more features,
> > those can be built partially and installed partially too.
> > 
> > To reproduce you can build qt5base.
> > After that, select qt5cinex,
> > then you should find that libraries are in STAGING_DIR but not in 
> > TARGET_DIR.
> > 
> > So to have it working properly you have to rm and rebuild qt5base and 
> > qt5declarative from beginning.
> > 
> > Did someone else noticed this?  
> 
> So in the end, is your patch needed ?
> 
> Peter (Seiderer), could you review this Qt5 related patch ? Thanks!

No qml expert, but a quick search for 'import QtQuick.Controls' (needed for usage of qt5quickcontrols) did
not yield any results:

	$ wget http://quitcoding.com/download/Qt5_CinematicExperience_rpi_1.0.tgz
	$ wget http://quitcoding.com/download/Qt5_CinematicExperience_1.0.tgz
	$ tar -xvzf Qt5_CinematicExperience_1.0.tgz 
	$ tar -xvzf Qt5_CinematicExperience_rpi_1.0.tgz 
	$ find . -name "*.qml" | xargs grep import

./Qt5_CinematicExperience_1.0/content/CurtainEffect.qml:import QtQuick 2.0
./Qt5_CinematicExperience_1.0/content/InfoView.qml:import QtQuick 2.0
./Qt5_CinematicExperience_1.0/content/Switch.qml:import QtQuick 2.0
./Qt5_CinematicExperience_1.0/content/Switch.qml:import QtQuick.Particles 2.0
./Qt5_CinematicExperience_1.0/content/DetailsView.qml:import QtQuick 2.0
./Qt5_CinematicExperience_1.0/content/InfoViewItem.qml:import QtQuick 2.0
./Qt5_CinematicExperience_1.0/content/Background.qml:import QtQuick 2.0
./Qt5_CinematicExperience_1.0/content/Background.qml:import QtQuick.Particles 2.0
./Qt5_CinematicExperience_1.0/content/RatingsItem.qml:import QtQuick 2.0
./Qt5_CinematicExperience_1.0/content/DelegateItem.qml:import QtQuick 2.0
./Qt5_CinematicExperience_1.0/content/MoviesModel.qml:import QtQuick 2.0
./Qt5_CinematicExperience_1.0/content/SettingsView.qml:import QtQuick 2.0
./Qt5_CinematicExperience_1.0/content/MainView.qml:import QtQuick 2.0
./Qt5_CinematicExperience_1.0/content/MainView.qml:import QtQuick.Particles 2.0
./Qt5_CinematicExperience_1.0/content/MainView.qml:import QtGraphicalEffects 1.0
./Qt5_CinematicExperience_1.0/content/Button.qml:import QtQuick 2.0
./Qt5_CinematicExperience_1.0/content/Button.qml:import QtQuick.Particles 2.0
./Qt5_CinematicExperience_1.0/Qt5_CinematicExperience.qml:import QtQuick 2.0
./Qt5_CinematicExperience_1.0/Qt5_CinematicExperience.qml:import "content"
./Qt5_CinematicExperience_rpi_1.0/content/CurtainEffect.qml:import QtQuick 2.0
./Qt5_CinematicExperience_rpi_1.0/content/InfoView.qml:import QtQuick 2.0
./Qt5_CinematicExperience_rpi_1.0/content/Switch.qml:import QtQuick 2.0
./Qt5_CinematicExperience_rpi_1.0/content/Switch.qml:import QtQuick.Particles 2.0
./Qt5_CinematicExperience_rpi_1.0/content/DetailsView.qml:import QtQuick 2.0
./Qt5_CinematicExperience_rpi_1.0/content/InfoViewItem.qml:import QtQuick 2.0
./Qt5_CinematicExperience_rpi_1.0/content/Background.qml:import QtQuick 2.0
./Qt5_CinematicExperience_rpi_1.0/content/Background.qml:import QtQuick.Particles 2.0
./Qt5_CinematicExperience_rpi_1.0/content/RatingsItem.qml:import QtQuick 2.0
./Qt5_CinematicExperience_rpi_1.0/content/FpsItem.qml:import QtQuick 2.0
./Qt5_CinematicExperience_rpi_1.0/content/DelegateItem.qml:import QtQuick 2.0
./Qt5_CinematicExperience_rpi_1.0/content/MoviesModel.qml:import QtQuick 2.0
./Qt5_CinematicExperience_rpi_1.0/content/SettingsView.qml:import QtQuick 2.0
./Qt5_CinematicExperience_rpi_1.0/content/MainView.qml:import QtQuick 2.0
./Qt5_CinematicExperience_rpi_1.0/content/MainView.qml:import QtQuick.Particles 2.0
./Qt5_CinematicExperience_rpi_1.0/content/MainView.qml:import QtGraphicalEffects 1.0
./Qt5_CinematicExperience_rpi_1.0/content/Button.qml:import QtQuick 2.0
./Qt5_CinematicExperience_rpi_1.0/content/Button.qml:import QtQuick.Particles 2.0
./Qt5_CinematicExperience_rpi_1.0/Qt5_CinematicExperience.qml:import QtQuick 2.0
./Qt5_CinematicExperience_rpi_1.0/Qt5_CinematicExperience.qml:import "content"

>>> qt5cinex fail to execute on runtime if qt5quickcontrols is not selected

Did not do a runtime test, but what is the error message (if any)?

Regards,
Peter

> 
> Best regards,
> 
> Thomas
Giulio Benetti Nov. 23, 2017, 9:47 p.m. UTC | #8
Hello everybody,

Il 23/11/2017 22:28, Peter Seiderer ha scritto:
> Hello *,
> 
> On Thu, 23 Nov 2017 21:45:46 +0100, Thomas Petazzoni <thomas.petazzoni@free-electrons.com> wrote:
> 
>> Hello,
>>
>> On Tue, 14 Nov 2017 18:03:53 +0100, Giulio Benetti wrote:
>>> Sorry, my mistake.
>>> It installed correctly on STAGING_DIR, but not in TARGET_DIR.
>>> This why it compiled fine, but on target it didn't find that
>>> libQtQuick.so library.
>>> This is a problem on building Qt5 I've noticed.
>>>
>>> If I build first time Qt5 with few features,
>>> than select other more features,
>>> those can be built partially and installed partially too.
>>>
>>> To reproduce you can build qt5base.
>>> After that, select qt5cinex,
>>> then you should find that libraries are in STAGING_DIR but not in
>>> TARGET_DIR.
>>>
>>> So to have it working properly you have to rm and rebuild qt5base and
>>> qt5declarative from beginning.
>>>
>>> Did someone else noticed this?
>>
>> So in the end, is your patch needed ?
>>
>> Peter (Seiderer), could you review this Qt5 related patch ? Thanks!
> 
> No qml expert, but a quick search for 'import QtQuick.Controls' (needed for usage of qt5quickcontrols) did
> not yield any results:
> 
> 	$ wget http://quitcoding.com/download/Qt5_CinematicExperience_rpi_1.0.tgz
> 	$ wget http://quitcoding.com/download/Qt5_CinematicExperience_1.0.tgz
> 	$ tar -xvzf Qt5_CinematicExperience_1.0.tgz
> 	$ tar -xvzf Qt5_CinematicExperience_rpi_1.0.tgz
> 	$ find . -name "*.qml" | xargs grep import
> 
> ./Qt5_CinematicExperience_1.0/content/CurtainEffect.qml:import QtQuick 2.0
> ./Qt5_CinematicExperience_1.0/content/InfoView.qml:import QtQuick 2.0
> ./Qt5_CinematicExperience_1.0/content/Switch.qml:import QtQuick 2.0
> ./Qt5_CinematicExperience_1.0/content/Switch.qml:import QtQuick.Particles 2.0
> ./Qt5_CinematicExperience_1.0/content/DetailsView.qml:import QtQuick 2.0
> ./Qt5_CinematicExperience_1.0/content/InfoViewItem.qml:import QtQuick 2.0
> ./Qt5_CinematicExperience_1.0/content/Background.qml:import QtQuick 2.0
> ./Qt5_CinematicExperience_1.0/content/Background.qml:import QtQuick.Particles 2.0
> ./Qt5_CinematicExperience_1.0/content/RatingsItem.qml:import QtQuick 2.0
> ./Qt5_CinematicExperience_1.0/content/DelegateItem.qml:import QtQuick 2.0
> ./Qt5_CinematicExperience_1.0/content/MoviesModel.qml:import QtQuick 2.0
> ./Qt5_CinematicExperience_1.0/content/SettingsView.qml:import QtQuick 2.0
> ./Qt5_CinematicExperience_1.0/content/MainView.qml:import QtQuick 2.0
> ./Qt5_CinematicExperience_1.0/content/MainView.qml:import QtQuick.Particles 2.0
> ./Qt5_CinematicExperience_1.0/content/MainView.qml:import QtGraphicalEffects 1.0
> ./Qt5_CinematicExperience_1.0/content/Button.qml:import QtQuick 2.0
> ./Qt5_CinematicExperience_1.0/content/Button.qml:import QtQuick.Particles 2.0
> ./Qt5_CinematicExperience_1.0/Qt5_CinematicExperience.qml:import QtQuick 2.0
> ./Qt5_CinematicExperience_1.0/Qt5_CinematicExperience.qml:import "content"
> ./Qt5_CinematicExperience_rpi_1.0/content/CurtainEffect.qml:import QtQuick 2.0
> ./Qt5_CinematicExperience_rpi_1.0/content/InfoView.qml:import QtQuick 2.0
> ./Qt5_CinematicExperience_rpi_1.0/content/Switch.qml:import QtQuick 2.0
> ./Qt5_CinematicExperience_rpi_1.0/content/Switch.qml:import QtQuick.Particles 2.0
> ./Qt5_CinematicExperience_rpi_1.0/content/DetailsView.qml:import QtQuick 2.0
> ./Qt5_CinematicExperience_rpi_1.0/content/InfoViewItem.qml:import QtQuick 2.0
> ./Qt5_CinematicExperience_rpi_1.0/content/Background.qml:import QtQuick 2.0
> ./Qt5_CinematicExperience_rpi_1.0/content/Background.qml:import QtQuick.Particles 2.0
> ./Qt5_CinematicExperience_rpi_1.0/content/RatingsItem.qml:import QtQuick 2.0
> ./Qt5_CinematicExperience_rpi_1.0/content/FpsItem.qml:import QtQuick 2.0
> ./Qt5_CinematicExperience_rpi_1.0/content/DelegateItem.qml:import QtQuick 2.0
> ./Qt5_CinematicExperience_rpi_1.0/content/MoviesModel.qml:import QtQuick 2.0
> ./Qt5_CinematicExperience_rpi_1.0/content/SettingsView.qml:import QtQuick 2.0
> ./Qt5_CinematicExperience_rpi_1.0/content/MainView.qml:import QtQuick 2.0
> ./Qt5_CinematicExperience_rpi_1.0/content/MainView.qml:import QtQuick.Particles 2.0
> ./Qt5_CinematicExperience_rpi_1.0/content/MainView.qml:import QtGraphicalEffects 1.0
> ./Qt5_CinematicExperience_rpi_1.0/content/Button.qml:import QtQuick 2.0
> ./Qt5_CinematicExperience_rpi_1.0/content/Button.qml:import QtQuick.Particles 2.0
> ./Qt5_CinematicExperience_rpi_1.0/Qt5_CinematicExperience.qml:import QtQuick 2.0
> ./Qt5_CinematicExperience_rpi_1.0/Qt5_CinematicExperience.qml:import "content"
> 

Peter you're right.
My mistake was:
- building all buildroot with only qt5base
- modifying .config enabling qt5cinex which selects qt5declarative
- giving another make(not building from scratch)

So I ended up having qt5base not aware of qt5declarative when installing 
qml modules.

At this point I rebuilt from scratch adding qt5quickcontrols to 
.config(mistake, don't why I thought it),
and it worked.

So I thought qt5quickcontrols was missing.
Instead qt5quickcontrols is not necessary,
and selecting qt5cinex did its job selecting correctly qt5declarative.

This is all the story about this useless patch.

>>>> qt5cinex fail to execute on runtime if qt5quickcontrols is not selected
> 
> Did not do a runtime test, but what is the error message (if any)?
> 
> Regards,
> Peter
> 
>>
>> Best regards,
>>
>> Thomas
>
diff mbox series

Patch

diff --git a/package/qt5cinex/Config.in b/package/qt5cinex/Config.in
index 5adf6c3..7cc459a 100644
--- a/package/qt5cinex/Config.in
+++ b/package/qt5cinex/Config.in
@@ -16,6 +16,7 @@  config BR2_PACKAGE_QT5CINEX
 	select BR2_PACKAGE_QT5BASE_WIDGETS
 	select BR2_PACKAGE_QT5BASE_EGLFS
 	select BR2_PACKAGE_QT5GRAPHICALEFFECTS
+	select BR2_PACKAGE_QT5QUICKCONTROLS
 	help
 	  This application demonstrates the power of Qt5 and few of the new
 	  additions available in QtQuick 2.0.
diff --git a/package/qt5cinex/qt5cinex.mk b/package/qt5cinex/qt5cinex.mk
index d9254f5..44c53a1 100644
--- a/package/qt5cinex/qt5cinex.mk
+++ b/package/qt5cinex/qt5cinex.mk
@@ -12,7 +12,7 @@  QT5CINEX_HD = "rpi_"
 endif
 
 QT5CINEX_SOURCE = Qt5_CinematicExperience_$(QT5CINEX_HD)$(QT5CINEX_VERSION).tgz
-QT5CINEX_DEPENDENCIES = qt5base qt5declarative
+QT5CINEX_DEPENDENCIES = qt5base qt5declarative qt5quickcontrols
 
 QT5CINEX_LICENSE = CC-BY-3.0
 QT5CINEX_LICENSE_FILES = README