On Wed, Apr 16, 2014 at 12:00 AM, Robert Yang <liezhi.y...@windriver.com> wrote:
>
>
> On 04/14/2014 10:51 PM, Valentin Popa wrote:
>>
>> Even if 'egl' is in PACKAGECONFIG, mesa egl support
>> can be disabled explicitly (changing configure flags
>> using a .bbappend, for example).
>> On dora, meta-fsl-arm is an example of this kind.
>> On master there are no known cases, and we should
>> encourge package configuration through PACKAGECONFIG.
>>
>> This patch adds another check for the existence
>> of eglplatform.h before 'sed' can alter it.
>>
>> Signed-off-by: Valentin Popa <valentin.p...@intel.com>
>
>
> Reviewed and Tested by: Robert Yang <liezhi.y...@windriver.com>
>
> // Robert
>
>
>> ---
>> meta/recipes-graphics/mesa/mesa_9.1.6.bb | 4 +++-
>> meta/recipes-graphics/mesa/mesa_git.bb | 4 +++-
>> 2 files changed, 6 insertions(+), 2 deletions(-)
>>
>> diff --git a/meta/recipes-graphics/mesa/mesa_9.1.6.bb
>> b/meta/recipes-graphics/mesa/mesa_9.1.6.bb
>> index 6e9cd82..388cfd7 100644
>> --- a/meta/recipes-graphics/mesa/mesa_9.1.6.bb
>> +++ b/meta/recipes-graphics/mesa/mesa_9.1.6.bb
>> @@ -19,6 +19,8 @@ S = "${WORKDIR}/Mesa-${PV}"
>> #make eglplatform.h independent of MESA_EGL_NO_X11_HEADER
>> do_install_append() {
>> if ${@base_contains('PACKAGECONFIG', 'egl', 'true', 'false', d)};
>> then
>> - sed -i -e 's/^#ifdef MESA_EGL_NO_X11_HEADERS/#if
>> ${@base_contains('DISTRO_FEATURES', 'x11', '0', '1', d)}/'
>> ${D}${includedir}/EGL/eglplatform.h
>> + if [ -e "${D}${includedir}/EGL/eglplatform.h" ]; then
>> + sed -i -e 's/^#ifdef MESA_EGL_NO_X11_HEADERS/#if
>> ${@base_contains('DISTRO_FEATURES', 'x11', '0', '1', d)}/'
>> ${D}${includedir}/EGL/eglplatform.h
>> + fi
>> fi
>> }
>> \ No newline at end of file
>> diff --git a/meta/recipes-graphics/mesa/mesa_git.bb
>> b/meta/recipes-graphics/mesa/mesa_git.bb
>> index 1babcc0..714911f 100644
>> --- a/meta/recipes-graphics/mesa/mesa_git.bb
>> +++ b/meta/recipes-graphics/mesa/mesa_git.bb
>> @@ -23,6 +23,8 @@ S = "${WORKDIR}/git"
>> #make eglplatform.h independent of MESA_EGL_NO_X11_HEADER
>> do_install_append() {
>> if ${@base_contains('PACKAGECONFIG', 'egl', 'true', 'false', d)};
>> then
>> - sed -i -e 's/^#ifdef MESA_EGL_NO_X11_HEADERS/#if
>> ${@base_contains('DISTRO_FEATURES', 'x11', '0', '1', d)}/'
>> ${D}${includedir}/EGL/eglplatform.h
>> + if [ -e "${D}${includedir}/EGL/eglplatform.h" ]; then
>> + sed -i -e 's/^#ifdef MESA_EGL_NO_X11_HEADERS/#if
>> ${@base_contains('DISTRO_FEATURES', 'x11', '0', '1', d)}/'
>> ${D}${includedir}/EGL/eglplatform.h
>> + fi
>> fi
>> }
>>
So WHAT is holding a bugfix for a regression to be merged? I am very
disappointed with the maintenance in the Dora branch.
1. a change has been added without testing other layers hosted in YP.
2. it has been merged without being ran in YP AB.
3. I reported the issue in the same day it has been merged
4. it has been 10 days and this has not yet been reverted/fixed.
So CAN THIS BE MERGED PLEASE?
--
Otavio Salvador O.S. Systems
http://www.ossystems.com.br http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854 Mobile: +1 (347) 903-9750
--
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core
