On Tue, Jan 20, 2015 at 09:19:32PM +0000, Burton, Ross wrote:
> On 10 December 2014 at 14:18, Martin Jansa <martin.ja...@gmail.com> wrote:
> 
> > +++ b/meta/recipes-qt/qt4/qt4-x11-free.inc
> > @@ -8,6 +8,9 @@ DEPENDS += "virtual/libgl virtual/libx11 fontconfig libxft
> > libxext libxrender li
> >  PROVIDES += "qt4-x11"
> >  QT4DEPENDS = ""
> >
> > +# depends on virtual/libx11
> > +REQUIRED_DISTRO_FEATURES = "x11"
> > +
> >
> 
> Isn't that redundant as qt4x11 already sets REQUIRED_DISTRO_FEATURES?
> 
> Also, if setting REQURED_DISTRO_FEATURES in a class, using += or _append
> seems like a sensible idea for future-proofing.

Done, good catch.

-- 
Martin 'JaMa' Jansa     jabber: martin.ja...@gmail.com

Attachment: signature.asc
Description: Digital signature

-- 
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Reply via email to