Patchwork [1/4] xserver-xorg: Add mesa-dri to DEPENDS

login
register
mail settings
Submitter Khem Raj
Date Oct. 20, 2011, 7:36 p.m.
Message ID <15763124703d12326c8ba4d296a2b2680528c2da.1319139126.git.raj.khem@gmail.com>
Download mbox | patch
Permalink /patch/13631/
State New, archived
Headers show

Comments

Khem Raj - Oct. 20, 2011, 7:36 p.m.
Fixes the compile problems e.g.

| checking for GL... yes
| checking for DRIPROTO... yes
| checking for DRI... no
| configure: error: Package requirements (glproto >= 1.4.14 dri >=
7.8.0) were not met:
|
| No package 'dri' found
|
| Consider adjusting the PKG_CONFIG_PATH environment variable if you
| installed software in a non-standard prefix.

Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
 .../xorg-xserver/xserver-xorg_1.11.1.bb            |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)
Martin Jansa - Oct. 20, 2011, 7:44 p.m.
On Thu, Oct 20, 2011 at 12:36:01PM -0700, Khem Raj wrote:
> Fixes the compile problems e.g.
> 
> | checking for GL... yes
> | checking for DRIPROTO... yes
> | checking for DRI... no
> | configure: error: Package requirements (glproto >= 1.4.14 dri >=
> 7.8.0) were not met:
> |
> | No package 'dri' found
> |
> | Consider adjusting the PKG_CONFIG_PATH environment variable if you
> | installed software in a non-standard prefix.
> 
> Signed-off-by: Khem Raj <raj.khem@gmail.com>
> ---
>  .../xorg-xserver/xserver-xorg_1.11.1.bb            |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.11.1.bb b/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.11.1.bb
> index 0f94fe2..2aa71e5 100644
> --- a/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.11.1.bb
> +++ b/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.11.1.bb
> @@ -1,2 +1,3 @@
>  require xserver-xorg.inc
>  require xserver-xorg-${PV}.inc
> +DEPENDS += "mesa-dri"

You can do this in xserver-xorg.inc
just change
LIB_DEPS += "virtual/libgl" to
LIB_DEPS += "mesa-dri"
Koen Kooi - Oct. 20, 2011, 9:42 p.m.
Op 20 okt. 2011, om 21:44 heeft Martin Jansa het volgende geschreven:

> On Thu, Oct 20, 2011 at 12:36:01PM -0700, Khem Raj wrote:
>> Fixes the compile problems e.g.
>> 
>> | checking for GL... yes
>> | checking for DRIPROTO... yes
>> | checking for DRI... no
>> | configure: error: Package requirements (glproto >= 1.4.14 dri >=
>> 7.8.0) were not met:
>> |
>> | No package 'dri' found
>> |
>> | Consider adjusting the PKG_CONFIG_PATH environment variable if you
>> | installed software in a non-standard prefix.
>> 
>> Signed-off-by: Khem Raj <raj.khem@gmail.com>
>> ---
>> .../xorg-xserver/xserver-xorg_1.11.1.bb            |    1 +
>> 1 files changed, 1 insertions(+), 0 deletions(-)
>> 
>> diff --git a/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.11.1.bb b/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.11.1.bb
>> index 0f94fe2..2aa71e5 100644
>> --- a/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.11.1.bb
>> +++ b/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.11.1.bb
>> @@ -1,2 +1,3 @@
>> require xserver-xorg.inc
>> require xserver-xorg-${PV}.inc
>> +DEPENDS += "mesa-dri"
> 
> You can do this in xserver-xorg.inc
> just change
> LIB_DEPS += "virtual/libgl" to
> LIB_DEPS += "mesa-dri"

that will drag it in for -lite as well, is that OK?

regards,

Koen
Martin Jansa - Oct. 20, 2011, 9:48 p.m.
On Thu, Oct 20, 2011 at 11:42:27PM +0200, Koen Kooi wrote:
> 
> Op 20 okt. 2011, om 21:44 heeft Martin Jansa het volgende geschreven:
> 
> > On Thu, Oct 20, 2011 at 12:36:01PM -0700, Khem Raj wrote:
> >> Fixes the compile problems e.g.
> >> 
> >> | checking for GL... yes
> >> | checking for DRIPROTO... yes
> >> | checking for DRI... no
> >> | configure: error: Package requirements (glproto >= 1.4.14 dri >=
> >> 7.8.0) were not met:
> >> |
> >> | No package 'dri' found
> >> |
> >> | Consider adjusting the PKG_CONFIG_PATH environment variable if you
> >> | installed software in a non-standard prefix.
> >> 
> >> Signed-off-by: Khem Raj <raj.khem@gmail.com>
> >> ---
> >> .../xorg-xserver/xserver-xorg_1.11.1.bb            |    1 +
> >> 1 files changed, 1 insertions(+), 0 deletions(-)
> >> 
> >> diff --git a/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.11.1.bb b/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.11.1.bb
> >> index 0f94fe2..2aa71e5 100644
> >> --- a/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.11.1.bb
> >> +++ b/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.11.1.bb
> >> @@ -1,2 +1,3 @@
> >> require xserver-xorg.inc
> >> require xserver-xorg-${PV}.inc
> >> +DEPENDS += "mesa-dri"
> > 
> > You can do this in xserver-xorg.inc
> > just change
> > LIB_DEPS += "virtual/libgl" to
> > LIB_DEPS += "mesa-dri"
> 
> that will drag it in for -lite as well, is that OK?

No it will not, this is not xserver-xorg-common.inc

I'll send patch for default-providers.inc to switch default to mesa-dri
too.
Khem Raj - Oct. 20, 2011, 10:39 p.m.
On Thu, Oct 20, 2011 at 12:44 PM, Martin Jansa <martin.jansa@gmail.com> wrote:
> On Thu, Oct 20, 2011 at 12:36:01PM -0700, Khem Raj wrote:
>> Fixes the compile problems e.g.
>>
>> | checking for GL... yes
>> | checking for DRIPROTO... yes
>> | checking for DRI... no
>> | configure: error: Package requirements (glproto >= 1.4.14 dri >=
>> 7.8.0) were not met:
>> |
>> | No package 'dri' found
>> |
>> | Consider adjusting the PKG_CONFIG_PATH environment variable if you
>> | installed software in a non-standard prefix.
>>
>> Signed-off-by: Khem Raj <raj.khem@gmail.com>
>> ---
>>  .../xorg-xserver/xserver-xorg_1.11.1.bb            |    1 +
>>  1 files changed, 1 insertions(+), 0 deletions(-)
>>
>> diff --git a/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.11.1.bb b/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.11.1.bb
>> index 0f94fe2..2aa71e5 100644
>> --- a/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.11.1.bb
>> +++ b/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.11.1.bb
>> @@ -1,2 +1,3 @@
>>  require xserver-xorg.inc
>>  require xserver-xorg-${PV}.inc
>> +DEPENDS += "mesa-dri"
>
> You can do this in xserver-xorg.inc
> just change
> LIB_DEPS += "virtual/libgl" to
> LIB_DEPS += "mesa-dri"

thanks. I have updates the patch accordingly in the pull request.
Koen Kooi - Oct. 21, 2011, 5:58 a.m.
Op 20 okt. 2011, om 23:48 heeft Martin Jansa het volgende geschreven:

> On Thu, Oct 20, 2011 at 11:42:27PM +0200, Koen Kooi wrote:
>> 
>> Op 20 okt. 2011, om 21:44 heeft Martin Jansa het volgende geschreven:
>> 
>>> On Thu, Oct 20, 2011 at 12:36:01PM -0700, Khem Raj wrote:
>>>> Fixes the compile problems e.g.
>>>> 
>>>> | checking for GL... yes
>>>> | checking for DRIPROTO... yes
>>>> | checking for DRI... no
>>>> | configure: error: Package requirements (glproto >= 1.4.14 dri >=
>>>> 7.8.0) were not met:
>>>> |
>>>> | No package 'dri' found
>>>> |
>>>> | Consider adjusting the PKG_CONFIG_PATH environment variable if you
>>>> | installed software in a non-standard prefix.
>>>> 
>>>> Signed-off-by: Khem Raj <raj.khem@gmail.com>
>>>> ---
>>>> .../xorg-xserver/xserver-xorg_1.11.1.bb            |    1 +
>>>> 1 files changed, 1 insertions(+), 0 deletions(-)
>>>> 
>>>> diff --git a/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.11.1.bb b/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.11.1.bb
>>>> index 0f94fe2..2aa71e5 100644
>>>> --- a/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.11.1.bb
>>>> +++ b/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.11.1.bb
>>>> @@ -1,2 +1,3 @@
>>>> require xserver-xorg.inc
>>>> require xserver-xorg-${PV}.inc
>>>> +DEPENDS += "mesa-dri"
>>> 
>>> You can do this in xserver-xorg.inc
>>> just change
>>> LIB_DEPS += "virtual/libgl" to
>>> LIB_DEPS += "mesa-dri"
>> 
>> that will drag it in for -lite as well, is that OK?
> 
> No it will not, this is not xserver-xorg-common.inc

Ah, I got confused by 'require xserver-xorg-${PV}.inc'. Time to get some more coffee.

regards,

Koen.

Patch

diff --git a/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.11.1.bb b/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.11.1.bb
index 0f94fe2..2aa71e5 100644
--- a/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.11.1.bb
+++ b/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.11.1.bb
@@ -1,2 +1,3 @@ 
 require xserver-xorg.inc
 require xserver-xorg-${PV}.inc
+DEPENDS += "mesa-dri"