[OE-core] [PATCH 3/3] gtk-icon-cache: clean up DEPENDS
Richard Purdie
richard.purdie at linuxfoundation.org
Mon Apr 1 10:44:48 UTC 2019
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 at 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
More information about the Openembedded-core
mailing list