hicolor-icon-theme 0.12: add BBCLASSEXTEND=native for xfce tools

Submitted by Koen Kooi on Nov. 10, 2011, 12:44 p.m.

Details

Message ID 1320929044-13077-1-git-send-email-koen@dominion.thruhere.net
State New
Headers show

Commit Message

Koen Kooi Nov. 10, 2011, 12:44 p.m.
Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
---
 .../hicolor-icon-theme/hicolor-icon-theme_0.12.bb  |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

Patch hide | download patch | download mbox

diff --git a/meta/recipes-gnome/hicolor-icon-theme/hicolor-icon-theme_0.12.bb b/meta/recipes-gnome/hicolor-icon-theme/hicolor-icon-theme_0.12.bb
index 759c9d8..aeb89c9 100644
--- a/meta/recipes-gnome/hicolor-icon-theme/hicolor-icon-theme_0.12.bb
+++ b/meta/recipes-gnome/hicolor-icon-theme/hicolor-icon-theme_0.12.bb
@@ -5,6 +5,8 @@  BUGTRACKER = "https://bugs.freedesktop.org/"
 LICENSE = "GPLv2"
 LIC_FILES_CHKSUM = "file://COPYING;md5=f08a446809913fc9b3c718f0eaea0426"
 
+BBCLASSEXTEND = "native"
+
 SECTION = "unknown"
 inherit gnome allarch
 

Comments

Richard Purdie Nov. 10, 2011, 1:20 p.m.
On Thu, 2011-11-10 at 13:44 +0100, Koen Kooi wrote:
> Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
> ---
>  .../hicolor-icon-theme/hicolor-icon-theme_0.12.bb  |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/meta/recipes-gnome/hicolor-icon-theme/hicolor-icon-theme_0.12.bb b/meta/recipes-gnome/hicolor-icon-theme/hicolor-icon-theme_0.12.bb
> index 759c9d8..aeb89c9 100644
> --- a/meta/recipes-gnome/hicolor-icon-theme/hicolor-icon-theme_0.12.bb
> +++ b/meta/recipes-gnome/hicolor-icon-theme/hicolor-icon-theme_0.12.bb
> @@ -5,6 +5,8 @@ BUGTRACKER = "https://bugs.freedesktop.org/"
>  LICENSE = "GPLv2"
>  LIC_FILES_CHKSUM = "file://COPYING;md5=f08a446809913fc9b3c718f0eaea0426"
>  
> +BBCLASSEXTEND = "native"
> +
>  SECTION = "unknown"
>  inherit gnome allarch

This is interesting dilemma since this is an allarch recipe. What issue
were you seeing to need this? Could we add the allarch sysroot directory
to PATH to resolve this instead? We could add a PROVIDES
"hicolor-icon-theme-native" too...

Cheers,

Richard
Koen Kooi Nov. 10, 2011, 1:38 p.m.
Op 10 nov. 2011, om 14:20 heeft Richard Purdie het volgende geschreven:

> On Thu, 2011-11-10 at 13:44 +0100, Koen Kooi wrote:
>> Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
>> ---
>> .../hicolor-icon-theme/hicolor-icon-theme_0.12.bb  |    2 ++
>> 1 files changed, 2 insertions(+), 0 deletions(-)
>> 
>> diff --git a/meta/recipes-gnome/hicolor-icon-theme/hicolor-icon-theme_0.12.bb b/meta/recipes-gnome/hicolor-icon-theme/hicolor-icon-theme_0.12.bb
>> index 759c9d8..aeb89c9 100644
>> --- a/meta/recipes-gnome/hicolor-icon-theme/hicolor-icon-theme_0.12.bb
>> +++ b/meta/recipes-gnome/hicolor-icon-theme/hicolor-icon-theme_0.12.bb
>> @@ -5,6 +5,8 @@ BUGTRACKER = "https://bugs.freedesktop.org/"
>> LICENSE = "GPLv2"
>> LIC_FILES_CHKSUM = "file://COPYING;md5=f08a446809913fc9b3c718f0eaea0426"
>> 
>> +BBCLASSEXTEND = "native"
>> +
>> SECTION = "unknown"
>> inherit gnome allarch
> 
> This is interesting dilemma since this is an allarch recipe. What issue
> were you seeing to need this? Could we add the allarch sysroot directory
> to PATH to resolve this instead? We could add a PROVIDES
> "hicolor-icon-theme-native" too...

XFCE needs some native tools to run, one of which has hicolor-icon-theme in its depends. Without this patch you can't build xfce. Andreas might know more about this.

regards,

Koen
Andreas Müller Nov. 10, 2011, 6:56 p.m.
On Thursday, November 10, 2011 02:38:33 PM you wrote:
> Op 10 nov. 2011, om 14:20 heeft Richard Purdie het volgende geschreven:
> > On Thu, 2011-11-10 at 13:44 +0100, Koen Kooi wrote:
> >> Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
> >> ---
> >> .../hicolor-icon-theme/hicolor-icon-theme_0.12.bb  |    2 ++
> >> 1 files changed, 2 insertions(+), 0 deletions(-)
> >> 
> >> diff --git
> >> a/meta/recipes-gnome/hicolor-icon-theme/hicolor-icon-theme_0.12.bb
> >> b/meta/recipes-gnome/hicolor-icon-theme/hicolor-icon-theme_0.12.bb
> >> index 759c9d8..aeb89c9 100644
> >> --- a/meta/recipes-gnome/hicolor-icon-theme/hicolor-icon-theme_0.12.bb
> >> +++ b/meta/recipes-gnome/hicolor-icon-theme/hicolor-icon-theme_0.12.bb
> >> @@ -5,6 +5,8 @@ BUGTRACKER = "https://bugs.freedesktop.org/"
> >> LICENSE = "GPLv2"
> >> LIC_FILES_CHKSUM = "file://COPYING;md5=f08a446809913fc9b3c718f0eaea0426"
> >> 
> >> +BBCLASSEXTEND = "native"
> >> +
> >> SECTION = "unknown"
> >> inherit gnome allarch
> > 
> > This is interesting dilemma since this is an allarch recipe. What issue
> > were you seeing to need this? Could we add the allarch sysroot directory
> > to PATH to resolve this instead? We could add a PROVIDES
> > "hicolor-icon-theme-native" too...
> 
> XFCE needs some native tools to run, one of which has hicolor-icon-theme in
> its depends. Without this patch you can't build xfce. Andreas might know
> more about this.
> 
Up to this mail I did not notice any problem but the bad guy seems xfce4-dev-
tools: Few xfce recipes depend on that and it contains only:

- m4 autoconf extensions
- two shell scripts (don't know what they are for...)

By inheriting xfce, it inherits gtk-icon-cache which seems to depend on hicolor-
icon-theme.

I don't remember what exactly my idea was, when adding native support on that.

Will make a change, build test it on fresh build dir and send a patch on meta-oe 
mailing list.

Apart from bad practice on xfce4-dev-tools: I wouldn't have expected trouble 
when a native recipe depends on an allarch one.

Andreas