All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2 0/2] glib-2.0 changes for nativesdk and packaging
@ 2017-02-22  5:58 Nathan Rossi
  2017-02-22  5:58 ` [PATCH v2 1/2] glib-2.0/glib.inc: Only apply MLPREFIX renaming to class-target Nathan Rossi
  2017-02-22  5:58 ` [PATCH v2 2/2] glib-2.0/glib.inc: If 'charset.alias' exists drop it Nathan Rossi
  0 siblings, 2 replies; 3+ messages in thread
From: Nathan Rossi @ 2017-02-22  5:58 UTC (permalink / raw)
  To: openembedded-core; +Cc: Alistair Francis

This series contains two patches. The first fixes up the renaming of
gio-querymodules so that it is only applied to class-target (so that
'nativesdk-gio-query-modules' is not generated). The second change makes
the removal of 'charset.alias' occur for all targets including musl
since it is not packaged by default and not desired in cases that it is
generated.
---
Changes in v2:
 * Rebased on top of current master


Nathan Rossi (2):
  glib-2.0/glib.inc: Only apply MLPREFIX renaming to class-target
  glib-2.0/glib.inc: If 'charset.alias' exists drop it

 meta/recipes-core/glib-2.0/glib.inc | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

-- 
2.11.0


^ permalink raw reply	[flat|nested] 3+ messages in thread

* [PATCH v2 1/2] glib-2.0/glib.inc: Only apply MLPREFIX renaming to class-target
  2017-02-22  5:58 [PATCH v2 0/2] glib-2.0 changes for nativesdk and packaging Nathan Rossi
@ 2017-02-22  5:58 ` Nathan Rossi
  2017-02-22  5:58 ` [PATCH v2 2/2] glib-2.0/glib.inc: If 'charset.alias' exists drop it Nathan Rossi
  1 sibling, 0 replies; 3+ messages in thread
From: Nathan Rossi @ 2017-02-22  5:58 UTC (permalink / raw)
  To: openembedded-core; +Cc: Alistair Francis

When renaming the 'gio-querymodules' binary to prefix with the current
MLPREFIX, only do this for target packages where the MLPREFIX makes
sense. This skips renaming for nativesdk targets which have MLPREFIX set
to 'nativesdk-'.

Signed-off-by: Nathan Rossi <nathan@nathanrossi.com>
---
 meta/recipes-core/glib-2.0/glib.inc | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/meta/recipes-core/glib-2.0/glib.inc b/meta/recipes-core/glib-2.0/glib.inc
index a5db455248..bbac7ebd32 100644
--- a/meta/recipes-core/glib-2.0/glib.inc
+++ b/meta/recipes-core/glib-2.0/glib.inc
@@ -91,11 +91,6 @@ do_install_append () {
 	if [ -f ${D}${bindir}/glib-mkenums ]; then
 		sed -i -e '1s,#!.*perl,#! ${USRBINPATH}/env perl,' ${D}${bindir}/glib-mkenums
 	fi
-
-        # Make sure gio-querymodules is unique among multilibs
-        if test "x${MLPREFIX}" != "x"; then
-                mv ${D}${libexecdir}/gio-querymodules ${D}${libexecdir}/${MLPREFIX}gio-querymodules
-        fi
 }
 
 do_install_append_class-target () {
@@ -106,6 +101,11 @@ do_install_append_class-target () {
 			rm ${D}${datadir}/installed-tests/glib/gdbus-serialization.test
 		fi
 	fi
+
+	# Make sure gio-querymodules is unique among multilibs
+	if test "x${MLPREFIX}" != "x"; then
+		mv ${D}${libexecdir}/gio-querymodules ${D}${libexecdir}/${MLPREFIX}gio-querymodules
+	fi
 }
 
 do_install_append_libc-musl () {
-- 
2.11.0



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [PATCH v2 2/2] glib-2.0/glib.inc: If 'charset.alias' exists drop it
  2017-02-22  5:58 [PATCH v2 0/2] glib-2.0 changes for nativesdk and packaging Nathan Rossi
  2017-02-22  5:58 ` [PATCH v2 1/2] glib-2.0/glib.inc: Only apply MLPREFIX renaming to class-target Nathan Rossi
@ 2017-02-22  5:58 ` Nathan Rossi
  1 sibling, 0 replies; 3+ messages in thread
From: Nathan Rossi @ 2017-02-22  5:58 UTC (permalink / raw)
  To: openembedded-core; +Cc: Alistair Francis

Delete the 'charset.alias' file during the do_install task if it exists.
This file is not shipped in any packages and on the targets that it is
generated (musl and mingw32) it is undesired.

Signed-off-by: Nathan Rossi <nathan@nathanrossi.com>
---
 meta/recipes-core/glib-2.0/glib.inc | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/meta/recipes-core/glib-2.0/glib.inc b/meta/recipes-core/glib-2.0/glib.inc
index bbac7ebd32..6600d33dcd 100644
--- a/meta/recipes-core/glib-2.0/glib.inc
+++ b/meta/recipes-core/glib-2.0/glib.inc
@@ -91,6 +91,10 @@ do_install_append () {
 	if [ -f ${D}${bindir}/glib-mkenums ]; then
 		sed -i -e '1s,#!.*perl,#! ${USRBINPATH}/env perl,' ${D}${bindir}/glib-mkenums
 	fi
+
+	if [ -e ${D}${libdir}/charset.alias ]; then
+		rm -f ${D}${libdir}/charset.alias
+	fi
 }
 
 do_install_append_class-target () {
@@ -108,10 +112,6 @@ do_install_append_class-target () {
 	fi
 }
 
-do_install_append_libc-musl () {
-	rm -f ${D}${libdir}/charset.alias
-}
-
 RDEPENDS_${PN}-ptest += "\
             gnome-desktop-testing \
             tzdata \
-- 
2.11.0



^ permalink raw reply related	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2017-02-22  5:59 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-02-22  5:58 [PATCH v2 0/2] glib-2.0 changes for nativesdk and packaging Nathan Rossi
2017-02-22  5:58 ` [PATCH v2 1/2] glib-2.0/glib.inc: Only apply MLPREFIX renaming to class-target Nathan Rossi
2017-02-22  5:58 ` [PATCH v2 2/2] glib-2.0/glib.inc: If 'charset.alias' exists drop it Nathan Rossi

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.