On 04.10.21 17:08, Peter Kjellerstedt wrote:
-----Original Message-----
From: openembedded-core@lists.openembedded.org <openembedded-
c...@lists.openembedded.org> On Behalf Of Konrad Weihmann
Sent: den 4 oktober 2021 15:29
To: openembedded-core@lists.openembedded.org
Cc: Konrad Weihmann <kweihm...@outlook.com>
Subject: [OE-core] [PATCH 2/3] glibc: fix append in DEPENDS
fix the append with a leading blank, otherwise this messes up
DEPENDS when globally inherited classes add dependecies of their own
Signed-off-by: Konrad Weihmann <kweihm...@outlook.com>
---
meta/recipes-core/glib-2.0/glib.inc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/meta/recipes-core/glib-2.0/glib.inc b/meta/recipes-core/glib-
2.0/glib.inc
index a34776b083..3436645f15 100644
--- a/meta/recipes-core/glib-2.0/glib.inc
+++ b/meta/recipes-core/glib-2.0/glib.inc
@@ -30,7 +30,7 @@ LEAD_SONAME = "libglib-2.0.*"
inherit meson gettext gtk-doc pkgconfig ptest-gnome upstream-version-is-
even bash-completion gio-module-cache manpages
-DEPENDS:append:class-target = "${@' gtk-doc' if d.getVar('GTKDOC_ENABLED') ==
'True' else ''}"
+DEPENDS:append:class-target = " ${@' gtk-doc' if d.getVar('GTKDOC_ENABLED') ==
'True' else ''}"
This change is not needed as there already is a space inside
the if statement (in ' gtk-doc').
Still this is very prone to errors, so I personally rather have a space
too much than to few (esp. as the else clause doesn't add a space).
IHMO it should be for readability purpose rather " ${@'gtk-doc' if
d.getVar('GTKDOC_ENABLED') == 'True' else ''}" than like it is now - but
that's just cosmetics and I'm sure one could discuss about that a lot
GTKDOC_MESON_OPTION = "gtk_doc"
--
2.25.1
//Peter
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#156611):
https://lists.openembedded.org/g/openembedded-core/message/156611
Mute This Topic: https://lists.openembedded.org/mt/86066304/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
