The patch set looks great. The only issue I see is what happens when a user wants to use Qt 4. I don't think Qt 4 and wayland works together so we should have a check to not include Wayland if Qt 4 is being used. Thoughts?
> -----Original Message----- > From: [email protected] [mailto:meta-arago- > [email protected]] On Behalf Of Dmytriyenko, Denys > Sent: Thursday, March 12, 2015 2:54 PM > To: Stiffler, Jacob > Cc: [email protected] > Subject: Re: [meta-arago] [PATCH v2 0/4] Implment the wayland > DISTRO_FEATURE. > > Looks good, thanks. > > On Thu, Mar 12, 2015 at 03:46:18PM -0400, Jacob Stiffler wrote: > > * Only include wayland, weston, and example packages if 'wayland' is a > > DISTRO_FEATURE. > > * By default, if 'screen' is a MACHINE_FEATURE, then add 'wayland' as a > > DISTRO_FEATURE. > > * Configure qt5 to use wayland if it is a DISTRO_FEATURE, otherwise use > eglfs. > > * Add processor-sdk brand to remove wayland from DISTRO_EFATURES for > the ti33x > > and ti43x soc families. > > > > > > Jacob Stiffler (4): > > arago.conf: Include wayland in DISTRO_FEATURES by default. > > packagegroup-*: Include wayland/weston if specified in > > DISTRO_FEATURES. > > qtbase: Use DISTRO_FEATURES to set appropriate qpa. > > branding-processor-sdk: Add processor-sdk brand. > > > > meta-arago-distro/conf/distro/arago.conf | 2 +- > > .../conf/distro/include/branding-processor-sdk.inc | 35 > ++++++++++++++++++++ > > .../packagegroups/packagegroup-arago-qte.bb | 4 +-- > > .../packagegroup-arago-tisdk-graphics.bb | 4 +-- > > .../packagegroups/packagegroup-arago-tisdk-qte.bb | 4 +-- > > .../packagegroups/packagegroup-linaro-tc-test.bb | 5 +-- > > meta-arago-distro/recipes-qt/qt5/qtbase/qt_env.sh | 1 - > > .../recipes-qt/qt5/qtbase_5.4.1.bbappend | 4 +-- > > 8 files changed, 47 insertions(+), 12 deletions(-) create mode > > 100644 > > meta-arago-distro/conf/distro/include/branding-processor-sdk.inc > > > > -- > > v2: Change order of patches. > > > > 1.7.9.5 > > > > _______________________________________________ > > meta-arago mailing list > > [email protected] > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago > _______________________________________________ > meta-arago mailing list > [email protected] > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago _______________________________________________ meta-arago mailing list [email protected] http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
