* [PATCH 1/3] Revert "gtk+3: add librsvg to DEPENDS"
@ 2019-03-29 14:29 Ross Burton
2019-03-29 14:29 ` [PATCH 2/3] adwaita-icon-theme: depend on librsvg Ross Burton
2019-03-29 14:29 ` [PATCH 3/3] gtk-icon-cache: clean up DEPENDS Ross Burton
0 siblings, 2 replies; 4+ messages in thread
From: Ross Burton @ 2019-03-29 14:29 UTC (permalink / raw)
To: openembedded-core
This was added so that adwaita-icon-theme can find the SVG loader for icon
generation, but the fix is in the wrong place. GTK+ does not depend on librsvg
at all, the tool in GTK+ which loads the icons uses gdk-pixbuf which uses
loadable modules.
This reverts commit 66828ff04d107b7719c9c8857d7c6c2ebf20a8bb.
Signed-off-by: Ross Burton <ross.burton@intel.com>
---
meta/recipes-gnome/gtk+/gtk+3.inc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/meta/recipes-gnome/gtk+/gtk+3.inc b/meta/recipes-gnome/gtk+/gtk+3.inc
index b58641a4702..77b6c31536b 100644
--- a/meta/recipes-gnome/gtk+/gtk+3.inc
+++ b/meta/recipes-gnome/gtk+/gtk+3.inc
@@ -5,7 +5,7 @@ HOMEPAGE = "http://www.gtk.org"
BUGTRACKER = "https://bugzilla.gnome.org/"
SECTION = "libs"
-DEPENDS = "glib-2.0 cairo pango atk jpeg libpng gdk-pixbuf librsvg \
+DEPENDS = "glib-2.0 cairo pango atk jpeg libpng gdk-pixbuf \
gdk-pixbuf-native"
LICENSE = "LGPLv2 & LGPLv2+ & LGPLv2.1+"
--
2.11.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH 2/3] adwaita-icon-theme: depend on librsvg
2019-03-29 14:29 [PATCH 1/3] Revert "gtk+3: add librsvg to DEPENDS" Ross Burton
@ 2019-03-29 14:29 ` Ross Burton
2019-03-29 14:29 ` [PATCH 3/3] gtk-icon-cache: clean up DEPENDS Ross Burton
1 sibling, 0 replies; 4+ messages in thread
From: Ross Burton @ 2019-03-29 14:29 UTC (permalink / raw)
To: openembedded-core
adwaita-icon-theme uses gtk-encode-symbolic-svg to generate icons, so DEPEND on
librsvg-native (and gtk+3-native, via gtk-icon-cache.bbclass) so that this tool
exists and can load SVG files.
Signed-off-by: Ross Burton <ross.burton@intel.com>
---
meta/recipes-gnome/gnome/adwaita-icon-theme_3.30.1.bb | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/meta/recipes-gnome/gnome/adwaita-icon-theme_3.30.1.bb b/meta/recipes-gnome/gnome/adwaita-icon-theme_3.30.1.bb
index 451ef39f1ad..8eadbd2eef7 100644
--- a/meta/recipes-gnome/gnome/adwaita-icon-theme_3.30.1.bb
+++ b/meta/recipes-gnome/gnome/adwaita-icon-theme_3.30.1.bb
@@ -8,7 +8,6 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=c84cac88e46fc07647ea07e6c24eeb7c"
inherit allarch autotools pkgconfig gettext gtk-icon-cache upstream-version-is-even
-
MAJ_VER = "${@oe.utils.trim_version("${PV}", 2)}"
SRC_URI = "${GNOME_MIRROR}/${BPN}/${MAJ_VER}/${BPN}-${PV}.tar.xz \
file://0001-Don-t-use-AC_CANONICAL_HOST.patch \
@@ -18,6 +17,8 @@ SRC_URI = "${GNOME_MIRROR}/${BPN}/${MAJ_VER}/${BPN}-${PV}.tar.xz \
SRC_URI[md5sum] = "db3fd812821d72fdd9a3c7d622a95b35"
SRC_URI[sha256sum] = "6d752a2b1bc668483956d4485c39cad1642d9358e133ff689526e43674a4e1ce"
+DEPENDS += "librsvg-native"
+
PACKAGES = "${PN}-cursors ${PN}-symbolic-hires ${PN}-symbolic ${PN}-hires ${PN}"
RREPLACES_${PN} = "gnome-icon-theme"
--
2.11.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH 3/3] gtk-icon-cache: clean up DEPENDS
2019-03-29 14:29 [PATCH 1/3] Revert "gtk+3: add librsvg to DEPENDS" Ross Burton
2019-03-29 14:29 ` [PATCH 2/3] adwaita-icon-theme: depend on librsvg Ross Burton
@ 2019-03-29 14:29 ` Ross Burton
2019-04-01 10:44 ` Richard Purdie
1 sibling, 1 reply; 4+ messages in thread
From: Ross Burton @ 2019-03-29 14:29 UTC (permalink / raw)
To: openembedded-core
Instead of using DEPENDS += use _append to ensure that these additions can't get
overwritten.
Also use gtk+3-native instead gtk-icon-utils-native as that recipe no longer
exists and is provided by gtk+3-native for compatibility.
Signed-off-by: Ross Burton <ross.burton@intel.com>
---
meta/classes/gtk-icon-cache.bbclass | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/meta/classes/gtk-icon-cache.bbclass b/meta/classes/gtk-icon-cache.bbclass
index d87167aec09..3572f62ebb6 100644
--- a/meta/classes/gtk-icon-cache.bbclass
+++ b/meta/classes/gtk-icon-cache.bbclass
@@ -1,8 +1,8 @@
FILES_${PN} += "${datadir}/icons/hicolor"
-DEPENDS += "${@['hicolor-icon-theme', '']['${BPN}' == 'hicolor-icon-theme']} gtk-icon-utils-native"
+DEPENDS_append = " ${@['hicolor-icon-theme', '']['${BPN}' == 'hicolor-icon-theme']} gtk+3-native"
-PACKAGE_WRITE_DEPS += "gtk-icon-utils-native gdk-pixbuf-native"
+PACKAGE_WRITE_DEPS += "gtk+3-native gdk-pixbuf-native"
gtk_icon_cache_postinst() {
if [ "x$D" != "x" ]; then
--
2.11.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH 3/3] gtk-icon-cache: clean up DEPENDS
2019-03-29 14:29 ` [PATCH 3/3] gtk-icon-cache: clean up DEPENDS Ross Burton
@ 2019-04-01 10:44 ` Richard Purdie
0 siblings, 0 replies; 4+ messages in thread
From: Richard Purdie @ 2019-04-01 10:44 UTC (permalink / raw)
To: Ross Burton, openembedded-core
On Fri, 2019-03-29 at 14:29 +0000, Ross Burton wrote:
> Instead of using DEPENDS += use _append to ensure that these additions can't get
> overwritten.
>
> Also use gtk+3-native instead gtk-icon-utils-native as that recipe no longer
> exists and is provided by gtk+3-native for compatibility.
>
> Signed-off-by: Ross Burton <ross.burton@intel.com>
> ---
> meta/classes/gtk-icon-cache.bbclass | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/meta/classes/gtk-icon-cache.bbclass b/meta/classes/gtk-icon-cache.bbclass
> index d87167aec09..3572f62ebb6 100644
> --- a/meta/classes/gtk-icon-cache.bbclass
> +++ b/meta/classes/gtk-icon-cache.bbclass
> @@ -1,8 +1,8 @@
> FILES_${PN} += "${datadir}/icons/hicolor"
>
> -DEPENDS += "${@['hicolor-icon-theme', '']['${BPN}' == 'hicolor-icon-theme']} gtk-icon-utils-native"
> +DEPENDS_append = " ${@['hicolor-icon-theme', '']['${BPN}' == 'hicolor-icon-theme']} gtk+3-native"
I don't mind the changes themselves but I do have a bit of an issue
with changing to use DEPENDS_append.
In general recipes should be doing DEPENDS += "xxx", unless its at the
start of the recipe before any inherits (and even then += is fine).
_append is much harder to override and whilst it can be needed (e.g.
the hoops base.bbclass has to jump through), I'd much prefer to
encourage people to use +=.
I appreciate its a style thing and there is no "right" answer but its
one do have a fairly strong preference over, mainly for the
overridability. Our _append/_remove/_prepend operators are operators of
last resort to me.
Cheers,
Richard
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-04-01 10:44 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-03-29 14:29 [PATCH 1/3] Revert "gtk+3: add librsvg to DEPENDS" Ross Burton
2019-03-29 14:29 ` [PATCH 2/3] adwaita-icon-theme: depend on librsvg Ross Burton
2019-03-29 14:29 ` [PATCH 3/3] gtk-icon-cache: clean up DEPENDS Ross Burton
2019-04-01 10:44 ` Richard Purdie
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.