On Wed, Aug 9, 2017 at 11:24 AM, Andre McCurdy <armccu...@gmail.com> wrote:
> On Wed, Aug 9, 2017 at 11:11 AM, Khem Raj <raj.k...@gmail.com> wrote:
>> 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
>
> According to the tests in the configure script it looks like a choice
> between OpenGL or OpenGL ES, not both.
>
>> 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)
>
> Either leave it disabled or enable it conditionally based on both the
> opengl and x11 distro features?
could be a way forward.
>
>>
>>>
>>>>
>>>> #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
>>
>>
--
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core
