[Buildroot] [PATCH 1/1] qt5location: copy PositioningQuick.so* for QtQuick

Peter Seiderer ps.report at gmx.net
Tue Oct 16 21:49:58 UTC 2018


Hello Alexander,

On Mon, 15 Oct 2018 23:22:20 +0200, Alexander 'z33ky' Hirsch <1zeeky at gmail.com> wrote:

> On Mon, 2018-10-15T22:15:54+0200, Peter Seiderer wrote:
> > Hello Alexander,
> > 
> > On Mon, 15 Oct 2018 21:02:17 +0200, Alexander 'z33ky' Hirsch <1zeeky at gmail.com> wrote:
> > 
> > > The Location module for QtQuick depends on this library, which was not
> > > being copied in the build rule.
> > > 
> > > Signed-off-by: Alexander 'z33ky' Hirsch <1zeeky at gmail.com>
> > > ---
> > >  package/qt5/qt5location/qt5location.mk | 4 ++++
> > >  1 file changed, 4 insertions(+)
> > > 
> > > diff --git a/package/qt5/qt5location/qt5location.mk b/package/qt5/qt5location/qt5location.mk
> > > index 18253e9..fe26b6e 100644
> > > --- a/package/qt5/qt5location/qt5location.mk
> > > +++ b/package/qt5/qt5location/qt5location.mk
> > > @@ -44,6 +44,9 @@ define QT5LOCATION_INSTALL_TARGET_LOCATION
> > >  	cp -dpf $(STAGING_DIR)/usr/lib/libQt5Location.so.* $(TARGET_DIR)/usr/lib
> > >  	cp -dpfr $(STAGING_DIR)/usr/lib/qt/plugins/geoservices $(TARGET_DIR)/usr/lib/qt/plugins/
> > >  endef
> > 
> > Should depend on BR2_PACKAGE_QT5_VERSION_LATEST because the library is not
> > build/installed in the Qt5.6.x case:
> > 
> > + ifeq ($(BR2_PACKAGE_QT5_VERSION_LATEST),y)
> > 
> > > +define QT5LOCATION_INSTALL_TARGET_POSITION_QUICK
> > > +	cp -dpf $(STAGING_DIR)/usr/lib/libQt5PositioningQuick.so.* $(TARGET_DIR)/usr/lib
> > > +endef
> > 
> > + endif
> > 
> > Thanks for the patch, mind to resend with the changes?
> > 
> > Regards,
> > Peter
> > 
> > >  ifeq ($(BR2_PACKAGE_QT5BASE_EXAMPLES),y)
> > >  define QT5LOCATION_INSTALL_TARGET_EXAMPLES
> > >  	cp -dpfr $(STAGING_DIR)/usr/lib/qt/examples/location $(TARGET_DIR)/usr/lib/qt/examples/
> > > @@ -59,6 +62,7 @@ endef
> > >  
> > >  define QT5LOCATION_INSTALL_TARGET_CMDS
> > >  	$(QT5LOCATION_INSTALL_TARGET_POSITION)
> > > +	$(QT5LOCATION_INSTALL_TARGET_POSITION_QUICK)
> > >  	$(QT5LOCATION_INSTALL_TARGET_LOCATION)
> > >  	$(QT5LOCATION_INSTALL_TARGET_QMLS)
> > >  	$(QT5LOCATION_INSTALL_TARGET_EXAMPLES)
> > 
> 
> Sure, thanks for the review.
> Wouldn't it be better to check if QT5LOCATION_VERSION >= 5.11 though? That seems to be the version where this library was added.

In principle your are right with your suggestion, but the current way to express
this one in buildroot is the dump BR2_PACKAGE_QT5_VERSION_LATEST check ;-)

Regards,
Peter

> 
> Regards
> Alexander
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot




More information about the buildroot mailing list