[PATCH] libdrm: Remove Cairo dependency

[PATCH] libdrm: Remove Cairo dependency

[Ross Burton]

From: Daniel Stone <daniel@fooishbar.org>

This causes a build loop, when DRM depends on Cairo depends on Mesa
depends on DRM.  We can safely remove it as it's only one libdrm example
program which uses Cairo, which we won't be needing.  At least it's not
worth the build loop.

Signed-off-by: Daniel Stone <daniel@fooishbar.org>
Signed-off-by: Ross Burton <ross.burton@intel.com>
---
 meta/recipes-graphics/drm/libdrm.inc |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-graphics/drm/libdrm.inc b/meta/recipes-graphics/drm/libdrm.inc
index 5ddab85..cc09791 100644
--- a/meta/recipes-graphics/drm/libdrm.inc
+++ b/meta/recipes-graphics/drm/libdrm.inc
@@ -9,9 +9,9 @@ LICENSE = "MIT"
 LIC_FILES_CHKSUM = "file://xf86drm.c;beginline=9;endline=32;md5=c8a3b961af7667c530816761e949dc71"
 SRC_URI = "http://dri.freedesktop.org/libdrm/libdrm-${PV}.tar.bz2"
 PROVIDES = "drm"
-DEPENDS = "libpthread-stubs udev cairo"
+DEPENDS = "libpthread-stubs udev"
 
-INC_PR = "r2"
+INC_PR = "r3"
 
 #libpciaccess is required starting from libdrm 2.4.26
 DEPENDS += " libpciaccess"
-- 
1.7.10

Re: [PATCH] libdrm: Remove Cairo dependency

[Martin Jansa]

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

On Mon, Oct 01, 2012 at 12:56:09PM +0100, Ross Burton wrote:
> From: Daniel Stone <daniel@fooishbar.org>
> 
> This causes a build loop, when DRM depends on Cairo depends on Mesa
> depends on DRM.  We can safely remove it as it's only one libdrm example
> program which uses Cairo, which we won't be needing.  At least it's not
> worth the build loop.
> 
> Signed-off-by: Daniel Stone <daniel@fooishbar.org>
> Signed-off-by: Ross Burton <ross.burton@intel.com>
> ---
>  meta/recipes-graphics/drm/libdrm.inc |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/meta/recipes-graphics/drm/libdrm.inc b/meta/recipes-graphics/drm/libdrm.inc
> index 5ddab85..cc09791 100644
> --- a/meta/recipes-graphics/drm/libdrm.inc
> +++ b/meta/recipes-graphics/drm/libdrm.inc
> @@ -9,9 +9,9 @@ LICENSE = "MIT"
>  LIC_FILES_CHKSUM = "file://xf86drm.c;beginline=9;endline=32;md5=c8a3b961af7667c530816761e949dc71"
>  SRC_URI = "http://dri.freedesktop.org/libdrm/libdrm-${PV}.tar.bz2"
>  PROVIDES = "drm"
> -DEPENDS = "libpthread-stubs udev cairo"
> +DEPENDS = "libpthread-stubs udev"
>  
> -INC_PR = "r2"
> +INC_PR = "r3"
>  
>  #libpciaccess is required starting from libdrm 2.4.26
>  DEPENDS += " libpciaccess"

shouldn't we disable that example explicitly, so that the output is not
different when cairo gets built before libdrm?

Cheers,

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

Re: [PATCH] libdrm: Remove Cairo dependency

[Burton, Ross]

On 1 October 2012 13:02, Martin Jansa <martin.jansa@gmail.com> wrote:
> shouldn't we disable that example explicitly, so that the output is not
> different when cairo gets built before libdrm?

The test (tests/modetest/) is noinst, so the output doesn't actually change.

Ross

Re: [PATCH] libdrm: Remove Cairo dependency

[Richard Purdie]

On Mon, 2012-10-01 at 13:04 +0100, Burton, Ross wrote:
> On 1 October 2012 13:02, Martin Jansa <martin.jansa@gmail.com> wrote:
> > shouldn't we disable that example explicitly, so that the output is not
> > different when cairo gets built before libdrm?
> 
> The test (tests/modetest/) is noinst, so the output doesn't actually change.

Shouldn't we just disable the things to gain a small bit of build speed
if they don't matter? :)

Cheers,

Richard

Re: [PATCH] libdrm: Remove Cairo dependency

[Burton, Ross]

On 2 October 2012 11:56, Richard Purdie
<richard.purdie@linuxfoundation.org> wrote:
> Shouldn't we just disable the things to gain a small bit of build speed
> if they don't matter? :)

That means carrying a patch and the maintenance burden when upgrading.
 I'm not entirely sure you'll notice all ten lines of Cairo code being
compiled if cairo happens to be installed when libdrm2 builds.

Ross

Re: [PATCH] libdrm: Remove Cairo dependency

[Richard Purdie]

On Tue, 2012-10-02 at 11:58 +0100, Burton, Ross wrote:
> On 2 October 2012 11:56, Richard Purdie
> <richard.purdie@linuxfoundation.org> wrote:
> > Shouldn't we just disable the things to gain a small bit of build speed
> > if they don't matter? :)
> 
> That means carrying a patch and the maintenance burden when upgrading.
>  I'm not entirely sure you'll notice all ten lines of Cairo code being
> compiled if cairo happens to be installed when libdrm2 builds.

I don't feel strongly about it but it sounds like the kind of thing it
would be useful to switch off and upstream would probably take such a
patch?

Cheers,

Richard

Re: [PATCH] libdrm: Remove Cairo dependency

[Saul Wold]

On 10/01/2012 04:56 AM, Ross Burton wrote:
> From: Daniel Stone <daniel@fooishbar.org>
>
> This causes a build loop, when DRM depends on Cairo depends on Mesa
> depends on DRM.  We can safely remove it as it's only one libdrm example
> program which uses Cairo, which we won't be needing.  At least it's not
> worth the build loop.
>
> Signed-off-by: Daniel Stone <daniel@fooishbar.org>
> Signed-off-by: Ross Burton <ross.burton@intel.com>
> ---
>   meta/recipes-graphics/drm/libdrm.inc |    4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/meta/recipes-graphics/drm/libdrm.inc b/meta/recipes-graphics/drm/libdrm.inc
> index 5ddab85..cc09791 100644
> --- a/meta/recipes-graphics/drm/libdrm.inc
> +++ b/meta/recipes-graphics/drm/libdrm.inc
> @@ -9,9 +9,9 @@ LICENSE = "MIT"
>   LIC_FILES_CHKSUM = "file://xf86drm.c;beginline=9;endline=32;md5=c8a3b961af7667c530816761e949dc71"
>   SRC_URI = "http://dri.freedesktop.org/libdrm/libdrm-${PV}.tar.bz2"
>   PROVIDES = "drm"
> -DEPENDS = "libpthread-stubs udev cairo"
> +DEPENDS = "libpthread-stubs udev"
>
> -INC_PR = "r2"
> +INC_PR = "r3"
>
>   #libpciaccess is required starting from libdrm 2.4.26
>   DEPENDS += " libpciaccess"
>
Merged into OE-Core

Thanks
	Sau!