On 8/9/17 8:41 AM, Andre McCurdy wrote:
> On Wed, Aug 9, 2017 at 7:19 AM, Khem Raj <raj.k...@gmail.com> wrote:
>> Signed-off-by: Khem Raj <raj.k...@gmail.com>
> 
> Any explanation? Was this previously non-deterministic? If so, how far
> back does the problem go?

it isnt non-deterministic atleast not after RSS. However, we need this
enabled when we have openGL/X11 enabled.

> 
>> ---
>>  meta/recipes-graphics/cairo/cairo.inc | 4 +++-
>>  1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/meta/recipes-graphics/cairo/cairo.inc 
>> b/meta/recipes-graphics/cairo/cairo.inc
>> index 8e1e2e1b88..fd376951bd 100644
>> --- a/meta/recipes-graphics/cairo/cairo.inc
>> +++ b/meta/recipes-graphics/cairo/cairo.inc
>> @@ -22,7 +22,8 @@ X11DEPENDS = "virtual/libx11 libsm libxrender libxext"
>>  DEPENDS = "libpng fontconfig pixman glib-2.0 zlib"
>>
>>  PACKAGECONFIG ??= "${@bb.utils.contains('DISTRO_FEATURES', 'x11', 'x11 
>> xcb', '', d)} \
>> -           ${@bb.utils.filter('DISTRO_FEATURES', 'directfb', d)}"
>> +           ${@bb.utils.filter('DISTRO_FEATURES', 'directfb', d)} \
>> +           ${@bb.utils.filter('DISTRO_FEATURES', 'opengl', d)}"
>>
>>  PACKAGECONFIG[x11] = "--with-x=yes -enable-xlib,--with-x=no 
>> --disable-xlib,${X11DEPENDS}"
>>  PACKAGECONFIG[xcb] = "--enable-xcb,--disable-xcb,libxcb"
>> @@ -30,6 +31,7 @@ PACKAGECONFIG[directfb] = "--enable-directfb=yes,,directfb"
>>  PACKAGECONFIG[valgrind] = 
>> "--enable-valgrind=yes,--disable-valgrind,valgrind"
>>  PACKAGECONFIG[egl] = "--enable-egl=yes,--disable-egl,virtual/egl"
>>  PACKAGECONFIG[glesv2] = "--enable-glesv2,--disable-glesv2,virtual/libgles2"
>> +PACKAGECONFIG[opengl] = "--enable-gl,--disable-gl,virtual/libgl"
> 
> Since the opengl distro feature can imply either OpenGL or OpenGL ES
> support, it doesn't seem right to use it to select one over the other
> (ie for recipes which support both, shouldn't the choice between
> OpenGL or OpenGL ES be independent of the opengl distro feature) ?

at some point we differentiated between opengl and gles at distro level
atleast for RDK, but I agree that marking opengl as a distro feature
encompassing all OpenGL implementations is better. Although, in this
particular case all openGL implementations can be enabled simultaneously
so may be we can enable egl and gles too with same knob. Alternatively
We can leave the packageconfig knob disabled which will make it same
defaults effectively ( which is disabled)

> 
>>
>>  #check for TARGET_FPU=soft and inform configure of the result so it can 
>> disable some floating points
>>  require cairo-fpu.inc
>> --
>> 2.14.0
>>
>> --
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-core


Attachment: signature.asc
Description: OpenPGP digital signature

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

Reply via email to