[PATCH 1/2] hicolor-icon-theme: keep inherited dependencies

[PATCH 1/2] hicolor-icon-theme: keep inherited dependencies

[Eric Bénard]

else, if an image installs hicolor-icon-theme without any other
gnome package, the dependency on gdk-pixbuf-native will be missing
and if it exists, the host gdk-pixbuf-query-loaders binary will be
used and it will try to update the host's cache

[YOCTO #4572]

Signed-off-by: Eric Bénard <eric@eukrea.com>
---
this patch will need some testing as it's not clear to me why this dependency was forced to "". At least that fix the problem described in the bug report.
 meta/recipes-gnome/hicolor-icon-theme/hicolor-icon-theme_0.12.bb | 2 --
 1 file changed, 2 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 7c911e0..8d48dec 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
@@ -8,8 +8,6 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=f08a446809913fc9b3c718f0eaea0426"
 SECTION = "unknown"
 inherit gnome allarch
 
-DEPENDS = ""
-
 PR = "r1"
 
 SRC_URI = "http://icon-theme.freedesktop.org/releases/${BPN}-${PV}.tar.gz \
-- 
1.8.1.4

[PATCH 2/2] gtk-icon-cache pixbufcache: add a clear dependency on gdk-pixbuf-native

[Eric Bénard]

not really sure if the dependency is always inherited but at least
there won't be bad surprise.

Signed-off-by: Eric Bénard <eric@eukrea.com>
---
 meta/classes/gtk-icon-cache.bbclass | 2 +-
 meta/classes/pixbufcache.bbclass    | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/classes/gtk-icon-cache.bbclass b/meta/classes/gtk-icon-cache.bbclass
index 789fa38..8ed7feb 100644
--- a/meta/classes/gtk-icon-cache.bbclass
+++ b/meta/classes/gtk-icon-cache.bbclass
@@ -1,6 +1,6 @@
 FILES_${PN} += "${datadir}/icons/hicolor"
 
-DEPENDS += "${@['hicolor-icon-theme', '']['${BPN}' == 'hicolor-icon-theme']} gtk-update-icon-cache-native"
+DEPENDS += "${@['hicolor-icon-theme', '']['${BPN}' == 'hicolor-icon-theme']} gtk-update-icon-cache-native gdk-pixbuf-native"
 
 gtk_icon_cache_postinst() {
 if [ "x$D" != "x" ]; then
diff --git a/meta/classes/pixbufcache.bbclass b/meta/classes/pixbufcache.bbclass
index 274d67d..4ac17ee 100644
--- a/meta/classes/pixbufcache.bbclass
+++ b/meta/classes/pixbufcache.bbclass
@@ -3,7 +3,7 @@
 # packages.
 #
 
-DEPENDS += "qemu-native"
+DEPENDS += "qemu-native gdk-pixbuf-native"
 inherit qemu
 
 PIXBUF_PACKAGES ??= "${PN}"
-- 
1.8.1.4

Re: [PATCH 1/2] hicolor-icon-theme: keep inherited dependencies

[Martin Jansa]

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

Doesn't it depend now on something which isn't allarch? IIRC gnome.bbclass
brings gconf.


On Thu, May 30, 2013 at 9:44 AM, Eric Bénard <eric@eukrea.com> wrote:

> else, if an image installs hicolor-icon-theme without any other
> gnome package, the dependency on gdk-pixbuf-native will be missing
> and if it exists, the host gdk-pixbuf-query-loaders binary will be
> used and it will try to update the host's cache
>
> [YOCTO #4572]
>
> Signed-off-by: Eric Bénard <eric@eukrea.com>
> ---
> this patch will need some testing as it's not clear to me why this
> dependency was forced to "". At least that fix the problem described in the
> bug report.
>  meta/recipes-gnome/hicolor-icon-theme/hicolor-icon-theme_0.12.bb | 2 --
>  1 file changed, 2 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 7c911e0..8d48dec 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
> @@ -8,8 +8,6 @@ LIC_FILES_CHKSUM =
> "file://COPYING;md5=f08a446809913fc9b3c718f0eaea0426"
>  SECTION = "unknown"
>  inherit gnome allarch
>
> -DEPENDS = ""
> -
>  PR = "r1"
>
>  SRC_URI = "http://icon-theme.freedesktop.org/releases/${BPN}-${PV}.tar.gz\
> --
> 1.8.1.4
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>

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

Re: [PATCH 1/2] hicolor-icon-theme: keep inherited dependencies

[Burton, Ross]

On 30 May 2013 09:09, Martin Jansa <martin.jansa@gmail.com> wrote:
> Doesn't it depend now on something which isn't allarch? IIRC gnome.bbclass
> brings gconf.

gnome.bbclass is an abomination and should be gutted.  Yes, this will
mean the build dependencies are massive, and I can't see why this
recipe needs gnome.bbclass.  Then again it doesn't need gtk-icon-cache
either as it doesn't actually have any icons in.  I'd be interested in
seeing a buildhistory-diff between the current recipe and one with
both inherit gnome and DEPENDS="" removed.

Ross

Re: [PATCH 1/2] hicolor-icon-theme: keep inherited dependencies

[Eric Bénard]

Le Thu, 30 May 2013 09:32:38 +0100,
"Burton, Ross" <ross.burton@intel.com> a écrit :

> On 30 May 2013 09:09, Martin Jansa <martin.jansa@gmail.com> wrote:
> > Doesn't it depend now on something which isn't allarch? IIRC gnome.bbclass
> > brings gconf.
> 
> gnome.bbclass is an abomination and should be gutted.  Yes, this will
> mean the build dependencies are massive, and I can't see why this
> recipe needs gnome.bbclass.  Then again it doesn't need gtk-icon-cache
> either as it doesn't actually have any icons in.  I'd be interested in
> seeing a buildhistory-diff between the current recipe and one with
> both inherit gnome and DEPENDS="" removed.
> 
when  removing gnome I get the following error at install :
DEBUG: Executing shell function do_install
install: cannot create regular file
`/scratch/setup-scripts/build/tmp-defaultsetup-eglibc-eglibc/work/all-oe-li\
nux/hicolor-icon-theme/0.12-r1/image//usr/share/icons/hicolor': No such
file or directory

Eric

Re: [PATCH 1/2] hicolor-icon-theme: keep inherited dependencies

[Eric Bénard]

Le Thu, 30 May 2013 10:09:36 +0200,
Martin Jansa <martin.jansa@gmail.com> a écrit :

> Doesn't it depend now on something which isn't allarch? IIRC gnome.bbclass
> brings gconf.
> 
true :
  * DEPENDS: added "shared-mime-info gnome-common-native
shared-mime-info-native gtk-update-icon-cache-native gconf gconf-native"

Eric

Re: [PATCH 1/2] hicolor-icon-theme: keep inherited dependencies

[Eric Bénard]

Hi Ross,

Le Thu, 30 May 2013 12:32:30 +0200,
Eric Bénard <eric@eukrea.com> a écrit :

> Le Thu, 30 May 2013 09:32:38 +0100,
> "Burton, Ross" <ross.burton@intel.com> a écrit :
> 
> > On 30 May 2013 09:09, Martin Jansa <martin.jansa@gmail.com> wrote:
> > > Doesn't it depend now on something which isn't allarch? IIRC gnome.bbclass
> > > brings gconf.
> > 
> > gnome.bbclass is an abomination and should be gutted.  Yes, this will
> > mean the build dependencies are massive, and I can't see why this
> > recipe needs gnome.bbclass.  Then again it doesn't need gtk-icon-cache
> > either as it doesn't actually have any icons in.  I'd be interested in
> > seeing a buildhistory-diff between the current recipe and one with
> > both inherit gnome and DEPENDS="" removed.
> > 
> when  removing gnome I get the following error at install :
> DEBUG: Executing shell function do_install
> install: cannot create regular file
> `/scratch/setup-scripts/build/tmp-defaultsetup-eglibc-eglibc/work/all-oe-li\
> nux/hicolor-icon-theme/0.12-r1/image//usr/share/icons/hicolor': No such
> file or directory
> 
turning on the brain and fixing do_install, the diff between original et
new (without gnome and DEPENDS="") is :

packages/all-oe-linux/hicolor-icon-theme/hicolor-icon-theme:
pkg_postinst cleared:
packages/all-oe-linux/hicolor-icon-theme/hicolor-icon-theme: pkg_postrm
cleared:

So that seems the right thing to do don't you think ?

Eric

Re: [PATCH 1/2] hicolor-icon-theme: keep inherited dependencies

[Richard Purdie]

On Thu, 2013-05-30 at 12:32 +0200, Eric Bénard wrote:
> Le Thu, 30 May 2013 09:32:38 +0100,
> "Burton, Ross" <ross.burton@intel.com> a écrit :
> 
> > On 30 May 2013 09:09, Martin Jansa <martin.jansa@gmail.com> wrote:
> > > Doesn't it depend now on something which isn't allarch? IIRC gnome.bbclass
> > > brings gconf.
> > 
> > gnome.bbclass is an abomination and should be gutted.  Yes, this will
> > mean the build dependencies are massive, and I can't see why this
> > recipe needs gnome.bbclass.  Then again it doesn't need gtk-icon-cache
> > either as it doesn't actually have any icons in.  I'd be interested in
> > seeing a buildhistory-diff between the current recipe and one with
> > both inherit gnome and DEPENDS="" removed.
> > 
> when  removing gnome I get the following error at install :
> DEBUG: Executing shell function do_install
> install: cannot create regular file
> `/scratch/setup-scripts/build/tmp-defaultsetup-eglibc-eglibc/work/all-oe-li\
> nux/hicolor-icon-theme/0.12-r1/image//usr/share/icons/hicolor': No such
> file or directory

I have a suspicion we really want a gnomebase inherit in the recipe
instead of gnome and that might solve serveral issues in one.

Cheers,

Richard

Re: [PATCH 1/2] hicolor-icon-theme: keep inherited dependencies

[Eric Bénard]

Le Thu, 30 May 2013 11:49:00 +0100,
Richard Purdie <richard.purdie@linuxfoundation.org> a écrit :

> On Thu, 2013-05-30 at 12:32 +0200, Eric Bénard wrote:
> > Le Thu, 30 May 2013 09:32:38 +0100,
> > "Burton, Ross" <ross.burton@intel.com> a écrit :
> > 
> > > On 30 May 2013 09:09, Martin Jansa <martin.jansa@gmail.com> wrote:
> > > > Doesn't it depend now on something which isn't allarch? IIRC gnome.bbclass
> > > > brings gconf.
> > > 
> > > gnome.bbclass is an abomination and should be gutted.  Yes, this will
> > > mean the build dependencies are massive, and I can't see why this
> > > recipe needs gnome.bbclass.  Then again it doesn't need gtk-icon-cache
> > > either as it doesn't actually have any icons in.  I'd be interested in
> > > seeing a buildhistory-diff between the current recipe and one with
> > > both inherit gnome and DEPENDS=""DEPENDS="" removed.
> > > 
> > when  removing gnome I get the following error at install :
> > DEBUG: Executing shell function do_install
> > install: cannot create regular file
> > `/scratch/setup-scripts/build/tmp-defaultsetup-eglibc-eglibc/work/all-oe-li\
> > nux/hicolor-icon-theme/0.12-r1/image//usr/share/icons/hicolor': No such
> > file or directory
> 
> I have a suspicion we really want a gnomebase inherit in the recipe
> instead of gnome and that might solve serveral issues in one.
> 
inherit gnomebase and remove DEPENDS=""
seems to work fine (the postinst and postrm scripts are removed).
removing DEPENDS="" is not a big issue as it only brings native
packages :
DEPENDS = autoconf-native automake-native gnome-common-native
gnu-config-native libtool-native pkgconfig-native

Eric

Re: [PATCH 1/2] hicolor-icon-theme: keep inherited dependencies

[Burton, Ross]

On 30 May 2013 12:11, Eric Bénard <eric@eukrea.com> wrote:
>> I have a suspicion we really want a gnomebase inherit in the recipe
>> instead of gnome and that might solve serveral issues in one.
>>
> inherit gnomebase and remove DEPENDS=""
> seems to work fine (the postinst and postrm scripts are removed).
> removing DEPENDS="" is not a big issue as it only brings native
> packages :
> DEPENDS = autoconf-native automake-native gnome-common-native
> gnu-config-native libtool-native pkgconfig-native

Sounds fine, they actually are the build-dependencies.

Ross

Re: [PATCH 1/2] hicolor-icon-theme: keep inherited dependencies

[Burton, Ross]

On 30 May 2013 12:11, Eric Bénard <eric@eukrea.com> wrote:
> inherit gnomebase and remove DEPENDS=""
> seems to work fine (the postinst and postrm scripts are removed).
> removing DEPENDS="" is not a big issue as it only brings native
> packages :
> DEPENDS = autoconf-native automake-native gnome-common-native
> gnu-config-native libtool-native pkgconfig-native

Will you send a patch for this?

Ross

Re: [PATCH 1/2] hicolor-icon-theme: keep inherited dependencies

[Eric Bénard]

Le Thu, 30 May 2013 19:26:03 +0100,
"Burton, Ross" <ross.burton@intel.com> a écrit :

> On 30 May 2013 12:11, Eric Bénard <eric@eukrea.com> wrote:
> > inherit gnomebase and remove DEPENDS=""
> > seems to work fine (the postinst and postrm scripts are removed).
> > removing DEPENDS="" is not a big issue as it only brings native
> > packages :
> > DEPENDS = autoconf-native automake-native gnome-common-native
> > gnu-config-native libtool-native pkgconfig-native
> 
> Will you send a patch for this?
> 
yes it's coming ;-)

Eric