[PATCH] directfb: don't patch pkg-config files

[PATCH] directfb: don't patch pkg-config files

[André Draszik]

We are currently getting build failures of projects that rely on
being able to access DirectFB's internal include directories, as
returned via pkg-config, since the include paths returned by
pkg-config are incomplete.

The reason for that is the patch that is being removed with this
change. It modified the cflags returned by pkg-config in an
incorrect way, causing us to miss important include paths:

For reference, pkg-config output with incorrect patch applied:
  ad@bril0118 #513 ~> pkg-config --cflags directfb-internal
-D_GNU_SOURCE -D_REENTRANT -I<builddir>/tmp/sysroots/<machine>/usr/include/directfb -I<builddir>/tmp/sysroots/<machine>/usr/include

Now, with the incorrect patch removed, the output is as expected:
  ad@bril0118 #514 ~> pkg-config --cflags directfb-internal
-D_GNU_SOURCE -D_REENTRANT -I<builddir>/tmp/sysroots/<machine>/usr/include/directfb-internal -I<builddir>/tmp/sysroots/<machine>/usr/include/directfb

Overall, the removed patch is not needed - pkg-config does the right
thing these days and we can simply use the correctly working upstream
versions of all DirectFB .pc files.

Signed-off-by: André Draszik <andre.draszik@linaro.org>
---
 meta/recipes-graphics/directfb/directfb.inc        |  1 -
 .../directfb-1.2.x-fix-pkgconfig-cflags.patch      | 48 ----------------------
 2 files changed, 49 deletions(-)
 delete mode 100644 meta/recipes-graphics/directfb/directfb/directfb-1.2.x-fix-pkgconfig-cflags.patch

diff --git a/meta/recipes-graphics/directfb/directfb.inc b/meta/recipes-graphics/directfb/directfb.inc
index cff8a7f..97a4c55 100644
--- a/meta/recipes-graphics/directfb/directfb.inc
+++ b/meta/recipes-graphics/directfb/directfb.inc
@@ -14,7 +14,6 @@ DEPENDS = "jpeg libpng freetype zlib tslib"
 INC_PR = "r0"
 
 SRC_URI = "http://directfb.org/downloads/Core/DirectFB-1.6/DirectFB-${PV}.tar.gz \
-           file://directfb-1.2.x-fix-pkgconfig-cflags.patch \
            file://configurefix.patch \
            file://mesa9.patch"
 
diff --git a/meta/recipes-graphics/directfb/directfb/directfb-1.2.x-fix-pkgconfig-cflags.patch b/meta/recipes-graphics/directfb/directfb/directfb-1.2.x-fix-pkgconfig-cflags.patch
deleted file mode 100644
index ee60718..0000000
--- a/meta/recipes-graphics/directfb/directfb/directfb-1.2.x-fix-pkgconfig-cflags.patch
+++ /dev/null
@@ -1,48 +0,0 @@
-directfb: Get this patch from Openembedded
-
-Upstream-Status: Inappropriate [configuration]
-Signed-off-by: Xiaofeng Yan <xiaofeng.yan@windriver.com>
-
-diff -Nur DirectFB-1.4.15/directfb-internal.pc.in DirectFB-1.4.15.new//directfb-internal.pc.in
---- DirectFB-1.4.15/directfb-internal.pc.in	2011-09-29 17:51:21.000000000 +0800
-+++ DirectFB-1.4.15.new//directfb-internal.pc.in	2011-11-03 15:14:37.000000000 +0800
-@@ -2,10 +2,10 @@
- exec_prefix=@exec_prefix@
- moduledir=@MODULEDIR@
- moduledirname=@MODULEDIRNAME@
--includedir=@INTERNALINCLUDEDIR@
-+includedir=@includedir@
- 
- Name: DirectFB-Internal
- Description: Third party module support package for DirectFB.
- Version: @VERSION@
- Requires: directfb = @VERSION@
--Cflags: @DFB_INTERNAL_CFLAGS@ -I@INTERNALINCLUDEDIR@
-+Cflags: @DFB_INTERNAL_CFLAGS@ -I${includedir}/directfb -I${includedir}
-diff -Nur DirectFB-1.4.15/directfb.pc.in DirectFB-1.4.15.new//directfb.pc.in
---- DirectFB-1.4.15/directfb.pc.in	2011-09-29 17:51:21.000000000 +0800
-+++ DirectFB-1.4.15.new//directfb.pc.in	2011-11-03 15:15:55.000000000 +0800
-@@ -9,4 +9,4 @@
- Requires: @DEP_VOODOO@ fusion direct
- Libs: -L${libdir} -ldirectfb @THREADLIB@ @OSX_LIBS@
- Libs.private: -L${libdir} @LIBM@ @DYNLIB@ @ZLIB_LIBS@
--Cflags: @THREADFLAGS@ -I@INCLUDEDIR@
-+Cflags: @THREADFLAGS@ -I${includedir}/directfb
-diff -Nur DirectFB-1.4.15/lib/fusion/fusion.pc.in DirectFB-1.4.15.new//lib/fusion/fusion.pc.in
---- DirectFB-1.4.15/lib/fusion/fusion.pc.in	2011-09-29 17:51:21.000000000 +0800
-+++ DirectFB-1.4.15.new//lib/fusion/fusion.pc.in	2011-11-03 15:16:46.000000000 +0800
-@@ -8,4 +8,4 @@
- Version: @VERSION@
- Requires: direct
- Libs: -L${libdir} -lfusion
--Cflags: -I@INCLUDEDIR@
-+Cflags: -I${includedir}/directfb -I${includedir}
-diff -Nur DirectFB-1.4.15/lib/voodoo/voodoo.pc.in DirectFB-1.4.15.new//lib/voodoo/voodoo.pc.in
---- DirectFB-1.4.15/lib/voodoo/voodoo.pc.in	2011-09-29 17:51:21.000000000 +0800
-+++ DirectFB-1.4.15.new//lib/voodoo/voodoo.pc.in	2011-11-03 15:17:34.000000000 +0800
-@@ -8,4 +8,4 @@
- Version: @VERSION@
- Requires: direct
- Libs: -L${libdir} -lvoodoo
--Cflags: -I@INCLUDEDIR@
-+Cflags: -I${includedir}/directfb -I${includedir}
-- 
1.8.2

Re: [PATCH] directfb: don't patch pkg-config files

[Nicolas Dechesne]

[-- Attachment #1: Type: text/plain, Size: 1602 bytes --]

On Fri, Jul 19, 2013 at 10:39 AM, André Draszik <andre.draszik@linaro.org>wrote:

> We are currently getting build failures of projects that rely on
> being able to access DirectFB's internal include directories, as
> returned via pkg-config, since the include paths returned by
> pkg-config are incomplete.
>
> The reason for that is the patch that is being removed with this
> change. It modified the cflags returned by pkg-config in an
> incorrect way, causing us to miss important include paths:
>
> For reference, pkg-config output with incorrect patch applied:
>   ad@bril0118 #513 ~> pkg-config --cflags directfb-internal
> -D_GNU_SOURCE -D_REENTRANT
> -I<builddir>/tmp/sysroots/<machine>/usr/include/directfb
> -I<builddir>/tmp/sysroots/<machine>/usr/include
>
> Now, with the incorrect patch removed, the output is as expected:
>   ad@bril0118 #514 ~> pkg-config --cflags directfb-internal
> -D_GNU_SOURCE -D_REENTRANT
> -I<builddir>/tmp/sysroots/<machine>/usr/include/directfb-internal
> -I<builddir>/tmp/sysroots/<machine>/usr/include/directfb
>
> Overall, the removed patch is not needed - pkg-config does the right
> thing these days and we can simply use the correctly working upstream
> versions of all DirectFB .pc files.
>
> Signed-off-by: André Draszik <andre.draszik@linaro.org>
>

i reviewed and tested this patch against a QT4e image build. it looks good
to me, and that offending patch being removed seems to come from quite
ancient stuff.

Tested-by: Nicolas Dechesne <nicolas.dechesne@linaro.org>

any chance this can get into 'dylan', too?

[-- Attachment #2: Type: text/html, Size: 2457 bytes --]

Re: [PATCH] directfb: don't patch pkg-config files

[Otavio Salvador]

On Fri, Jul 19, 2013 at 11:46 AM, Nicolas Dechesne
<nicolas.dechesne@linaro.org> wrote:
>
> On Fri, Jul 19, 2013 at 10:39 AM, André Draszik <andre.draszik@linaro.org>
> wrote:
>>
>> We are currently getting build failures of projects that rely on
>> being able to access DirectFB's internal include directories, as
>> returned via pkg-config, since the include paths returned by
>> pkg-config are incomplete.
>>
>> The reason for that is the patch that is being removed with this
>> change. It modified the cflags returned by pkg-config in an
>> incorrect way, causing us to miss important include paths:
>>
>> For reference, pkg-config output with incorrect patch applied:
>>   ad@bril0118 #513 ~> pkg-config --cflags directfb-internal
>> -D_GNU_SOURCE -D_REENTRANT
>> -I<builddir>/tmp/sysroots/<machine>/usr/include/directfb
>> -I<builddir>/tmp/sysroots/<machine>/usr/include
>>
>> Now, with the incorrect patch removed, the output is as expected:
>>   ad@bril0118 #514 ~> pkg-config --cflags directfb-internal
>> -D_GNU_SOURCE -D_REENTRANT
>> -I<builddir>/tmp/sysroots/<machine>/usr/include/directfb-internal
>> -I<builddir>/tmp/sysroots/<machine>/usr/include/directfb
>>
>> Overall, the removed patch is not needed - pkg-config does the right
>> thing these days and we can simply use the correctly working upstream
>> versions of all DirectFB .pc files.
>>
>> Signed-off-by: André Draszik <andre.draszik@linaro.org>
>
>
> i reviewed and tested this patch against a QT4e image build. it looks good
> to me, and that offending patch being removed seems to come from quite
> ancient stuff.
>
> Tested-by: Nicolas Dechesne <nicolas.dechesne@linaro.org>
>
> any chance this can get into 'dylan', too?

Paul, I think it is your business ;-)

--
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://projetos.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750

Re: [PATCH] directfb: don't patch pkg-config files

[Paul Eggleton]

On Friday 19 July 2013 14:52:00 Otavio Salvador wrote:
> On Fri, Jul 19, 2013 at 11:46 AM, Nicolas Dechesne
> 
> <nicolas.dechesne@linaro.org> wrote:
> > On Fri, Jul 19, 2013 at 10:39 AM, André Draszik <andre.draszik@linaro.org>
> > 
> > wrote:
> >> We are currently getting build failures of projects that rely on
> >> being able to access DirectFB's internal include directories, as
> >> returned via pkg-config, since the include paths returned by
> >> pkg-config are incomplete.
> >> 
> >> The reason for that is the patch that is being removed with this
> >> change. It modified the cflags returned by pkg-config in an
> >> incorrect way, causing us to miss important include paths:
> >> 
> >> For reference, pkg-config output with incorrect patch applied:
> >>   ad@bril0118 #513 ~> pkg-config --cflags directfb-internal
> >> 
> >> -D_GNU_SOURCE -D_REENTRANT
> >> -I<builddir>/tmp/sysroots/<machine>/usr/include/directfb
> >> -I<builddir>/tmp/sysroots/<machine>/usr/include
> >> 
> >> Now, with the incorrect patch removed, the output is as expected:
> >>   ad@bril0118 #514 ~> pkg-config --cflags directfb-internal
> >> 
> >> -D_GNU_SOURCE -D_REENTRANT
> >> -I<builddir>/tmp/sysroots/<machine>/usr/include/directfb-internal
> >> -I<builddir>/tmp/sysroots/<machine>/usr/include/directfb
> >> 
> >> Overall, the removed patch is not needed - pkg-config does the right
> >> thing these days and we can simply use the correctly working upstream
> >> versions of all DirectFB .pc files.
> >> 
> >> Signed-off-by: André Draszik <andre.draszik@linaro.org>
> > 
> > i reviewed and tested this patch against a QT4e image build. it looks good
> > to me, and that offending patch being removed seems to come from quite
> > ancient stuff.
> > 
> > Tested-by: Nicolas Dechesne <nicolas.dechesne@linaro.org>
> > 
> > any chance this can get into 'dylan', too?
> 
> Paul, I think it is your business ;-)

Indeed. I've added it to my paule/dylan-next branch for testing.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre